search

GetmeUK / ContentTools

A JS library for building WYSIWYG editors for HTML content.
http://getcontenttools.com
MIT License
3.94k stars 393 forks source link

Fix for tiny toolbox bug #541

Closed Cweet closed 4 years ago

Cweet commented 5 years ago

Hi again! I noticed that sometimes when tried to click on the Heading element it would select lists instead. It was really annoying. Then I realized since I have a translated version and my tooltips are a bit longer and breaking into two lines, it sometimes covers the other buttons before they are visible.

Here are some screenshots: Without fix, hitting the tooltip selects the toolstips element:

Skjermbilde 2019-07-08 kl 22 18 14

After fix, selecting through the tooltip.

Skjermbilde 2019-07-08 kl 22 18 40

A small fix for this is to add pointer-events: none; to the tooltips:after element :) Hope this can be in the next patch as well for others who might not have figured out why. I also added some styling to stop the wordbreak.

.ct-tool:after {
     pointer-events: none;

     width: initial;
     max-width: 150px;
     white-space: nowrap;
}
anthonyjb commented 5 years ago

@Candielope thank you for the report and taking the time to offer a solution :+1:

anthonyjb commented 4 years ago

@Candielope just wanted to thank you again for the report and suggestion this fix has been pushed and will be in the next release.