search

keycdn / cache-enabler

A lightweight caching plugin for WordPress that makes your website faster by generating static HTML files.
https://wordpress.org/plugins/cache-enabler/
123 stars 46 forks source link

Admin toolbar navigation entries messing up layout #41

Closed rexblack closed 5 years ago

rexblack commented 5 years ago

First of all, thanks for providing CacheEnabler-Plugin. It works great and appears to be the best solution I tried so far.

When having CacheEnabler, Autoptimizer plus (probably) one other Plugin (e.g. Yoast SEO which itself only shows an icon here) being displayed on the adminbar, it messes up layout for smaller desktop screens.

bildschirmfoto 2018-10-24 um 11 33 07

Actually one could hardly call this a bug and also, it's not related to CacheEnabler in particular. Moreover it is because of the plugin nature of Wordpress and its dynamic toolbar. Might also not be seen as critical because only a certain range of dimensions is affected in some cases. It's a common issue with header navigation having lots of entries, in particular regarding localization.

But since it could be easily fixed or let's say "disarmed" on your side, it would be great if this could be improved.

Possible Solutions would of course be, e.g. Using an icon with the label as title-tooltip or having both menu items collapsed in a dropdown.

TomaszGasior commented 5 years ago

I am against using icon. Text label is more readable and consistent.

This should be fixed in WordPress core. WordPress front-end scripts should deal with situations like this, in way similar to desktop applications: using popup menu. See: https://i.imgur.com/3WdV9Ve.png -> https://i.imgur.com/Ud3j31l.png I am suggesting filling bug against WordPress and closing this one.

svenba commented 5 years ago

@TomaszGasior agreed.