> Please review a change to add a collapsible "hamburger" navigation bar to > javadoc pages for small screens. The collapsible navbar is activated when the > browser width is below 920px, which is the point where the normal navigation > bar starts to wrap for some javadoc pages. > > The feature is implemented using CSS and JavaScript techniques employed by > popular frameworks and has been successfully tested on several browsers and > platforms, including various Android and iOS devices as well as IE and Edge > browsers on Windows 10. > > Sample documentation generated with this change is provided at the URL below. > For testing, please review it on a mobile phone or tablet, or using the > device emulation features provided with the developer tools of many desktop > browsers. > > http://cr.openjdk.java.net/~hannesw/8273034/api.00/index.html > > One peculiarity of this change is that the sub-navigation links are rendered > separately for the normal and the mobile navbar. The reason for this is that > the links are structured differently (flat list vs nested lists) and reside > in different containers. > > On the CSS side, apart from the collapsible navbar I overhauled the styles > for the search input, especially the small reset "X" button. The positioning > of the reset button is now simpler and more robust by using `absolute` > instead of `relative` position, and doesn't require moving the search input > and label to the right anymore. > > The collapsible menu uses the `aria-controls`, `aria-expanded` attributes as > described in the following wiki page: > > https://www.w3.org/WAI/GL/wiki/Using_the_WAI-ARIA_aria-expanded_state_to_mark_expandable_and_collapsible_regions > > Furthermore, the `aria-label` attribute is used to provide a human-readable > lable for the button. > > https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/ARIA_Techniques/Using_the_aria-label_attribute
Hannes Wallnöfer has updated the pull request incrementally with one additional commit since the last revision: JDK-8273034: tweak button height ------------- Changes: - all: https://git.openjdk.java.net/jdk/pull/5360/files - new: https://git.openjdk.java.net/jdk/pull/5360/files/0c027e79..da47f090 Webrevs: - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=5360&range=01 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=5360&range=00-01 Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod Patch: https://git.openjdk.java.net/jdk/pull/5360.diff Fetch: git fetch https://git.openjdk.java.net/jdk pull/5360/head:pull/5360 PR: https://git.openjdk.java.net/jdk/pull/5360
