Opened 15 years ago

Last modified 8 years ago

#5503 confirmed Bug

Font-size overlaps background-color

Reported by: Garry Yao Owned by:
Priority: Normal Milestone:
Component: Core : Styles Version:
Keywords: IBM Cc: satya_minnekanti@…

Description (last modified by Garry Yao)

Reproducing Procedures

  1. Open any of the sample page, load the document with the following content and selection;
    <p> [some sample] text </p>
    
  2. Apply the 'Background-color' style with 'red' value.
  3. Make the following selection based on the result document.
    <p><span style="background-color: rgb(255, 0, 0);">some [sample</span> text]</p>
    
  4. Apply the 'Font-size' style with value '72pt';
    • Actual Result:
      1. Firefox, IE8 and Safari:
      2. IE6/7:
  • Expected Result:

Attachments (3)

firefox_ie8_safari.png (1.8 KB) - added by Garry Yao 15 years ago.
ie6_ie7.png (2.1 KB) - added by Garry Yao 15 years ago.
expected.png (2.0 KB) - added by Garry Yao 15 years ago.

Download all attachments as: .zip

Change History (19)

Changed 15 years ago by Garry Yao

Attachment: firefox_ie8_safari.png added

Changed 15 years ago by Garry Yao

Attachment: ie6_ie7.png added

Changed 15 years ago by Garry Yao

Attachment: expected.png added

comment:1 Changed 15 years ago by Garry Yao

Description: modified (diff)

comment:2 Changed 15 years ago by Garry Yao

This's a continuation of bug #3599.

comment:3 Changed 15 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.3CKEditor 3.x

comment:4 Changed 14 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.x

Milestone CKEditor 3.x deleted

comment:5 Changed 14 years ago by Satya Minnekanti

Cc: satya_minnekanti@… added
Keywords: IBM added

comment:6 Changed 13 years ago by Jakub Ś

Tickets #8437, #2675, #8250 and #8181 were marked as duplicates

comment:7 Changed 13 years ago by Jakub Ś

From the ticket #8437 you can see that same thing can be reproduced when pasting from Word.


There are couple of solution here:

  1. Since this is correct HTML and it does what it is told, we can tell users to use HTML in such order to get what they want. Not very user friendly
  1. CKEditor could wisely switch spans from
    <span style="background:lime;">
           <span style="color:white;">
               <span style="font-size:36.0pt;">WORD</span>
           </span>
    </span>
    

to (size around background color)

<span style="font-size:36.0pt;">
    <span style="background:teal;">
         <span style="color:white;">ABC</span>
</span>

But what if someone actually wanted something like that- not a good idea.

  1. Perhaps best solution would me merging span tags into one. When one wants to add style new rule would have to be added to span and when one wants to remove certain style, only this style would have to be removed and not whole span.
    There is still problem with bold/underline/italics. By default these are represented by strong/em/u. We can't expect editor to merge put these inside span. Either users should know what to use first and what second (can be described in docs) or devs creating apps with CKEditor should set these styles to use spans instead of default tags so as to get merging effect.

This isn't easy issue and there are many cases to be considered. Some of them have been described: #8045, #7498, #8686.

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

comment:8 Changed 13 years ago by Jakub Ś

Just a note - #3599 says that this issue is supposed to be fixed.

comment:9 Changed 12 years ago by Jakub Ś

#10478 was marked as duplicate.

comment:10 Changed 11 years ago by Jakub Ś

This ticket is in fact part of larger #4505.

comment:11 Changed 11 years ago by Jakub Ś

#11100 was marked as duplicate.

comment:12 Changed 9 years ago by Anna Tomanek

Summary: Font-size overlap Background-colorFont-size overlap background-color

comment:13 Changed 9 years ago by Anna Tomanek

Summary: Font-size overlap background-colorFont-size overlaps background-color

comment:14 Changed 9 years ago by Jakub Ś

Similar issue for PFW has been fixed in branch t/9991 and will be introduced with milestone 4.6.

comment:15 Changed 8 years ago by Jakub Ś

#16760 was marked as duplicate.

comment:16 Changed 8 years ago by Jakub Ś

#16758 was marked as duplicate.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy