Opened 12 years ago
Closed 12 years ago
#9258 closed Bug (fixed)
Incorrect handling of multiple font-sizes chosen at the same time
Reported by: | Piotrek Koszuliński | Owned by: | Garry Yao |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 4.0 |
Component: | General | Version: | 4.0 |
Keywords: | Cc: |
Description
Tested on Firefox and Chrome.
- Open any sample.
- Place caret in the editor.
- Choose font size 20.
- Type few letters.
- Choose font size 28.
- Type few letters.
- Choose font size 28 again so it will be removed.
- Type few letters.
- Typed letters has font size 20, but 28 is still highlighted.
- Open combo again - 28 is highlighted.
- Click 20.
- Everything is ok now - size in combo and for typed text is back to default.
Change History (3)
comment:1 Changed 12 years ago by
Owner: | set to Garry Yao |
---|---|
Status: | new → review |
comment:2 Changed 12 years ago by
Status: | review → review_passed |
---|
The list block implementation is not wrong. It is just considering a feature which we're not using on any of our plugins, where non-multiselect lists don't have the toggle effect (like select boxes).
But considering that we're not using that feature, we can accept this change, because all our combos act like selection boxes with toggle behavior. If we'll have troubles with this, we'll be able to revert this and try another approach.
comment:3 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
It's a regression, caused by our immediate selection change event in v4, but the culprit is instead inside of list block impl.