Opened 14 years ago

Closed 14 years ago

#5452 closed Bug (expired)

[IE+Quirks] Unscrollable combo dropdown list

Reported by: Garry Yao Owned by: Garry Yao
Priority: Normal Milestone: CKEditor 3.3
Component: UI : Floating Panel Version: SVN (CKEditor) - OLD
Keywords: Confirmed IE Review- Cc:

Description

In Quirks mode, the dropdown list of all combos don't have vertical scrollbar.

Attachments (3)

5452.patch (1.6 KB) - added by Garry Yao 14 years ago.
5452_2.patch (1.3 KB) - added by Garry Yao 14 years ago.
5452_3.patch (1.1 KB) - added by Garry Yao 14 years ago.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 14 years ago by Garry Yao

Version: SVN (CKEditor)

Regression brought in 3.2.

Changed 14 years ago by Garry Yao

Attachment: 5452.patch added

comment:2 Changed 14 years ago by Garry Yao

Keywords: Review? added
Owner: set to Garry Yao
Status: newassigned

Precisely, regression of [4943].

comment:3 Changed 14 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

There is an evident issue in the patch: "cke_browser_iequirk s"

Changed 14 years ago by Garry Yao

Attachment: 5452_2.patch added

comment:4 Changed 14 years ago by Garry Yao

Keywords: Review? added; Review- removed

It's turned out that the scroll reset styles are unnecessary anymore.

Changed 14 years ago by Garry Yao

Attachment: 5452_3.patch added

comment:5 Changed 14 years ago by Garry Yao

Only IE6 plays as an exception for the previous patch, it safer to just keep the scroll reset for all the browsers.

comment:6 Changed 14 years ago by Frederico Caldeira Knabben

Keywords: Review- added; Review? removed

I'm not able to reproduce this problem in any browser with the current trunk. Can you please confirm it, including the browser information?

comment:7 Changed 14 years ago by Garry Yao

Resolution: expired
Status: assignedclosed

That's weird, WFM now in current trunk.

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