Opened 15 years ago
Closed 15 years ago
#5623 closed Bug (fixed)
Apply two styles problematic
| Reported by: | Garry Yao | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.3 |
| Component: | Core : Styles | Version: | 3.2.1 |
| Keywords: | Confirmed Firefox Review+ | Cc: |
Description
Environment
Firefox
Reproducing Procedures
- Open the 'replacebyclass' sample page and make the following selection:
<p> [This is ]some...
- Apply 'Bold' style on the selected content;
- Make the new selection as following and apply 'Italic' Style:
<p> <strong>This [is ]</strong> some...
- Actual Result: The italic style has expanded onto the previous word.
- Expected Result: Only the selected content receive the new style.
Attachments (1)
Change History (4)
Changed 15 years ago by
| Attachment: | 5623.patch added |
|---|
comment:1 Changed 15 years ago by
| Keywords: | Review? added |
|---|---|
| Owner: | set to Garry Yao |
| Status: | new → assigned |
comment:2 Changed 15 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
Note: See
TracTickets for help on using
tickets.
I remember we've used bookmarks in the past, but for some reason we've changed it to the current code. Maybe we were simply trying to use the original bookmark, while you're proposing creating a new bookmark after enlarging the range.
The change is good, but we must test and watch the style system pretty well before the release.