Opened 8 years ago
Closed 7 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 garry.yao)
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 8 years ago by garry.yao
- Keywords Confirmed added
- Owner set to garry.yao
- Status changed from new to assigned
Changed 8 years ago by garry.yao
comment:2 Changed 8 years ago by garry.yao
- Keywords Review? added
comment:3 Changed 8 years ago by fredck
- Keywords Review- added; Review? removed
- Milestone changed from CKEditor 3.0 to 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 7 years ago by garry.yao
comment:4 Changed 7 years ago by garry.yao
- Description modified (diff)
- Keywords Review? added; Review- removed
Update the description for accurate reproducing.
comment:5 Changed 7 years ago by fredck
- Keywords Review+ added; Review? removed
On commit, please:
- Move the test to the new proper location.
- Add the changelog entry for it.
comment:6 Changed 7 years ago by garry.yao
- Resolution set to fixed
- Status changed from assigned to closed
Fixed with [4318].
Note: See
TracTickets for help on using
tickets.
Caused by font style text various in format:
vs