User tests: Successful: Unsuccessful:
See #7587 for instructions.
After patch I get the subs OK:
I have added the new css:
.nav-collapse.in.collapse {
overflow: visible;
height: 0;
}
in /media/jui/less/responsive-navbar.less and ran generatecss.php
If judged necessary we could limit it to protostar only.
Rel_Number | ⇒ | 7587 | |
Relation Type | ⇒ | Pull Request for |
Category | ⇒ | Templates (site) |
My screenshot was with chrome
On 5 August 2015 at 10:40, infograf768 notifications@github.com wrote:
I do not get this here (Firefox)
[image: screen shot 2015-08-05 at 11 38 27]
https://cloud.githubusercontent.com/assets/869724/9082799/9d41de00-3b66-11e5-8ae2-b092d98d3d2f.png—
Reply to this email directly or view it on GitHub
#7629 (comment).
Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/
I confirm on chrome and safari with very long menu aliases. IMHO not a bid deal but this is a different issue than the one solved here. Not sure how to solve.
Awesome guys! The fix given here fa38882 to these files fixes the issue on my site. If you wish to test or verify: www.djexplosionsd.com
I don't know if I should start a new tracker, but is there also a way to fix the time on how long a parent menu with child menus can be pressed on a phone before it becomes active?
For example, press the "MUSIC" menu on my site with an Android phone, and you'll see a child menu under it. But it opens briefly not allowing you to select anything else since the parent menu also takes you to a page. Hopefully you understand what I mean.
If judged necessary we could limit it to protostar only.
Yes, please do it only in Protostar (and maybe ISIS).
The /media/jui/less/responsive-navbar.less files is coming from Bootstrap and the only changes we made to it are to cope with the MooTools modals.
Labels |
Added:
?
|
Can you test by first patching with this PR, then adding the z-index in protostar template.css, line 7410:
@media (max-width: 979px) {
.nav-collapse.in.collapse {
overflow: visible;
height: 0;
z-index: 100;
}
}
Current PR fixes the issue for Protostar. Thanks!
@infograf768 yes that z-index works in chrome
Status | New | ⇒ | Ready to Commit |
RTC. Thanks for testing.
Labels |
Added:
?
|
Status | Ready to Commit | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-08-05 16:44:42 |
Closed_By | ⇒ | rdeutz |
Labels |
Removed:
?
|
Milestone |
Added: |
Almost but not quite. See screenshot
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/7629.