- Issue created by @melissag
- 🇮🇩Indonesia gausarts
Thank you.
Yes, it is 3.x intentional feature to avoid CSS complexity with media queries device width detection.
Since 3.0.6, you can just as easily override the ulimenu CSS files if fine-grained layouts are required, see CR:
https://www.drupal.org/node/3447576#ultimenu-touch-replacement →However I see a point if users want to make it optional, that is, to make a configurable breakpoint for different menu displays.
Feel free to patch, or chime in for 2-3-hour works, as a new feature.
- First commit to issue fork.
- Merge request !6Issue #3514188: Hover class being removed on dektop touch screen fix → (Open) created by hablat
- 🇺🇸United States hablat
We ran into this issue as well where we have laptops that have touch screens. The javascript seems to assume any touchscreen device is a mobile device. In our case it wasn't and the is-ultihover class was being removed on touch screen laptops. Created a patch to check, using media query, if the device has no hover capability before removing the class. This seems to solve it for us. Added is a MR to the ticket
- 🇮🇩Indonesia gausarts
Thank you.
Your fix should be in another thread, because #2 is more than your enhancement. I don't mind it here, though. We'll change it back to a feature after commiting yours.
ATM, there are two
data.hover
in it, please update them both. Not sure if moving it intovar data
will be equally working, other than more efficient. Feel free to decide. - 🇺🇸United States hablat
@gausarts sorry for the mixup on the ticket and thanks for being flexible. I've added the change to the other data.hover you mentioned. I hope this is what you were looking for. Thanks.
- 🇮🇩Indonesia gausarts
There are options at /admin/block/ULTIMENUBLOCK like
Always use hamburger
orUse caret
which appears to result in a conflict against your logic.Try having the same menu (clones) with different/ conflicting options as above, but placed at different pages to immediately see if any regression.
If you or anyone can verify any combo of those options are good, that would be very helpful. Otherwise I may need some time to crosscheck them.
Thank you again.