Opened 15 years ago
Closed 14 years ago
#4082 closed Bug (fixed)
[IE+Quirks] Preview text in the image dialog is not wrapping
| Reported by: | Frederico Caldeira Knabben | Owned by: | Minh Nguyen |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.3 |
| Component: | General | Version: | SVN (CKEditor) - OLD |
| Keywords: | Confirmed IE Review+ | Cc: |
Description
The preview text displayed in the Image dialog is not wrapping with IE7+Quirks.
Attachments (5)
Change History (15)
Changed 14 years ago by
| Attachment: | 4082.patch added |
|---|
comment:1 Changed 14 years ago by
| Keywords: | Review? added |
|---|---|
| Milestone: | CKEditor 3.x → CKEditor 3.3 |
| Owner: | set to Minh Nguyen |
| Status: | new → assigned |
comment:2 Changed 14 years ago by
comment:3 Changed 14 years ago by
| Keywords: | Review- added; Review? removed |
|---|
It works while make some of the dialog fields label wrapping incorrectly.
We should narrow down the selector if possible.
Changed 14 years ago by
| Attachment: | 4082_2.patch added |
|---|
comment:4 Changed 14 years ago by
| Keywords: | Review? added; Review- removed |
|---|
Changed 14 years ago by
| Attachment: | 4082_3.patch added |
|---|
comment:5 Changed 14 years ago by
| Keywords: | Review- added; Review? removed |
|---|
comment:6 Changed 14 years ago by
| Keywords: | Review? added; Review- removed |
|---|
comment:7 Changed 14 years ago by
| Keywords: | Review- added; Review? removed |
|---|
The latest patch is heading the correct way, while reset.css should only perform browser normalization instead of holding rules of a specific structure.
Providing a patch for reference.
Changed 14 years ago by
| Attachment: | 4082_ref.patch added |
|---|
comment:8 Changed 14 years ago by
| Keywords: | Review? added; Review- removed |
|---|
comment:9 Changed 14 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
It's a nice solution, table is highly welcomed by browsers in quirks :)
Note: See
TracTickets for help on using
tickets.
IE very interesting. I cannot explain why use '*' instead of '#ImagePreviewBox' in this case.