Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#6461 closed Bug (fixed)

BIDI: Paragraph alignment lost when switching list to paragraph

Reported by: Damian Owned by: Sa'ar Zac Elias
Priority: Normal Milestone: CKEditor 3.4.2
Component: General Version:
Keywords: IBM Cc: joek, Satya Minnekanti, james, c

Description

When a list is unset, the paragraph alignment for each list item is lost.

To reproduce:

  1. Create the following sample content:
      <p>Paragraph 1</p>
      <p style="text-align: center;">Paragraph 2</p>
      <p style="text-align: right;">Paragraph 3</p>
    
  1. Select the paragraphs and click list button to make a list

At this stage the markup is:

<ul>
	<li>Paragraph 1</li>
	<li style="text-align: center;">Paragraph 2</li>
	<li style="text-align: right;">Paragraph 3</li>
</ul>
  1. Select the list and click list button again to undo list

Result:

  <p>Paragraph 1</p>
  <p>Paragraph 2</p>
  <p>Paragraph 3</p>

Expected:

  <p>Paragraph 1</p>
  <p style="text-align: center;">Paragraph 2</p>
  <p style="text-align: right;">Paragraph 3</p>

Attachments (2)

6461.patch (589 bytes) - added by Sa'ar Zac Elias 13 years ago.
6461_2.patch (1.4 KB) - added by Sa'ar Zac Elias 13 years ago.

Download all attachments as: .zip

Change History (9)

comment:1 Changed 13 years ago by Sa'ar Zac Elias

Status: newconfirmed

Not much of a bidi thing though.

Changed 13 years ago by Sa'ar Zac Elias

Attachment: 6461.patch added

comment:2 Changed 13 years ago by Sa'ar Zac Elias

Milestone: CKEditor 3.4.2
Owner: set to Sa'ar Zac Elias
Status: confirmedreview

comment:3 Changed 13 years ago by Garry Yao

Status: reviewreview_failed

The change must happen around L115 instead of the position in the patch, otherwise it will breaks enterMode = BR.

Changed 13 years ago by Sa'ar Zac Elias

Attachment: 6461_2.patch added

comment:4 Changed 13 years ago by Sa'ar Zac Elias

Status: review_failedreview

comment:5 Changed 13 years ago by Garry Yao

Status: reviewreview_passed

comment:6 Changed 13 years ago by Sa'ar Zac Elias

Resolution: fixed
Status: review_passedclosed

Fixed with [5941].

comment:7 Changed 12 years ago by Frederico Caldeira Knabben

Cc: james c added; James removed
Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy