Ticket #8687 (closed Bug: duplicate)

Opened 2 years ago

Last modified 2 years ago

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:

  1. open the demo,
  2. go to Source mode and paste this:
    <p><!-- BUG -->loren ipsum</p>
    
  3. go back to WYSIWYG,
  4. 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

comment:1 Changed 2 years ago by j.swiderski

  • Status changed from new to closed
  • Keywords comment tag, range selection removed
  • Version 3.6.3 (SVN - trunk) deleted
  • Resolution set to duplicate

DUP of #8732

Last edited 2 years ago by j.swiderski (previous) (diff)
Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy