Opened 8 years ago

Closed 8 years ago

#2349 closed Bug (fixed)

[IE] Dialog system is broken with rtl

Reported by: Dekel Owned by: martinkou
Priority: Normal Milestone: FCKeditor 2.6.4
Component: UI : Dialogs Version: FCKeditor 2.6.3 Beta
Keywords: Confirmed IE Review+ Cc:

Description

When using RTL language on IE6, the new dialog system is broken (there are two scroll bars, and some of the dialog is missing).

The image attached is using the nightly build, with IE6 and 'he' in the accept_language.
Can be done also with:

FCKConfig.AutoDetectLanguage	= false ;
FCKConfig.DefaultLanguage	= 'he' ;

in the fckconfig.js file.

Attachments (3)

fck_rtl_dialog_ie6bug.png (13.7 KB) - added by Dekel 8 years ago.
rtl_dialog.patch (734 bytes) - added by Dekel 8 years ago.
Fix for this bug
2349.patch (1.3 KB) - added by martinkou 8 years ago.

Download all attachments as: .zip

Change History (12)

Changed 8 years ago by Dekel

comment:1 Changed 8 years ago by w.olchawa

  • Keywords Confirmed IE added; dialog rtl hebrew removed
  • Summary changed from New dialog system is broken with rtl on ie6 to [IE] Dialog system is broken with rtl
  • Version set to FCKeditor 2.6.3 Beta

Confirmed using FCKeditor 2.6.3 beta and the latest SVN version in IE6. IE7 and FF seem no to be affected.

comment:2 Changed 8 years ago by martinkou

The bug happened from 2.6.1 but didn't happen in 2.6. So some changes in 2.6.1 must have caused this.

Changed 8 years ago by Dekel

Fix for this bug

comment:3 Changed 8 years ago by Dekel

Actually the bug does happen in 2.6 (and even in 2.6beta). Attached a patch that fixes this bug. The patch was checked with the following browsers:

  • ie6 - he/en
  • ie7 - he/en
  • ff2 - he/en
  • ff3 - he/en

(all of those on a windows xp machine).

comment:4 Changed 8 years ago by alfonsoml

  • Keywords HasPatch added

comment:5 Changed 8 years ago by fredck

  • Milestone set to FCKeditor 2.6.4

comment:6 Changed 8 years ago by martinkou

  • Owner set to martinkou
  • Status changed from new to assigned

Changed 8 years ago by martinkou

comment:7 Changed 8 years ago by martinkou

  • Keywords Review? added

Thanks Dekel, your patch works. I've only added the change message to the new patch.

comment:8 Changed 8 years ago by alfonsoml

  • Keywords Review+ added; HasPatch Review? removed

comment:9 Changed 8 years ago by martinkou

  • Resolution set to fixed
  • Status changed from assigned to closed

Fixed with [2560].

Click here for more info about our SVN system.

Note: See TracTickets for help on using tickets.
© 2003 – 2016 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy