Opened 13 years ago
Closed 13 years ago
#6684 closed Bug (fixed)
Toolbar isn't wrapped in Webkit when editor is in the table
Reported by: | kresimir | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.5.1 |
Component: | UI : Toolbar | Version: | 3.0 |
Keywords: | WebKit | Cc: |
Description
When toolbar has many items (e.g. Full toolbar) and editor is placed in table which is in div block with fixed width, editor doesn't wrap toolbar in WebKit (Chrome and Safari) and puts whole toolbar in one line.
To reproduce this bug, add following around a form which has editor with full toolbar:
<div style="width: 400px; border: 1px solid red;"><table><tr><td> </div>
Attachments (3)
Change History (11)
Changed 13 years ago by
Attachment: | jqueryadapter_table.html added |
---|
Changed 13 years ago by
Attachment: | 6684.patch added |
---|
comment:1 Changed 13 years ago by
Component: | General → UI : Toolbar |
---|---|
Milestone: | → CKEditor 3.5.1 |
Owner: | set to Garry Yao |
Status: | new → review |
Version: | → 3.0 |
Look serious, unless we come with a reason to keep that line.
comment:2 Changed 13 years ago by
Keywords: | WebKit added |
---|
comment:3 Changed 13 years ago by
Works good, but i'm not sure if we should remove inline-table for all browsers, when it creates issue only on Webkit.
Changed 13 years ago by
Attachment: | 6684_2.patch added |
---|
comment:4 Changed 13 years ago by
I've failed to check the original commits due the "lost" of the original kama branch, so please somebody remembers.
comment:5 follow-up: 6 Changed 13 years ago by
Status: | review → review_failed |
---|
In Chrome, clicking on collapse toolbar button, the width of the editor decreases.
comment:6 Changed 13 years ago by
Status: | review_failed → review |
---|
Replying to Saare:
In Chrome, clicking on collapse toolbar button, the width of the editor decreases.
Quite unfortunate but AFAIK that's an existing bug.
comment:8 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [6304].
Reproducing the bug with modifying ckeditor/_samples/jqueryadapter.html