?
Referenced as Pull Request for: # 10987
avatar viocassel
viocassel
29 Jun 2016

Admin menu dropdown in Joomla! 3.5.1

Admin menu dropdown in Joomla! 3.6.0 RC1

This is a bug or a feature? ?

Votes

# of Users Experiencing Issue
3/3
Average Importance Score
4.00

avatar viocassel viocassel - open - 29 Jun 2016
avatar brianteeman
brianteeman - comment - 29 Jun 2016

I assume you refer to the missing pointer?


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10972.

avatar brianteeman brianteeman - change - 29 Jun 2016
Category Templates (admin) UI/UX
avatar viocassel
viocassel - comment - 29 Jun 2016

Yes. And dropdowns appear crooked.

avatar brianteeman
brianteeman - comment - 29 Jun 2016

Probably related to #9375

On 29 June 2016 at 22:49, Vio Cassel notifications@github.com wrote:

Yes. And dropdowns appear crooked.

https://camo.githubusercontent.com/65c853f6b87aeb80831d5c64052b81aaddfc0a83/687474703a2f2f696d6167652e70726e747363722e636f6d2f696d6167652f30363838633766376637356234653031393933373536626666366533646638622e706e67


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#10972 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/ABPH8SMU2fs6ex4HRjDxp-VQjeL-Es7eks5qQuhUgaJpZM4JBmAA
.

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar StefanSTS
StefanSTS - comment - 30 Jun 2016

.navbar .nav > li ul has the overflow-y and overflow-x set to auto and hidden, this causes that the triangle is not appearing, since it is attached to the ul with the pseudo selector. It is positioned outside the ul container and is therefore not shown.

A solution could be to attach it to the higher level li.dropdown.open with the :after selector. Haven't tried that though.

avatar StefanSTS
StefanSTS - comment - 30 Jun 2016

For the difference in height for the second ul: There is a margin: 2px 0 0; on .dropdown-menu. This causes the submenu to move down 2px while the first dropdown has that overridden in .navbar .nav > li ul with margin: 0;.
Suggested solution would be to remove the 2px top margin on .dropdown-menu.

avatar brianteeman brianteeman - change - 30 Jun 2016
Labels Added: ?
avatar brianteeman
brianteeman - comment - 30 Jun 2016

@joomlic could you take a look please


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10972.

avatar RonakParmar
RonakParmar - comment - 1 Jul 2016

I have checked in Joomla 3.5.1 and latest Joomla Staging branch and I am able to reproduce this issue.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10972.

avatar RonakParmar
RonakParmar - comment - 1 Jul 2016

I have changes administrator template to "hathor" and menu selection is working fine in this template, but having issue in "isis" template.


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10972.

avatar gunjanpatel gunjanpatel - change - 1 Jul 2016
Status New Confirmed
avatar kalpesh681
kalpesh681 - comment - 1 Jul 2016

I am able to reproduce same issue


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10972.

avatar JoomliC
JoomliC - comment - 1 Jul 2016

@brianteeman i don't know exactly why you request me here to have a look... maybe you meant @C-Lodder ;-)

I have checked a bit, and you're right, issue is related to #9375
The missing pointer is due to the overflow set to the all container (ul) : https://github.com/joomla/joomla-cms/pull/9375/files#diff-92bd4bc9fc868397bd2dcf5f303c0ec4R7160
And the extra margin at top of sub menu : https://github.com/joomla/joomla-cms/pull/9872/files#diff-6337832bd45e0f9af1521049cb78e03eR77 (a PR which was supposed to fix the 2 issues here already reported here : #9841 )
Here, i wonder why +5 (if set to 8, it's ok for sub menu top placement, but i don't think using value inside the script will be consistent in the future...) or -13 in script template.js ?

As no PR yet, i have created one #10987 (@StefanSTS you fix proposal for missing pointer is the right one! ? )
@C-Lodder could check this PR too ;-)

Thanks!

avatar brianteeman brianteeman - change - 1 Jul 2016
Status Confirmed Pending
avatar brianteeman
brianteeman - comment - 1 Jul 2016

Please test #10987


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10972.

avatar brianteeman
brianteeman - comment - 3 Jul 2016

Closed as we have a PR


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/10972.

avatar brianteeman brianteeman - change - 3 Jul 2016
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2016-07-03 06:59:41
Closed_By brianteeman
avatar brianteeman brianteeman - close - 3 Jul 2016

Add a Comment

Login with GitHub to post a comment