Opened 15 years ago
Last modified 15 years ago
#6107 closed Bug
Remove block style via the list — at Version 7
| Reported by: | Sa'ar Zac Elias | Owned by: | Sa'ar Zac Elias |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.5.1 |
| Component: | Core : Styles | Version: | 3.0 |
| Keywords: | Cc: |
Description (last modified by )
Currently it's impossible to remove block styles via the combo by clicking on the active option.
Change History (9)
comment:1 Changed 15 years ago by
| Owner: | set to Sa'ar Zac Elias |
|---|---|
| Status: | new → assigned |
Changed 15 years ago by
| Attachment: | 6107.patch added |
|---|
comment:2 Changed 15 years ago by
| Status: | assigned → review |
|---|
comment:3 follow-up: 6 Changed 15 years ago by
comment:4 Changed 15 years ago by
| Milestone: | → CKEditor 3.5.1 |
|---|
comment:5 Changed 15 years ago by
| Status: | review → review_failed |
|---|
The patch mixes "applyBlockStyle" and "removeBlockStyle" logic together which is bad.
comment:6 Changed 15 years ago by
Replying to tobiasz.cudnik:
Seems to be DUP of #5336.
We're talking of block style, no longer object style here.
comment:7 Changed 15 years ago by
| Description: | modified (diff) |
|---|---|
| Status: | review_failed → assigned |
| Summary: | Remove object style via the list → Remove block style via the list |
Yep, block styles this time.
Changed 15 years ago by
| Attachment: | 6107_2.patch added |
|---|
Note: See
TracTickets for help on using
tickets.
Seems to be DUP of #5336.