Skip to content
This repository has been archived by the owner on Apr 2, 2021. It is now read-only.

Resizing images does not work with pretty theme #784

Open
matthijskooijman opened this issue Mar 7, 2017 · 1 comment · May be fixed by #787
Open

Resizing images does not work with pretty theme #784

matthijskooijman opened this issue Mar 7, 2017 · 1 comment · May be fixed by #787

Comments

@matthijskooijman
Copy link

matthijskooijman commented Mar 7, 2017

When using the pretty theme, I can't seem to resize images properly. When hovering over the image, a yellow border appears, that says "drag this to resize" in the bottom, but a part of the yellow border is obscured by a white area:

selection_005

When I move my mouse down to the "drag this to resize", it passes over the white area and the yellow border disappears. If I carefully navigate the mouse over the left side of the yellow border, around the white area, I can reach the "drag this to resize" and resize the image as expected.

The above was tested using Chrome 49 and this example: https://wymeditor.github.io/wymeditor/dist/examples/17-pretty-theme.html. In Firefox 45, I see something similar in my embedded version of wymeditor, though in that particular example (as well as the basic example), resizing does not work at all with Firefox.

@matthijskooijman
Copy link
Author

I just also tried in Chrome 56, which shows the same behaviour. I also noticed that, depending on the positioning of the image, the white bar is sometimes completely covering the yellow, even the sides, making the workaround useless.

matthijskooijman added a commit to matthijskooijman/wymeditor that referenced this issue Mar 7, 2017
These elements are not intended as part of the content to be edited, so
there is no point in showing annotations on them.

In particular, the annotations were previously added on the image resize
handle, partly obscuring it from view and preventing the mouse pointer
from reaching the handle, making image resizing impossible using the
pretty theme.

This fixes wymeditor#784.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant