Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fake selection allows for removing in a readonly mode #1516

Closed
jacekbogdanski opened this issue Jan 29, 2018 · 2 comments · Fixed by #1525
Closed

Fake selection allows for removing in a readonly mode #1516

jacekbogdanski opened this issue Jan 29, 2018 · 2 comments · Fixed by #1525
Labels
status:confirmed An issue confirmed by the development team. target:minor Any docs related issue that can be merged into a master or major branch. type:bug A bug.
Milestone
4.9.0

Comments

@jacekbogdanski
Copy link
Member

jacekbogdanski commented Jan 29, 2018 •
edited
Loading

Are you reporting a feature request or a bug?

Bug

Check if the issue is already reported

Put all reference links here…

Provide detailed reproduction steps (if any)

  1. Visit https://cdn.ckeditor.com/4.8.0/full-all/samples/old/image2/image2.html
  2. Set editor as readonly CKEDITOR.instances.editor1.setReadOnly( true )
  3. Focus image widget
  4. Blur editor
  5. Focus image widget once again
  6. Press delete/backspace

Expected result

Image widget shouldn't change.

Actual result

Image widget has been deleted.

Other details

Needed to fix #1489

  • Browser: Any
  • OS: Any
  • CKEditor version: Any
@jacekbogdanski jacekbogdanski added the type:bug A bug. label Jan 29, 2018
@mlewand mlewand changed the title Fake selection allows to delete widget with readonly mode Fake selection allows for removing in a readonly mode Jan 29, 2018
@mlewand mlewand added status:confirmed An issue confirmed by the development team. target:minor Any docs related issue that can be merged into a master or major branch. labels Jan 29, 2018
@mlewand
Copy link
Contributor

mlewand commented Jan 29, 2018

This is also reproducible with table selection.

@jacekbogdanski jacekbogdanski mentioned this issue Jan 30, 2018
Merged
2 tasks
@Comandeer
Copy link
Member

It's also possible to cut widget after blurring and refocusing.

@Comandeer Comandeer mentioned this issue Feb 2, 2018
Closed
@Comandeer Comandeer added this to the 4.9.0 milestone Feb 6, 2018
@Comandeer Comandeer mentioned this issue Feb 7, 2018
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
status:confirmed An issue confirmed by the development team. target:minor Any docs related issue that can be merged into a master or major branch. type:bug A bug.
Projects
None yet
Milestone
4.9.0
Development

Successfully merging a pull request may close this issue.

Fixed: fake selection allows for removing in a readonly mode ckeditor/ckeditor4
3 participants
@Comandeer @mlewand @jacekbogdanski