I have noticed the viewport meta tag is included by default with this module. Now I understand it is documented in the module about maybe removing this and have themes manage the viewport tag, and that many in fact do not.
Unfortunately some themes are hard-coding the viewport tag right in the html.tpl.php template, such as Zen. This makes it impossible to check for the viewport tag in preprocess and process functions, or even the template_html_head_alter() function. Otherwise we could simply search for it and choose whether to include it with Adminimal Administration Menu.
Perhaps the best option would be including a module setting to add the viewport tag? A decision to be made, tell me if you agree/disagree on this, would be whether to include the viewport tag by default. The administration menu does become rather unusable on smaller devices without it, so being added by default seems sensible to me. Then those who want to disable it can easily do so on the current module's configuration page.
I set this to a support request since the module already includes the viewport tag, but also because I want to open it up to discussion before it is considered as a feature request.
| Comment | File | Size | Author |
|---|---|---|---|
| viewport-meta-tag.png | 17.13 KB | Richard Buchanan |
Comments
Comment #2
valderama commentedI would suggest to remove the viewport statement for the module and let themes handle it.
Comment #3
szy commentedI can see double menu with Adminimal Theme now. Fresh 7.x installation.
Szy.
Comment #4
tessa bakkerThe viewport needs to be removed, this is a theme thing and also an accessibility issue, users can't zoom in.
Comment #6
tessa bakkerA setting to disable/enable the viewport is needed.
Comment #8
tessa bakkerCan be disabled now on the configuration page.
Comment #9
tessa bakker