Integrated: JDK-8276239: Better tables in java.util.random package summary

2021-11-08 Thread Ludvig Janiuk
On Mon, 1 Nov 2021 17:10:56 GMT, Ludvig Janiuk wrote: > The tables are now striped, and they use row headers (which is a nice-to-have > for accessibility). This pull request has now been integrated. Changeset: d8b0dee6 Author:Ludvig Janiuk Committer: Claes Redestad URL:

Re: RFR: JDK-8276239: Better tables in java.util.random package summary

2021-11-08 Thread Ludvig Janiuk
On Mon, 1 Nov 2021 17:17:18 GMT, Jim Laskey wrote: >> The tables are now striped, and they use row headers (which is a >> nice-to-have for accessibility). > > Marked as reviewed by jlaskey (Reviewer). @JimLaskey Could I ask you to sponsor? - PR:

Integrated: JDK-8276653: Missing row headers in j.l.Character docs

2021-11-04 Thread Ludvig Janiuk
On Thu, 4 Nov 2021 16:52:48 GMT, Ludvig Janiuk wrote: > The first table is missing row headers, adding them to aid screen readers. This pull request has now been integrated. Changeset: 8ec80c4b Author: Ludvig Janiuk Committer: Naoto Sato URL: https://git.openjdk.java.net/jdk/com

Re: RFR: JDK-8276649: MethodHandles.Lookup docs: replace the table in the cross-module access check section with list

2021-11-04 Thread Ludvig Janiuk
On Thu, 4 Nov 2021 17:03:54 GMT, Mandy Chung wrote: >> Inferring from the flow of the text, the table should have been a list all >> along, so I've made it that way. Before and after for comparison: >>

Integrated: JDK-8276652: Missing row headers in MethodHandles.Lookup docs

2021-11-04 Thread Ludvig Janiuk
On Thu, 4 Nov 2021 16:41:11 GMT, Ludvig Janiuk wrote: > Added row headers missing in the table captioned "Access mode summary". This > carries no visual change, but might be useful for screen readers. This pull request has now been integrated. Changeset: 7e87f946 Author:

Integrated: JDK-8276649: MethodHandles.Lookup docs: replace the table in the cross-module access check section with list

2021-11-04 Thread Ludvig Janiuk
On Thu, 4 Nov 2021 16:18:45 GMT, Ludvig Janiuk wrote: > Inferring from the flow of the text, the table should have been a list all > along, so I've made it that way. Before and after for comparison: > ![image](https://user-images.githubusercontent.com/6298393/140376551-6f8ac0c9-5336-

Re: RFR: JDK-8276649: MethodHandles.Lookup docs: replace the table in the cross-module access check section with list

2021-11-04 Thread Ludvig Janiuk
On Thu, 4 Nov 2021 16:18:45 GMT, Ludvig Janiuk wrote: > Inferring from the flow of the text, the table should have been a list all > along, so I've made it that way. Before and after for comparison: > ![image](https://user-images.githubusercontent.com/6298393/140376551-6f8ac0c9-5336-

RFR: JDK-8276653: Missing row headers in j.l.Character docs

2021-11-04 Thread Ludvig Janiuk
The first table is missing row headers, adding them to aid screen readers. - Commit messages: - Added row headers in Character docs Changes: https://git.openjdk.java.net/jdk/pull/6262/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk=6262=00 Issue:

RFR: JDK-8276652: Missing row headers in MethodHandles.Lookup docs

2021-11-04 Thread Ludvig Janiuk
Added row headers missing in the table captioned "Access mode summary". This carries no visual change, but might be useful for screen readers. - Commit messages: - Added table headers Changes: https://git.openjdk.java.net/jdk/pull/6261/files Webrev:

RFR: JDK-8276649: MethodHandles.Lookup docs: table could be list

2021-11-04 Thread Ludvig Janiuk
Inferring from the flow of the text, the table should have been a list all along, so I've made it that way. Before and after for comparison: ![image](https://user-images.githubusercontent.com/6298393/140376551-6f8ac0c9-5336-4da0-8504-01af806ecae0.png)

Integrated: JDK-8276236: Table headers missing in Formatter api docs

2021-11-02 Thread Ludvig Janiuk
On Mon, 1 Nov 2021 15:59:22 GMT, Ludvig Janiuk wrote: > Adds table headers to all tables in Formatter api docs. I inferred the header > "Unicode" for the columns that contained unicode > codepoints. > > Formatter api docs: > https://docs.oracle.com/en/java/javas

RFR: JDK-8276239: Better tables in java.util.random package summary

2021-11-01 Thread Ludvig Janiuk
The tables are now striped, and they use row headers (which is a nice-to-have for accessibility). - Commit messages: - Striped tables and table row headers Changes: https://git.openjdk.java.net/jdk/pull/6196/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk=6196=00

RFR: JDK-8276236: Table headers missing in Formatter api docs

2021-11-01 Thread Ludvig Janiuk
Adds table headers to all tables in Formatter api docs. I inferred the header "Unicode" for the columns that contained unicode codepoints. Formatter api docs: https://docs.oracle.com/en/java/javase/17/docs/api/java.base/java/util/Formatter.html - Commit messages: - Fixes