Opened 15 years ago
Closed 15 years ago
#4512 closed Bug (fixed)
TC from dt/plugins/styles is broken because of semicolon fixing
Reported by: | Garry Yao | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 3.1 |
Component: | Core : Styles | Version: | SVN (CKEditor) - OLD |
Keywords: | Safari Confirmed Review+ | Cc: |
Description
Extra semicolons get introduced due to the following codes:
for ( var style in stylesDef ) stylesText += style + ':' + stylesDef[ style ] + ';';
Attachments (2)
Change History (7)
Changed 15 years ago by
Attachment: | 4512.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 |
---|
Changed 15 years ago by
Attachment: | 4512_2.patch added |
---|
comment:3 Changed 15 years ago by
Keywords: | Review? added; Review- removed |
---|
comment:4 Changed 15 years ago by
Keywords: | Review+ added; Review? removed |
---|
I know it's a small repetition, but for this simple case, let's just try having the code simpler, amending just the following: