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.

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

Color picker for forecolor unusable because it is clipped by browser window #3639

Closed
davidearl opened this issue May 11, 2017 · 2 comments
Closed

Comments

@davidearl
Copy link

Do you want to request a feature or report a bug?

bug

What is the current behavior?

Using the forecolor menu item for textcolor, when it is the last button in the row and the row is more-or-less the same width as the browser window, then the color picker is largely clipped by the browser window and therefore unusable

This is a particular problem in full window mode on narrow-ish windows, and especially on smaller screens with the browser set as wide as it will go, with a long row of buttons where the color picker happens to be last in a group and the group is last in the row, which is only just wide enough to fit the window. In that case, the user is essentially stuck if they want to use this button.

colorpickerclipped

**If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via fiddle.tinymce.com or similar.

https://codepen.io/anon/pen/YVaYaL

What is the expected behavior?

I'd expect it to adjust the color picker position so as to be entirely visible, not to be off the edge.

Which versions of TinyMCE, and which browser / OS are affected by this issue? Did this work in previous versions of TinyMCE?

The one in your codepen fails, as does 4.5.7 in my own project. Certainly Chrome and Firefox: I don't suppose it is browser specific.

@tinydylan
Copy link
Contributor

It looks like this issue has been corrected since you logged it. If the issue still occurs, please let us know.

@davidearl
Copy link
Author

Yes, the color picker has been completely changed in 5. I can confirm it works now. Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants