Opened 16 years ago
Closed 15 years ago
#4107 closed Bug (fixed)
Font combo inconsistency
| Reported by: | Garry Yao | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.1 |
| Component: | Core : Styles | Version: | |
| Keywords: | Confirmed Review+ | Cc: |
Description (last modified by )
Reproducing Procedures
- Open the 'replace by class' sample page in Firefox;
- Assign 'Georgia' font to word 'This';
- Open the 'replace by class' sample page in IE7;
- Copy the source of editor from FF into IE;
- Click inside word 'This';
- Actual Result: The 'Font' combo is not updated.
- Expected Result: The 'Font' combo is populated with 'Georgia'.
Attachments (2)
Change History (8)
comment:1 Changed 16 years ago by
| Keywords: | Confirmed added |
|---|---|
| Owner: | set to Garry Yao |
| Status: | new → assigned |
Changed 16 years ago by
| Attachment: | 4107.patch added |
|---|
comment:2 Changed 16 years ago by
| Keywords: | Review? added |
|---|
comment:3 Changed 16 years ago by
| Keywords: | Review- added; Review? removed |
|---|---|
| Milestone: | CKEditor 3.0 → CKEditor 3.1 |
I understand the generalization here, but this is supposed to be a localized issue, exclusive to the fonts combo (we'll rarely have it elsewhere), so the fix could go directly to that combo code.
Also, that regex can be simplified, if we're simply looking for replacing spaces after the commas.
Changed 15 years ago by
| Attachment: | 4107_2.patch added |
|---|
comment:4 Changed 15 years ago by
| Description: | modified (diff) |
|---|---|
| Keywords: | Review? added; Review- removed |
Update the description for accurate reproducing.
comment:5 Changed 15 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
On commit, please:
- Move the test to the new proper location.
- Add the changelog entry for it.
Note: See
TracTickets for help on using
tickets.
Caused by font style text various in format:
vs