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

Add/edit Radio button, enabled status is not saved under IE 9-11 #834

Closed
jswiderski opened this issue Aug 28, 2017 · 2 comments · Fixed by #845
Closed

Add/edit Radio button, enabled status is not saved under IE 9-11 #834

jswiderski opened this issue Aug 28, 2017 · 2 comments · Fixed by #845
Assignees
@msamsel
Labels
status:confirmed An issue confirmed by the development team. support An issue reported by a commercially licensed client. target:minor Any docs related issue that can be merged into a master or major branch. type:bug A bug.
Milestone
4.8.0

Comments

@jswiderski
Copy link
Contributor

Are you reporting a feature request or a bug?

Bug

Check if the issue is already reported

Provide detailed reproduction steps (if any)

  1. Open any sample
  2. Clear content
  3. Press form element -> radio button
  4. Fill all fields and check "enabled"
  5. Press OK

Expected result

Radio button is added to edit area, with Enabled status

Actual result

Radio button is enabled visually, but after viewing the source of radio button in dev-tools checked="checked" is not there. Of course after double click on radio, status Enabled is not set in dialog window as well.

Other details

  • Browser: IE9-11
  • OS: Windows
  • CKEditor version: 3.0+
  • Installed CKEditor plugins: Forms

We have fixed this issue recently for checkboxes.
@jswiderski jswiderski added type:bug A bug. support An issue reported by a commercially licensed client. labels Aug 28, 2017
@mlewand mlewand added this to the Backlog milestone Aug 28, 2017
@mlewand mlewand added the status:confirmed An issue confirmed by the development team. label Aug 28, 2017
@msamsel msamsel self-assigned this Aug 29, 2017
@msamsel
Copy link
Contributor

msamsel commented Aug 29, 2017 •
edited
Loading

Upstream: https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/13497247/
Edge 15 and 16 works properly problem seems to exists only on IE and Edge <= 14

@msamsel msamsel mentioned this issue Aug 30, 2017
Merged
2 tasks
@mlewand mlewand added the target:minor Any docs related issue that can be merged into a master or major branch. label Aug 30, 2017
Comandeer added a commit that referenced this issue Sep 12, 2017
@Comandeer
Merge pull request #845 from ckeditor/t/834
b2f48a3 
[#834] Fix radio button selection problem when used in IE browser.
@mlewand
Copy link
Contributor

mlewand commented Sep 14, 2017

Fixed with #845.

@mlewand mlewand closed this as completed Sep 14, 2017
@mlewand mlewand modified the milestones: Backlog, 4.8.0 Sep 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@msamsel msamsel

Labels
status:confirmed An issue confirmed by the development team. support An issue reported by a commercially licensed client. 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.8.0
Development

Successfully merging a pull request may close this issue.

[#834] Fix radio button selection problem when used in IE browser. ckeditor/ckeditor4
3 participants
@jswiderski @mlewand @msamsel