Opened 10 years ago
Closed 8 years ago
#12465 closed Bug (fixed)
Can't update checkbox/radio button state with dialog when it is opened with double-click
Reported by: | Jakub Ś | Owned by: | Tomasz Jakut |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | General | Version: | 3.3 |
Keywords: | Blink Webkit Support | Cc: |
Description (last modified by )
In Blink and Webkit it is impossible to change state of checkboxes/radiobuttons with properties dialog when this dialog is opened with double-click.
To reproduce:
- Open checkbox/radio button properties dialog.
- Enter all the values and press OK.
- Double-click on checkbox/radio button to open the checkbox/radio button properties dialog again.
- Click on checkbox in ckeckbox/radio button properties dialog to make it unselected and press OK.
Result: the state of checkbox/radio button is not changed (if it was checked, it is still checked).
Problem can be reproduced from CKEditor 3.3 (when opening properties dialog with double-click was made possible) in Blink and Webkit browsers.
NOTE: When you right-click on checkbox/radio button to open dialog and change its state then it works.
Change History (8)
comment:1 Changed 10 years ago by
Status: | new → confirmed |
---|
comment:2 Changed 10 years ago by
Description: | modified (diff) |
---|---|
Summary: | Can't update checkbox state with Dialog. → Can't update checkbox/radio button state with dialog when it is opened with double-click |
Version: | 3.0 → 3.3 |
comment:3 Changed 10 years ago by
comment:4 Changed 9 years ago by
Owner: | set to Tomasz Jakut |
---|---|
Status: | confirmed → assigned |
comment:5 Changed 9 years ago by
Status: | assigned → review |
---|
It seems that the bug is caused only by the double-click made by user - doing it programmatically doesn't raise an error.
The fix was simple: checked
property, not only attribute, should be changed. Pushed branch:t/12465.
comment:6 Changed 8 years ago by
Status: | review → review_failed |
---|
If the fix is only for Blink/Webkit, probably CKEDITOR.env.webkit
could be used so it is only applied on those browsers?
There is already bender.ignore()
in manual test, so the ###ONLY IN BLINK/WEBKIT###
information is not necessary IMHO.
Apart from these tiny details above, everything looks good.
comment:7 Changed 8 years ago by
Status: | review_failed → review |
---|
Pushed fixes for tests and checking for WebKit to branch:t/12465.
comment:8 Changed 8 years ago by
Resolution: | → fixed |
---|---|
Status: | review → closed |
Nothing broke due to this change, LGTM.
Fixed with git:b6612e607f32a3b6b2c94975fd7ec05da6d9be0e.
#13072 was marked as duplicate.