Ticket #3767 (closed Bug: fixed)

Opened 5 years ago

Last modified 5 years ago

IE: Second level menu may not have borders

Reported by: fredck Owned by: tobiasz.cudnik
Priority: Normal Milestone: CKEditor 3.0
Component: UI : Context Menu Version: SVN (CKEditor) - OLD
Keywords: Confirmed IE 3.0RC Review+ Cc:

Description

In IE, the second level menus may be shown without border. The border appears as soon as we do some rollover in the menu (not always though).

This is a but random, but I'm able to constantly reproduce with the SCAYT plugin. It looks like a pure IE rendering bug. We should find a workaround for it.

Attachments

IE7_Screenshot.png (28.8 KB) - added by fredck 5 years ago.
IE7 screenshot showing the issue
3767.patch (2.2 KB) - added by tobiasz.cudnik 5 years ago.
3767_2.patch (681 bytes) - added by tobiasz.cudnik 5 years ago.
3767_3.patch (731 bytes) - added by tobiasz.cudnik 5 years ago.

Change History

Changed 5 years ago by fredck

IE7 screenshot showing the issue

comment:1 Changed 5 years ago by fredck

  • Keywords 3.0RC added

comment:2 Changed 5 years ago by tobiasz.cudnik

  • Status changed from new to assigned
  • Owner set to tobiasz.cudnik

Changed 5 years ago by tobiasz.cudnik

comment:3 Changed 5 years ago by tobiasz.cudnik

  • Keywords Review? added

comment:4 Changed 5 years ago by fredck

  • Keywords Review- added; Review? removed

Holly hack! But it looks like something else ended up by mistake into the patch.

Also, can you improve the coding style a bit?

Changed 5 years ago by tobiasz.cudnik

comment:5 Changed 5 years ago by tobiasz.cudnik

  • Keywords Review? added; Review- removed

Changed 5 years ago by tobiasz.cudnik

comment:6 Changed 5 years ago by fredck

  • Keywords Review+ added; Review? removed

Please try the following option before committing:

panel.element.getChild( 0 ).$.style.cssText += '';

comment:7 Changed 5 years ago by tobiasz.cudnik

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

Fixed with [3739] using Frederico's suggestion.

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