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

Colordialog is incorrectly positioned when used with another dialog #3559

Closed
jacekbogdanski opened this issue Oct 9, 2019 · 2 comments
Closed

Colordialog is incorrectly positioned when used with another dialog #3559

jacekbogdanski opened this issue Oct 9, 2019 · 2 comments
Assignees
@Dumluregn
Labels
plugin:colordialog The plugin which probably causes the issue. plugin:dialog The plugin which probably causes the issue. regression This issue is a regression. status:confirmed An issue confirmed by the development team. type:bug A bug.
Milestone
4.13.1

Comments

@jacekbogdanski
Copy link
Member

Type of report

Bug

Provide detailed reproduction steps (if any)

  1. Open CKEditor4 sample.
  2. Open color dialog using colorbutton plugin and change its position.
  3. Close color dialog.
  4. Add a new table to the editor.
  5. Open color dialog again from cell properties dialog.

Expected result

The color dialog is responsive, correctly positioned and overflow table properties dialog. See 4.12.1 how it worked.

Actual result

The color dialog is hidden under the table properties dialog. The table dialog is not dimmed. The color dialog is not responsive at all.

colorbutton

Other details

  • CKEditor version: regression provided with 4.13.0
@jacekbogdanski jacekbogdanski added type:bug A bug. status:confirmed An issue confirmed by the development team. plugin:dialog The plugin which probably causes the issue. regression This issue is a regression. plugin:colordialog The plugin which probably causes the issue. labels Oct 9, 2019
@f1ames f1ames added the target:minor Any docs related issue that can be merged into a master or major branch. label Oct 9, 2019
@f1ames f1ames added this to the 4.13.1 milestone Oct 9, 2019
@Dumluregn Dumluregn self-assigned this Oct 9, 2019
@Dumluregn Dumluregn mentioned this issue Oct 11, 2019
Merged
2 tasks
@Dumluregn
Copy link
Contributor

For the record - regression has been introduced in #2413.

@f1ames f1ames removed the target:minor Any docs related issue that can be merged into a master or major branch. label Nov 12, 2019
@jacekbogdanski
Copy link
Member Author

Closed via #3572

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Dumluregn Dumluregn

Labels
plugin:colordialog The plugin which probably causes the issue. plugin:dialog The plugin which probably causes the issue. regression This issue is a regression. status:confirmed An issue confirmed by the development team. type:bug A bug.
Projects
None yet
Milestone
4.13.1
Development

No branches or pull requests
3 participants
@f1ames @jacekbogdanski @Dumluregn