On Thu, 30 Jun 2022 15:14:40 GMT, Hannes Wallnöfer <hann...@openjdk.org> wrote:
> Please review an enhancement to the preview page to add a list of preview > features and allow users to explore the preview APIs by feature. > > While the changes for the enhancement itself are not overly complex, the work > entailed a moderate cleanup of other summary API lists (Deprecated and New > API pages) as well as a few unrelated summary pages that happened to share > some of the CSS classes and HTML structure. > > The change itself starts with a new internal > `jdk.internal.javac.PreviewFeature.JEP` annotation interface to add JEP > information to the nearby `jdk.internal.javac.PreviewFeature.Feature` enum. > This JEP information is retrieved by new methods in the javadoc `Utils` class > and then fetched by `PreviewAPIListBuilder` which passes it on to > `PreviewListWriter`. > > The superclass of `PreviewListWriter` and other API summary writer classes, > `SummaryListWriter`, gets a new `addContentSelectors(Content)` method to add > the checkboxes to show/hide various parts of the page content. The class is > also now abstract as this and other methods do not have useful > implementations in the base class. > > Another change to `SummaryListWriter` is that `pageMode`, `description`, > `headContent` and `titleKey` are no longer fields in the class but rather > sent to the `generateSummaryListFile` method as parameters since they are > mostly just used locally in that method. On the other hand, the associated > Builder is used in many places and previously had to be retrieved from the > configuration object or passed around as parameter, so that is now set as a > final field in `SummaryListWriter`. > > Finally, a few words about changes in CSS and HTML: The "Contents" list > containing various kinds of elements in the API list was previously contained > in the `<div class="header">` element that also contains the page header, and > the stylesheet used the `.header ul` selector to style the list. Now that > the checkboxes separate the page header from the contents list it felt wrong > to put all this in the header div (most other pages only use the div for the > header itself). I also didn't like the indirect CSS selector too much, > `.header ul` is not very indicative of what it applies to. > > I therefore decided to create a new dedicated CSS class for the contents list > called `contents-list`, and move the `<ul>` element out of the header div. > Apart from the summary API pages, this change also affects the "Constant > Values" page as well as the top and package level "Tree" (class hierarchy) > pages. I made sure the contents list in these pages look the same as before. > > The list items in the contents list now each have an `id` attribute to allow > hiding the list item if no content for the element kind is currently > selected/visible. The value of the `id` attribute uses the format > `contents-<element-kind>`. > > There is one additional CSS class called `preview-feature-list` used for the > list of preview feature JEPs. > > Demo output for the new preview page as well as other pages is available here: > http://cr.openjdk.java.net/~hannesw/8287597/api.03/preview-list.html This pull request has been closed without being integrated. ------------- PR: https://git.openjdk.org/jdk/pull/9336
