Skip to content

Conversation

cjcenizal
Copy link
Contributor

Instead the user can scroll to bring clipped nav items into view.

NOTE: I haven't been able to test this in IE due to problems with my VM.

Original problem

sidebar_links_collide

Solution

sidebar_links_collide_fix

… the window is very short, and instead allows the user to scroll.
@cjcenizal cjcenizal force-pushed the improvement/scrolling-nav-overflow branch from 0256933 to 9ce2cd5 Compare September 9, 2016 03:33
@ppisljar
Copy link
Contributor

ppisljar commented Sep 9, 2016

i tested this in IE11 and EDGE ... in both it works ok.
code also looks fine.
LGTM

@bevacqua
Copy link
Contributor

bevacqua commented Sep 9, 2016

LGTM

@bevacqua bevacqua removed their assignment Sep 9, 2016
@cjcenizal cjcenizal merged commit 0718a94 into elastic:master Sep 9, 2016
@cjcenizal cjcenizal deleted the improvement/scrolling-nav-overflow branch September 9, 2016 15:19
elastic-jasper added a commit that referenced this pull request Sep 9, 2016
---------

**Commit 1:**
Add globalNav styles that prevent the nav items from overlapping when the window is very short, and instead allows the user to scroll.

* Original sha: 9ce2cd5
* Authored by CJ Cenizal <cj@cenizal.com> on 2016-09-09T03:28:45Z
@cjcenizal cjcenizal removed the review label Sep 12, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants