Opened 14 years ago
Closed 14 years ago
#6445 closed Bug (fixed)
Find/Replace highlight styles were incorrectly left in DOM
| Reported by: | Garry Yao | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.4.2 |
| Component: | Core : Styles | Version: | 3.4.2 |
| Keywords: | Opera | Cc: |
Description ¶
- Find anything that matches in the document content;
- After matches is highlighted, close the dialog; *Actual Result: Highlight marker is left uncollected in DOM.
Find this issue on GitHubChange History (5)
comment:1 Changed 14 years ago by
| Status: | new → confirmed |
|---|
Changed 14 years ago by
| Attachment: | 6445.patch added |
|---|
comment:2 Changed 14 years ago by
| Owner: | set to Garry Yao |
|---|---|
| Status: | confirmed → review |
Revert part of [5847] where 'CKEDITOR.style.getStyleText' must be used to properly normalize the style values.
comment:3 Changed 14 years ago by
| Status: | review → review_passed |
|---|
It's sad to see that dead code has been left there, even after review.
comment:4 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | review_passed → closed |
Fixed with [5978].
Find this issue on GitHubNote: See
TracTickets for help on using
tickets.
Regression of [5847].