Re: RFR: JDK-8325874: Improve checkbox-based interface in summary pages [v2]

2024-03-13 Thread Hannes Wallnöfer
> Please review a change to improve the user experience and implementation for > the checkbox-based interface to selectively display content on API summary > pages: > > - Add a checkbox to toggle (select/unselect) all checkboxes in the page. > - Remove the

Re: RFR: JDK-8325874: Improve checkbox-based interface in summary pages

2024-03-13 Thread Pavel Rappo
On Wed, 13 Mar 2024 14:30:01 GMT, Hannes Wallnöfer wrote: > > if you check "all" and then uncheck something else, then "all" will remain > > selected. That is a confusing state. > > I agree, but I think the behavior of the checkbox is as it should be, and > it's the label that is slightly

Re: RFR: JDK-8325874: Improve checkbox-based interface in summary pages

2024-03-13 Thread Hannes Wallnöfer
On Mon, 11 Mar 2024 18:09:41 GMT, Pavel Rappo wrote: > if you check "all" and then uncheck something else, then "all" will remain > selected. That is a confusing state. I agree, but I think the behavior of the checkbox is as it should be, and it's the label that is slightly misleading. The

Re: RFR: JDK-8325874: Improve checkbox-based interface in summary pages

2024-03-13 Thread Pavel Rappo
On Mon, 11 Mar 2024 13:37:54 GMT, Hannes Wallnöfer wrote: > Please review a change to improve the user experience and implementation for > the checkbox-based interface to selectively display content on API summary > pages: > > - Add a checkbox to toggle (select/unselect) all checkboxes in

Re: RFR: JDK-8325874: Improve checkbox-based interface in summary pages

2024-03-11 Thread Pavel Rappo
On Mon, 11 Mar 2024 13:37:54 GMT, Hannes Wallnöfer wrote: > Please review a change to improve the user experience and implementation for > the checkbox-based interface to selectively display content on API summary > pages: > > - Add a checkbox to toggle (select/unselect) all checkboxes in

RFR: JDK-8325874: Improve checkbox-based interface in summary pages

2024-03-11 Thread Hannes Wallnöfer
Please review a change to improve the user experience and implementation for the checkbox-based interface to selectively display content on API summary pages: - Add a checkbox to toggle (select/unselect) all checkboxes in the page. - Remove the `Table.setAlwaysShowDefaultTab(boolean)` setter