[OpenJDK 2D-Dev] Integrated: JDK-8272374: doclint should report missing "body" comments

2021-08-16 Thread Jonathan Gibbons
On Fri, 13 Aug 2021 03:51:23 GMT, Jonathan Gibbons wrote: > Please review a relatively simple update to have doclnt check for empty > "descriptions" -- the body of a doc comment, before the block tags. > > It is already the case that doclint checks for missing/empty de

Re: [OpenJDK 2D-Dev] RFR: JDK-8272374: doclint should report missing "body" comments [v2]

2021-08-16 Thread Jonathan Gibbons
their > over-simplistic minimal comments. It was not possible, in general, to avoid > updating the source code while preserving line numbers, so in many cases, the > golden `*.out` files had to be updated as well. > > A new test is added, focussing on the different forms of empty/

Re: [OpenJDK 2D-Dev] RFR: JDK-8272374: doclint should report missing "body" comments

2021-08-16 Thread Jonathan Gibbons
On Mon, 16 Aug 2021 17:08:13 GMT, Jonathan Gibbons wrote: >> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclint/Checker.java line >> 203: >> >>> 201: // Don't report an empty description if the >>>

Re: [OpenJDK 2D-Dev] RFR: JDK-8272374: doclint should report missing "body" comments

2021-08-16 Thread Jonathan Gibbons
On Fri, 13 Aug 2021 09:21:40 GMT, Hannes Wallnöfer wrote: >> Please review a relatively simple update to have doclnt check for empty >> "descriptions" -- the body of a doc comment, before the block tags. >> >> It is already the case that doclint checks for missing/empty descriptions in >> bloc

Re: [OpenJDK 2D-Dev] RFR: JDK-8272374: doclint should report missing "body" comments

2021-08-16 Thread Jonathan Gibbons
On Fri, 13 Aug 2021 09:20:19 GMT, Hannes Wallnöfer wrote: >> Please review a relatively simple update to have doclnt check for empty >> "descriptions" -- the body of a doc comment, before the block tags. >> >> It is already the case that doclint checks for missing/empty descriptions in >> bloc

[OpenJDK 2D-Dev] RFR: JDK-8272374: doclint should report missing "body" comments

2021-08-12 Thread Jonathan Gibbons
Please review a relatively simple update to have doclnt check for empty "descriptions" -- the body of a doc comment, before the block tags. It is already the case that doclint checks for missing/empty descriptions in block tags, like @param, @return, etc. There are three cases to consider: * T

Re: [OpenJDK 2D-Dev] RFR: 8260314: Replace border="1" on tables with CSS

2021-01-26 Thread Jonathan Gibbons
On Tue, 26 Jan 2021 18:33:48 GMT, Alexey Ivanov wrote: >>> Probably we can import the CSS used by the javadoc itself? >> >> We do. For example, [AWT Modality in JDK >> 15](https://docs.oracle.com/en/java/javase/15/docs/api/java.desktop/java/awt/doc-files/Modality.html) >> has borders because o

Re: [OpenJDK 2D-Dev] RFR: 8257733: Move module-specific data from make to respective module [v2]

2020-12-07 Thread Jonathan Gibbons
On Mon, 7 Dec 2020 14:27:45 GMT, Magnus Ihse Bursie wrote: >> A lot (but not all) of the data in make/data is tied to a specific module. >> For instance, the publicsuffixlist is used by java.base, and fontconfig by >> java.desktop. (A few directories, like mainmanifest, is *actually* used by >

Re: [OpenJDK 2D-Dev] RFR: 8252999: Cleanup: replace .equals("") with .isEmpty() within all codebase

2020-09-10 Thread Jonathan Gibbons
On Thu, 10 Sep 2020 12:04:48 GMT, Dmitriy Dumanskiy wrote: > I have in mind dozens of improvements all over the code like this one. That sounds scary. Broad updates like these cause unnecessary churn in the codebase, and can make merging and back porting harder. Changes should be discussed ah

Re: [OpenJDK 2D-Dev] [13] RFR JDK-8225368:broken links in java.desktop files

2019-06-06 Thread Jonathan Gibbons
06/06/2019 12:50, Jonathan Gibbons wrote: You should be able to use {@link} to refer to other Java elements; It is possible even across the different modules?(I remember there was some related issue) -- Jon On 06/06/2019 12:46 PM, Sergey Bylokhov wrote: Hi, Prasanta. Can you please d

Re: [OpenJDK 2D-Dev] [13] RFR JDK-8225368:broken links in java.desktop files

2019-06-06 Thread Jonathan Gibbons
Sergey, Prasanta, You should be able to use {@link} to refer to other Java elements;  you cannot (yet) use {@link} to link to user-defined anchors in other files (but it's on the list) -- Jon On 06/06/2019 12:46 PM, Sergey Bylokhov wrote: Hi, Prasanta. Can you please double check is it po

Re: [OpenJDK 2D-Dev] [10] Review Request: 8182410, 8183508, 8181289

2017-11-22 Thread Jonathan Gibbons
op: 129px;     margin-top: -129px; } so nothing specific comes from browser or " I think the next link may help you http://nicolasgallagher.com/jump-links-and-viewport-positioning/demo/ --Semyon On 11/22/2017 02:53 PM, Jonathan Gibbons wrote: Semyon, I have reconstructed a very simple, very a

Re: [OpenJDK 2D-Dev] [10] Review Request: 8182410, 8183508, 8181289

2017-11-22 Thread Jonathan Gibbons
Semyon, Using the files I previously posted, I confirm that I see the same display problem on a Mac, using the latest OS (High Sierra) and Safari. I also see the problem on the same Mac, with Firefox 55.0.2 -- Jon On 11/22/2017 02:53 PM, Jonathan Gibbons wrote: Semyon, I have

Re: [OpenJDK 2D-Dev] [10] Review Request: 8182410, 8183508, 8181289

2017-11-22 Thread Jonathan Gibbons
fix as well. This bug is about following to the HTML5 standards, so let's follow them in full and not to return to this once again. We have a good chance to provide documentation in clean HTML5 after the fix without any workarounds. --Semyon On 11/14/2017 09:16 AM, Jonathan Gibbons wrote:

Re: [OpenJDK 2D-Dev] [10] Review Request: 8182410, 8183508, 8181289

2017-11-14 Thread Jonathan Gibbons
10/25/17 6:31 PM, Semyon Sadetsky wrote: Hi Jonathan, On 10/24/2017 03:20 PM, Jonathan Gibbons wrote: Semyon, Although id is a global attribute and can be used to identify any node, some browsers do better navigation/scrolling when the id is in an tag.  We have seen poor autoscrolling beh

Re: [OpenJDK 2D-Dev] [10] Review Request: 8182410, 8183508, 8181289

2017-10-24 Thread Jonathan Gibbons
Semyon, Although id is a global attribute and can be used to identify any node, some browsers do better navigation/scrolling when the id is in an tag. We have seen poor autoscrolling behavior when the id is an a header tag, such that the header ends up obscured under the navigation bar at

Re: [OpenJDK 2D-Dev] [9] Review Request: 8180326 Update the tables in java.desktop to be HTML-5 friendly

2017-05-17 Thread Jonathan Gibbons
se .. -phil. On 05/17/2017 11:52 AM, Jonathan Gibbons wrote: Phil, The bottom line is that in the JDK docs, tables should not have a summary attribute and should have a caption. This comes down to accessibility requirements, where we are slowly raising the bar on our docs, to be in accordance

Re: [OpenJDK 2D-Dev] [9] Review Request: 8180326 Update the tables in java.desktop to be HTML-5 friendly

2017-05-17 Thread Jonathan Gibbons
;d want it to be visible when we were fine without. But being there and invisible may be pointless unless screen readers look for it even if invisible. But if its not doing any harm I guess we can leave it as proposed I still need to look at the rest of the changes. -phil. On 05/12/2017 05

Re: [OpenJDK 2D-Dev] [9] Review Request: 8180326 Update the tables in java.desktop to be HTML-5 friendly

2017-05-12 Thread Jonathan Gibbons
Sergey, FWIW, the invisible caption should be regarded as a temporary solution, until content authors can review/update the text of the caption and make it visible. The general guideline in this conversion work has been to avoid changing the visible text of the specification, and captions fa

Re: [OpenJDK 2D-Dev] [9] Review Request: 8180326 Update the tables in java.desktop to be HTML-5 friendly

2017-05-12 Thread Jonathan Gibbons
Phil, 1. javadoc now provides support for 3 named styles in the default stylesheet: borderless: no borders plain: simple 1px borders around tables and cells striped: reduced borders; rows have alternating white and light grey backgrounds 2. summary attributes are n

Re: [OpenJDK 2D-Dev] [9] Review Request: 8180326 Update the tables in java.desktop to be HTML-5 friendly

2017-05-12 Thread Jonathan Gibbons
On 05/12/2017 05:03 PM, Philip Race wrote: On 5/12/17, 4:58 PM, Jonathan Gibbons wrote: Phil, 1. javadoc now provides support for 3 named styles in the default stylesheet: borderless: no borders plain: simple 1px borders around tables and cells striped: reduced

Re: [OpenJDK 2D-Dev] [9] Review Request: 8179596 Update java.desktop to be HTML-5 friendly

2017-05-05 Thread Jonathan Gibbons
Sergey, This is not a comment on the content of the review; it is a minor comment on your interpretation of . is a block-level tag that is implicitly terminated by any of many block-level tags. For the full list, see the W3C definition of the p element in HTML 5: https://www.w3.org/TR/html