We are migrating CKEditor issue tracking to GitHub. Please, use GitHub to report any new issues.

The former tracking system (this website) will still be available in the read-only mode. All issues reported in the past will still be available publicly and can be referenced.

Important: we decided not to transfer all the tickets to GitHub, as many of them are not reproducible anymore or simply no longer requested by the community.
If the issue you are interested in, can be still reproduced in the latest version of CKEditor, feel free to report it again on GitHub.
At the same time please note that issues reported on this website are still taken into consideration when picking up candidates for next milestones.

To report a new issue, go to https://github.com/ckeditor/ckeditor-dev/issues and follow the instructions in the issue template.

Context Navigation

← Previous Ticket
Next Ticket →

Opened 13 years ago

Closed 13 years ago

#8687 closed Bug (duplicate)

Presence of comment tags in the Source breaks range selection

Reported by:	dgaspar	Owned by:
Priority:	Normal	Milestone:
Component:	General	Version:
Keywords:		Cc:

Description

How to reproduce:

open the demo,
go to Source mode and paste this:
```
<p>loren ipsum</p>
```
go back to WYSIWYG,
press ctrl+A (select all), ctrl+B (for bold) --> BUG!

A quick fix: core/dom/range.js, enlarge() method:

if ( ( sibling.$.offsetWidth > 0 || excludeBrs && sibling.is && sibling.is( 'br' ) ) && !sibling.data( 'cke-bookmark' ) )

The comment nodes are represented by CKEDITOR.dom.node (which doesn't have the is() method) instead of the expected CKEDITOR.dom.element

Change History (1)

comment:1 Changed 13 years ago by Jakub Ś

Keywords:	comment tag range selection removed
Resolution:	→ duplicate
Status:	new → closed
Version:	3.6.3 (SVN - trunk)

DUP of #8732

Last edited 13 years ago by Jakub Ś (previous) (diff)

Note: See TracTickets for help on using tickets.

Download in other formats: