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). What is the purpose of the padding change? I see the padding is now narrower, but the bar is still thicker than in 25. The removal of dedicated CSS configurations for system settings looks nice. I even think that we might just drop our support for the dark theme if users don't allow javascript. So the changes look good to me in general. ------------- PR Comment: https://git.openjdk.org/jdk/pull/28085#issuecomment-3475117066
