#6416 closed Bug (fixed)
Disabled selection of text by mouse in Source editing under IE9
| Reported by: | Krzysztof Studnik | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.4.2 |
| Component: | UI : Source View | Version: | 3.4.1 |
| Keywords: | IE | Cc: |
Description
Under IE9
- open CKEditor demo
- switch to source editing
- try to select source text by mouse
Expected: Range of text is selected
Actual:
selection is disabled for mouse. I am able to select text by keyboard.
In WYSIWYG, selection by mouse works fine
Attachments (2)
Change History (12)
comment:1 Changed 14 years ago by
| Keywords: | IE9 added |
|---|
comment:2 Changed 14 years ago by
| Keywords: | IE added; IE9 removed |
|---|---|
| Milestone: | → CKEditor 3.4.2 |
comment:3 Changed 14 years ago by
| Component: | General → UI : Source View |
|---|---|
| Owner: | set to Alfonso Martínez de Lizarrondo |
| Status: | new → review |
Changed 14 years ago by
| Attachment: | 6416_2.patch added |
|---|
comment:4 Changed 14 years ago by
| Status: | review → review_failed |
|---|
Let's just remove it now until somebody shout out the usage of that line.
comment:5 Changed 14 years ago by
| Owner: | changed from Alfonso Martínez de Lizarrondo to Garry Yao |
|---|---|
| Status: | review_failed → review |
comment:6 Changed 14 years ago by
| Status: | review → review_failed |
|---|
If the onmousedown handler is removed, clicking the Source button generates an error in Firefox 3.6
editor.document is null -> editor.document.$.designMode = 'off';
comment:7 Changed 14 years ago by
| Status: | review_failed → review |
|---|
'onmousedown=false' on container is actually a focus stopper for toolbar buttons, that prevent click on toolbar button from stealing focus away from editor, we must keep in still. R+ for 6416, but may alter tune on "by some yet unknown reason".
comment:8 Changed 14 years ago by
| Status: | review → review_passed |
|---|
comment:9 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [5965].
comment:10 Changed 14 years ago by
Filed #6644 to try to remove the code as proposed in the second patch.
Note: See
TracTickets for help on using
tickets.
It seems that IE9 is following just the same behavior that other browsers have, so now we have to apply a workaround also for it.
It would be nice to really understand why this is happening and avoid it altogether for every browser, but meanwhile this will make the mouse work in source mode for IE9