On Tue, 14 Oct 2025 16:26:24 GMT, Nizar Benalla <[email protected]> wrote:

>> Please review this patch to add a toggle to order the member details in the 
>> table of contents in lexical order. The selected choice is stored and 
>> preserved.
>> 
>> Here is a preview of the new toggle.
>> 
>> 
>> https://github.com/user-attachments/assets/55c81e4b-5fc0-416e-8946-53aede419640
>
> Nizar Benalla has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   remove redundant css

src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/resources/script.js.template
 line 822:

> 820:     }
> 821: 
> 822:     document.addEventListener("click", function(ev){

Event listener should be added directly to the buttons, so you can avoid the 
first two lines of determining the event target.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/26322#discussion_r2432513407

Reply via email to