Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fauxbar Sub-menus not showing #36

Open
seanryan-seanryan opened this issue Apr 30, 2018 · 1 comment
Open

Fauxbar Sub-menus not showing #36

seanryan-seanryan opened this issue Apr 30, 2018 · 1 comment
Labels

Comments

@seanryan-seanryan
Copy link

On the Fauxbar page, sub-menus for Bookmarks and Extensions are not visible or accessible: when you hover over - or click on - an entry containing a sub-menu arrow icon you can see about 1mm of the sub-menu's left hand border showing just left of the main menu's vertical scroll-bar. It's as if the sub-menu is hidden under the vertical scroll bar (though of course it's expected that the sub-menu should pop up over / to the right of the scroll-bar).
The sub-menu pops up correctly for all menus where the menu has no scroll-bar, and does not pop up for any menu where the menu does contain a scroll-bar.

@ChrisNZL ChrisNZL added the bug label Apr 30, 2018
@ChrisNZL ChrisNZL added this to the 1.8.0 milestone May 22, 2019
@ChrisNZL
Copy link
Owner

Seems to be an issue with CSS' overflow property,
menubar.js, line ~388

$(this).addClass('exceededMaxHeight') .css('height',maxHeight+'px') .css('overflow-y','scroll') .css('overflow-x','visible') .css('top',$('#menubar').height()+4+'px');

https://stackoverflow.com/a/6433475/206410

Sounds like CSS won't work with nested child elements and scrollbars for this. Probably need to refactor to have each column be placed absolutely side-by-side rather than nested as children.

@ChrisNZL ChrisNZL removed this from the 1.8.0 milestone May 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants