Opened 14 years ago
Closed 14 years ago
#5516 closed Bug (fixed)
IE8: Toolbar buttons have higher bottom padding
| Reported by: | Frederico Caldeira Knabben | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.3 |
| Component: | UI : Toolbar | Version: | 3.2.1 |
| Keywords: | IE8 Confirmed Review+ | Cc: | Wiktor Walc |
Description
This cannot be reproduced with the sample files, but in the online demo of CKEditor, the buttons have an extra padding in the bottom with IE8 in IE8 mode. This is easy to notice in the last row, when compared with the combos.
Attachments (4)
Change History (12)
Changed 14 years ago by
| Attachment: | 5516.patch added |
|---|
comment:1 Changed 14 years ago by
| Keywords: | Review? added |
|---|---|
| Owner: | set to Minh Nguyen |
| Status: | new → assigned |
comment:2 Changed 14 years ago by
| Keywords: | Review- added; Review? removed |
|---|
comment:3 Changed 14 years ago by
| Keywords: | Confirmed Review? added; Review- removed |
|---|---|
| Owner: | changed from Minh Nguyen to Garry Yao |
| Status: | assigned → new |
After some investigation, it looks for me a IE8 only layout problem instead of any culprit on demo page.
Changed 14 years ago by
| Attachment: | 5516_2.patch added |
|---|
comment:4 Changed 14 years ago by
| Cc: | Wiktor Walc added |
|---|---|
| Status: | new → assigned |
Changed 14 years ago by
| Attachment: | ie6_5516_2.png added |
|---|
comment:5 Changed 14 years ago by
| Keywords: | Review- added; Review? removed |
|---|
Review- because float:left broke IE6 (see attachment).
Changed 14 years ago by
| Attachment: | 5516_3.patch added |
|---|
comment:6 Changed 14 years ago by
| Keywords: | Review? added; Review- removed |
|---|
comment:7 Changed 14 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
Actually I can't reproduce the problem in IE6 with previous patch, but it's OK to continue with your patch which fixes the problem as well.
comment:8 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
The problem with previous patch occurred in IE6 when I've changed the resize_minWidth setting and made CKEditor smaller with the resize grip.
Fixed with [5424].
Note: See
TracTickets for help on using
tickets.
The patch breaks the sample files, I think we should instead check whether some styles from the demo site has caused this problem.