Opened 14 years ago
Closed 13 years ago
#6212 closed Bug (fixed)
Chrome resize jumps to top of page
Reported by: | Danial Pearce | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.6.2 |
Component: | General | Version: | 3.4 |
Keywords: | WebKit | Cc: | Eric K |
Description
This bug is reproducible on the stock demo.
If you enter lots of content, and make the editor bigger than your browser, so scroll bars appear on the outer document, then scroll down on your page, when you resize the editor the scroll will jump back to the top. This makes it unusable if your editor is at the bottom of a large document.
This only happens in webkit browsers such as chrome/safari.
If you revert changset:5516 the bug disappears, but I'm sure it was there for a reason, so not sure what bug will reappear?
Attachments (2)
Change History (18)
comment:1 Changed 14 years ago by
Resolution: | → duplicate |
---|---|
Status: | new → closed |
comment:2 Changed 14 years ago by
Leave closed if you like, but I believe this bug is completely separate from ticket:6195 as it refers to the elementspath plugin not wrapping correctly.
This bug I have mentioned for above is introduced many times throughout trying to fix together hacks in ticket:5729 and ticket:6195 and is specific to the resizing of the editor.
comment:3 Changed 14 years ago by
Cc: | Eric K added |
---|---|
Version: | 3.4 → 3.4.3 |
This is still a problem in 3.4.3 and 3.5, and can still be seen in the demo. Using either Chrome or Safari on Mac or Windows,
- load http://ckeditor.com/demo in a window short enough to cause the scrollbar to appear
- scroll to the bottom of the window
- attempt to resize the editor
and note how the page scrolls up suddenly. In some cases the window will scroll up only slightly; in others it will jump to the top of the page. This does not occur in IE or Gecko-based browsers.
comment:4 Changed 14 years ago by
If you manually revert changeset:5516 it fixes the problem - and with the latest version of Safari, the following doesn't apply at all:
Resize the width first. WEBKIT BUG: Webkit requires that we put the editor off from display when we resize it. If we don't, the browser crashes!
comment:5 Changed 14 years ago by
Keywords: | Webkit added; webkit chrome resize removed |
---|---|
Resolution: | duplicate |
Status: | closed → reopened |
comment:6 Changed 14 years ago by
Status: | reopened → confirmed |
---|
comment:7 Changed 13 years ago by
Keywords: | resize added |
---|---|
Version: | 3.4.3 → 3.6.2 (SVN - trunk) |
comment:8 Changed 13 years ago by
Keywords: | WebKit added; Webkit resize removed |
---|---|
Version: | 3.6.2 (SVN - trunk) → 3.4 |
@EricFromCanada The version field should be set to the oldest version where issue is reproducible, see TicketSpecs for more details.
Changed 13 years ago by
Attachment: | 6212.patch added |
---|
comment:9 Changed 13 years ago by
Owner: | set to Garry Yao |
---|---|
Status: | confirmed → review |
comment:11 Changed 13 years ago by
Status: | review → review_failed |
---|
Resizing in Safari scrolls the host page, not to top though, just above the editor.
comment:12 Changed 13 years ago by
Status: | review_failed → review |
---|
Manual tc added with [7120]. @Saar can't reproduce the issue you mentioned.
Changed 13 years ago by
Attachment: | 6212_2.patch added |
---|
comment:13 Changed 13 years ago by
comment:14 Changed 13 years ago by
Milestone: | → CKEditor 3.6.2 |
---|---|
Status: | review → review_passed |
It seems that we have no crashing issues any more without that code. Cool!
comment:15 Changed 13 years ago by
comment:16 Changed 13 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed with [7213].
DUP of #6195.