Opened 14 years ago
Closed 14 years ago
#6000 closed Bug (fixed)
Safari scrolls to top on initial click and headline formatting
| Reported by: | André | Owned by: | Frederico Caldeira Knabben |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.4 |
| Component: | General | Version: | 3.3 |
| Keywords: | WebKit | Cc: |
Description
You can reproduce it while going to the demo, scroll down to the end and click on the last paragraph. The focus will be set right but the editor window will scroll up. Same happens when going down to the last paragraph and make a headline out of a normal text.
This is totally annoying for long texts and in the case of headline it make the editing a pain, as you have to find you headline each time you create one :-(
Mac OS X 10.6.4 Safari 5.0
Attachments (2)
Change History (13)
comment:1 Changed 14 years ago by
| Keywords: | Scroll removed |
|---|---|
| Status: | new → confirmed |
comment:2 Changed 14 years ago by
This was introduced in 3.3, in 3.2.2 it worked correctly. So rather a CKEditor bug. It seems that the position of the cursor does get lost.
comment:3 Changed 14 years ago by
| Milestone: | → CKEditor 3.4 |
|---|
comment:5 Changed 14 years ago by
| Keywords: | WebKit added; Safari removed |
|---|
Changed 14 years ago by
| Attachment: | 6000.patch added |
|---|
comment:7 Changed 14 years ago by
| Owner: | set to Sa'ar Zac Elias |
|---|---|
| Status: | confirmed → review |
comment:8 Changed 14 years ago by
| Owner: | changed from Sa'ar Zac Elias to Frederico Caldeira Knabben |
|---|---|
| Status: | review → assigned |
I've found the source of the issue. In fact, we're focusing the documentElement, which is not anymore the editable container, which is not <body>.
I'll come with a new patch for it.
Changed 14 years ago by
| Attachment: | 6000_2.patch added |
|---|
comment:9 Changed 14 years ago by
| Status: | assigned → review |
|---|
The problem is created by code introduced with [5509] for #5713. That one was good for designMode, but now things changed with contenteditable.
It's enough to remove part of that code, which looks like not needed anymore. I've tested the patch with Chrome and Safari Windows and Mac, and it works well. I see no problems with selection being lost, not even with the focus and blur events.
comment:10 Changed 14 years ago by
| Status: | review → review_passed |
|---|
comment:11 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [5785].
Looks like a browser bug.. need some investigation.