Opened 17 years ago
Last modified 10 years ago
#1646 confirmed Bug
Color for Underline and Strikethrough inconsistent
Reported by: | Jon Brooks | Owned by: | |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | Core : Styles | Version: | 3.0 |
Keywords: | Cc: | satya_minnekanti@… |
Description
The color that Underline and Strikethrough take is inconsistent. Consider these two examples:
- begin typing text
- switch Font Color to Red
- Select 'Strikethrough,' type some more
Result:
The strikethrough is red.
- Type some text
- Highlight the text, change the color to Red,
- With the selection still highlighted, select 'Strikethrough'
Result:
The strikethrough is black.
Reversing the order in which styles are added in both scenarios produce the opposite result.
Expected behavior should probably be that the strikethrough and underline always take the current color of the text.
Change History (6)
comment:1 Changed 17 years ago by
Keywords: | Confirmed added |
---|
comment:2 Changed 13 years ago by
Version: | FCKeditor 2.5 → 3.0 |
---|
Occurs still in CKE3.6.2Trunk confirmed under FF, IE8, chrome
comment:4 Changed 11 years ago by
Resolution: | fixed |
---|---|
Status: | closed → reopened |
I'm still able to reproduce it both on Chrome and Firefox.
comment:5 Changed 11 years ago by
Status: | reopened → confirmed |
---|
This issue is in fact a part of #10675.
Using different tags 's' for strikethrough and span for color-text is a "won't fix" issue but if user decides to use spans for strikethrough then styles should be merged.
Edit: As discussed with @wwalc, we could make spans always wrapping tags like b, s, u in such cases but I can't shake the impression that there might be people that want to have b,u,s tags outside of spans and what then? We would make editor "smarter" than our users and this is newer a good thing. On the other hand such wrapping would really solve some of our styling problems.
We have to make some search for real-life scenarios where tags like b,u,s, sup, sub wrap spans with text. I personally can’t remember any :)
Confirmed on IE, IE7 and FF