Topbar 2nd level bugfix #1169

Merged
merged 2 commits into from Nov 15, 2012

Conversation

Projects
None yet
3 participants

EdCharbeneau added some commits Nov 12, 2012

@EdCharbeneau EdCharbeneau Added multiple dropdowns to topbar test
Added multiple dropdowns to test for reproducing a bug with multiple
drop down behaviors on mobile. Main Item 3 >Dropdown Level 1a & 1b

Opening Level 1a on mobile reveals both Level 1a & 1b
d27f7e5
@EdCharbeneau EdCharbeneau Fixed bug with multiple 2nd level dropdowns 5008160
Contributor

mvrhov commented Nov 13, 2012

I reported this about a month ago and I was told it's by design #1036. I still disagree.

Contributor

EdCharbeneau commented Nov 13, 2012

Considering that the dropdowns occlude each other and break completely when you go back up the tree, I would have to disagree as well.

@hatefulcrawdad hatefulcrawdad pushed a commit that referenced this pull request Nov 15, 2012

@zurbchris zurbchris Merge pull request #1169 from EdCharbeneau/topbar-2nd-level-bugfix
Topbar 2nd level bugfix
fd7d673

@hatefulcrawdad hatefulcrawdad merged commit fd7d673 into zurb:master Nov 15, 2012

Thanks @EdCharbeneau! @mvrhov, your issue was different than this one.

Contributor

EdCharbeneau commented Nov 16, 2012

Welcome, I needed it fixed for my site, so I might as well share it back.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment