Opened 14 years ago

Closed 14 years ago

#6098 closed Bug (fixed)

It is possible to change the direction of a readonly element

Reported by: Sa'ar Zac Elias Owned by: Frederico Caldeira Knabben
Priority: Normal Milestone: CKEditor 3.4
Component: General Version: SVN (CKEditor) - OLD
Keywords: Cc:

Description

  • Load the following:
    <p contenteditable="false">
    	aa</p>
    
  • Click on the paragraph and click on one of the bidi icons.

Attachments (3)

6098.patch (596 bytes) - added by Sa'ar Zac Elias 14 years ago.
6098_2.patch (801 bytes) - added by Frederico Caldeira Knabben 14 years ago.
6098_3.patch (1.0 KB) - added by Frederico Caldeira Knabben 14 years ago.

Download all attachments as: .zip

Change History (11)

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

Owner: set to Sa'ar Zac Elias
Status: newassigned

Changed 14 years ago by Sa'ar Zac Elias

Attachment: 6098.patch added

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

Status: assignedreview

comment:3 Changed 14 years ago by Frederico Caldeira Knabben

Status: reviewreview_failed
  1. Load this HTML:
<p>This is some <strong contenteditable="false">sample text</strong> for testing.</p>
  1. CTRL+A to select all.
  2. Apply the RTL direction.

Nothing happens.

Changed 14 years ago by Frederico Caldeira Knabben

Attachment: 6098_2.patch added

comment:4 Changed 14 years ago by Frederico Caldeira Knabben

Owner: changed from Sa'ar Zac Elias to Frederico Caldeira Knabben
Status: review_failedreview

The length check is still valid, so I've included it also.

comment:5 Changed 14 years ago by Sa'ar Zac Elias

Status: reviewreview_failed

The patch doesn't work for the inital TC.

Changed 14 years ago by Frederico Caldeira Knabben

Attachment: 6098_3.patch added

comment:6 Changed 14 years ago by Frederico Caldeira Knabben

Status: review_failedreview

comment:7 Changed 14 years ago by Sa'ar Zac Elias

Status: reviewreview_passed

comment:8 Changed 14 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: review_passedclosed

Fixed with [5800].

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