Ticket #4084 (closed Bug: fixed)
[FF] Image dialog using Kama skin sticks to left viewport border
| Reported by: | tobiasz.cudnik | Owned by: | tobiasz.cudnik |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.0 |
| Component: | General | Version: | |
| Keywords: | Firefox Confirmed Review+ | Cc: |
Description (last modified by tobiasz.cudnik) (diff)
In Firefox (3, 3.5) using Kama skin, after opening image dialog it sticks to left viewport border. Drag&drop it to the right is impossible.
Quirks mode only, on pages with 3 editors, each with different skin.
Attachments
Change History
comment:1 Changed 4 years ago by tobiasz.cudnik
- Status changed from new to assigned
- Owner set to tobiasz.cudnik
- Keywords Confirmed added
comment:3 Changed 4 years ago by tobiasz.cudnik
Dialogs size is incorrectly read during initial creation. Later it's ok, but old value remains inside dialog system.
Seems to be caused by "lock ratio" and "reset size" buttons (removing them resolves the issue).
Random.
comment:4 Changed 4 years ago by tobiasz.cudnik
- Keywords Review? added
Longer timeout seems to resolve this issue, although it may still exists on slower machines.
I think that dialog system should contain onLoad event, which would be the safe start for all dialog manipulation functions.
comment:5 Changed 4 years ago by fredck
- Keywords Review- added; Review? removed
Ideally we should have no delay at all there. Having 100ms is not the best thing, but acceptable. But now 500ms is definitely too much and decisivelly noticeable.
comment:6 Changed 4 years ago by tobiasz.cudnik
- Keywords Review? added; Review- removed
It seems we can use definition size with success for initial position, as actual dialog layout is fixed. Doing such, we can completely loose timeout and prevent this issue.
#4095 looks like a DUP of this ticket.
comment:7 Changed 4 years ago by tobiasz.cudnik
Previous patch suffered from height problem, as definition size defines dialog contents size, not whole dialog (title & footer).
Third patch uses height from offsetHeight and width from definition. Unfortunately previous timeout of 100ms is still needed.
comment:10 Changed 4 years ago by tobiasz.cudnik
- Status changed from assigned to closed
- Resolution set to fixed
