Opened 12 years ago
Closed 12 years ago
#10119 closed Bug (fixed)
[FF] Click on "body" in elements path selects only on first block element
Reported by: | Piotrek Koszuliński | Owned by: | Frederico Caldeira Knabben |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 4.0.2 |
Component: | Core : Selection | Version: | 4.0.2 |
Keywords: | Firefox | Cc: |
Description
TC1:
- Open replacebyclass sample.
- Without focusing editor, click 'body' in elements path.
- Only header and image were selected.
TC2:
- Write two paragraphs.
- Switch to source and back to wysiwyg.
- Without focusing editor, click 'body' in elements path.
- Only first paragraphs was selected.
Reproduced on FF, couldn't on Chrome, haven't checked the rest.
Change History (5)
comment:1 Changed 12 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 12 years ago by
comment:3 Changed 12 years ago by
Owner: | set to Frederico Caldeira Knabben |
---|---|
Status: | confirmed → review |
It looks like the problem is that we're having two selection attempts happening at the same time when the editor gets focused. Firefox is buggy and is not able to handle this properly, making a random selection out of it.
I've proposed a fix on t/10119@cksource. It doesn't change much the logic of the code. It just demonstrates that the focus logic is very delicate and even the order of things matter. In any case, we're not able to do much about all this, until browsers will not be strong and stable about selection.
comment:4 Changed 12 years ago by
Status: | review → review_passed |
---|
OK, I haven't found any regressions and issue seems to be fixed. It's good to understand that this patch just targets more precisely patch for #9507.
PS. Is it possible to write test for this issue?
comment:5 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Fixed on master with git:a2136ce.
Introduced with git:a9d4df7.