On Fri, 31 Oct 2025 13:30:06 GMT, Hannes Wallnöfer <[email protected]> wrote:
> Please review a change to simplify the implementation of CSS themes by solely > relying on an attribute set by script instead of the dual script plus media > queries approach. This allows us to get rid of duplicate declaration of CSS > theme variables. On the downside, JavaScript must be enabled to recognize the > system theme, but this is an acceptable compromise as documentation remains > fully usable with JavaScript disabled. This change also makes it much easier > for users to override or extend JavaDoc stylesheets. > > Also included is a fix for the keyboard tab order of the theme switcher. When > opening the theme switcher and hitting the tab key, focus now goes to the > theme switcher and no longer goes to the first link in the breadcrumb > navigation if there is one. This required moving the HTML for the theme > switcher next to the theme button, which required a whole series of test > changes. I considered removing the theme switcher markup from these tests, > but decided against it, mostly to make sure there are no other unchecked > links in the navigation bar. This fix also required reducing the `z-index` of > the mobile table of contents. > > Another improvement related to focus handling is to close the theme switcher > when it loses focus, and when the `Enter` or `Escape` keys are pressed. Also > included is a small change to slightly reduce the padding of the recently > introduced TOC sort button. > > Sample documentation [can be viewed > here](https://cr.openjdk.org/~hannesw/8370612/api.00/java.base/module-summary.html). This pull request has now been integrated. Changeset: 49f51f94 Author: Hannes Wallnöfer <[email protected]> URL: https://git.openjdk.org/jdk/commit/49f51f9450ac3b923f83ba7d9089e5560e25ec7a Stats: 468 lines in 9 files changed: 49 ins; 354 del; 65 mod 8370612: Simplify implementation of dark theme 8371021: Tab order in theme picker is broken Reviewed-by: jlamperth, liach ------------- PR: https://git.openjdk.org/jdk/pull/28085
