Ticket #2349 (closed Bug: fixed)

Opened 20 months ago

Last modified 17 months ago

[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

fck_rtl_dialog_ie6bug.png Download (13.7 KB) - added by Dekel 20 months ago.
rtl_dialog.patch Download (0.7 KB) - added by Dekel 18 months ago.
Fix for this bug
2349.patch Download (1.3 KB) - added by martinkou 17 months ago.

Change History

Changed 20 months ago by Dekel

Changed 20 months ago by w.olchawa

  • keywords Confirmed IE added; dialog,rtl,hebrew removed
  • version set to FCKeditor 2.6.3 Beta
  • summary changed from New dialog system is broken with rtl on ie6 to [IE] Dialog system is broken with rtl

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

Changed 20 months 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 18 months ago by Dekel

Fix for this bug

Changed 18 months 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).

Changed 18 months ago by alfonsoml

  • keywords HasPatch added

Changed 18 months ago by fredck

  • milestone set to FCKeditor 2.6.4

Changed 17 months ago by martinkou

  • owner set to martinkou
  • status changed from new to assigned

Changed 17 months ago by martinkou

Changed 17 months ago by martinkou

  • keywords Review? added

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

Changed 17 months ago by alfonsoml

  • keywords Review+ added; HasPatch Review? removed

Changed 17 months ago by martinkou

  • status changed from assigned to closed
  • resolution set to fixed

Fixed with [2560].

Click here for more info about our SVN system.

Note: See TracTickets for help on using tickets.