Opened 15 years ago
Closed 15 years ago
#4174 closed Bug (fixed)
IE : Dialogs may have scrollbars
| Reported by: | Frederico Caldeira Knabben | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.1 |
| Component: | UI : Dialogs | Version: | SVN (CKEditor) - OLD |
| Keywords: | Confirmed IE Review? | Cc: |
Attachments (2)
Change History (9)
comment:1 Changed 15 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | new → assigned |
comment:2 Changed 15 years ago by
| Priority: | Normal → High |
|---|
Changed 15 years ago by
| Attachment: | 4174.patch added |
|---|
comment:3 Changed 15 years ago by
| Keywords: | Review? added |
|---|
comment:4 Changed 15 years ago by
| Priority: | High → Normal |
|---|
comment:5 Changed 15 years ago by
| Keywords: | Review- added; Review? removed |
|---|
Changed 15 years ago by
| Attachment: | 4174_2.patch added |
|---|
comment:6 Changed 15 years ago by
| Keywords: | Review? added; Review- removed |
|---|
Apply layout adjustments to those dialogs which have the risk of overflow.
comment:7 Changed 15 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
As discussed on #4521, we could simply close this ticket now without any patch made here.
Note: See
TracTickets for help on using
tickets.
We need to make it simple, and these fixes for the dialog system issues are starting to make other parts of the code dirty. Let's take the easy approach for now, and then definitely fix it when reviewing the dialog system.
Our main problem here is the image dialog. I would instead propose making changes in the dialog to uniformize it to the standards, having the fields title above them, instead of at the left. In this way we correct two things at the same time.