Opened 8 years ago

Last modified 4 years ago

#8958 confirmed Bug

can not escape style for a whole line in firefox

Reported by: yiminghe Owned by:
Priority: Normal Milestone:
Component: Core : Styles Version: 3.6.1
Keywords: Firefox Cc:

Description

Steps:

  1. input multiple lines of text as pic1 in firefox.
  1. ctrl + a to select all text, then make all selected text bold/underline/... as pic2.
  1. select one line (such as the second line) to unbold the whole line of text as pic3.
  1. place cursor at the end of second line and input "ab"

Expected result:

"ab" is not bold.

Actual result:

"ab" is bold.

Possible cause:

when style in firefox, br is included in "strong" tag as pic4 and when unstyle br is not removed with "strong" tag as pic5.

Attachments (5)

1.png (8.9 KB) - added by yiminghe 8 years ago.
2.png (15.2 KB) - added by yiminghe 8 years ago.
3.png (15.0 KB) - added by yiminghe 8 years ago.
4.png (3.4 KB) - added by yiminghe 8 years ago.
5.png (4.6 KB) - added by yiminghe 8 years ago.

Download all attachments as: .zip

Change History (10)

Changed 8 years ago by yiminghe

Attachment: 1.png added

Changed 8 years ago by yiminghe

Attachment: 2.png added

Changed 8 years ago by yiminghe

Attachment: 3.png added

Changed 8 years ago by yiminghe

Attachment: 4.png added

Changed 8 years ago by yiminghe

Attachment: 5.png added

comment:1 Changed 8 years ago by Jakub Ś

Keywords: Firefox added; firefox styles removed
Status: newconfirmed
Version: 3.6.4 (SVN - trunk)3.6.1

Code:

<p>
	abc</p>
<p>
	def</p>
<p>
	ghi</p>

Reproducible only in Firefox from CKEditor 3.6.1 rev [7045].

Most likely because of "when style in firefox, br is included in 'strong' tag" after this TC you get below result in editor:

<p><strong></strong>def<strong><br></strong></p>
Last edited 6 years ago by Jakub Ś (previous) (diff)

comment:2 Changed 7 years ago by Jakub Ś

Can be reproduced in CKEditor 4.x (v4)

comment:3 Changed 6 years ago by Jakub Ś

It seems we have whole buch of issues basic styles and selection problem across all browsers:

comment:4 Changed 6 years ago by Jakub Ś

I think these three are kind of strongly related and some approach has to be prepared taking all these tickets into consideration: #8659, #10872, #8958.

comment:5 Changed 4 years ago by kkrzton

Still broken in FF 43.0 and CKEditor 4.5.6.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy