1.71 has the following change to the responsive css file:
@media only screen and (min-width: 768px) {
#main-navigation {
display: block!important;
}
}
A few problems with this:
1. The biggest problem is that the media selector is wrong.
As it stands, any desktop browser size between width 768-1024 displays a hard-coded mobile navigation menu. For resized browser windows, these desktop widths aren’t uncommon. Also, iphone widths aren’t this big even in landscape.
2. Because !important was used, the toggle button doesn’t actually toggle the menu, since the state of the mobile menu is now hardcoded to open for any width that triggers this rule, which makes it worse.
I don’t fully understand the intent: to default the iphone landscape menu to open, possibly? If you want to do this for some reason, here’s the correct code to target iphones in landscape mode:
@media only screen and (min-device-width: 320px) and (max-device-width: 568px) and (orientation: landscape) {
#main-navigation {
display: block;
}
}
I tested this out and it works, defaulting the iphone menu to open in landscape mode while still being toggleable. Again, not sure why you’d want this, but it’s in the code, so I guess someone does!
Attached is an image of our site on a desktop with the locked mobile menu.
Can this either be removed or updated to the code above in the next release? Thanks for your time.
Wordpress 3.8, 1.71
Home
.