On Fri, 14 Jun 2024 19:01:56 GMT, Chen Liang wrote:
>> Nizar Benalla has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Simplify method
>
> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclint/HtmlTag.java line
> 670:
>
>> 668:
> Can I please get a review for this change, that aims to add support for
> Global HTML tags.
> Here is the
> [link](https://cr.openjdk.org/~nbenalla/javadocGlobalPR/pkg1/package-summary.html)
> to the generated docs.
> Thanks in advance.
Nizar Benalla has updated the pull request
On Fri, 14 Jun 2024 19:00:41 GMT, Nizar Benalla wrote:
>> Can I please get a review for this change, that aims to add support for
>> Global HTML tags.
>> Here is the
>> [link](https://cr.openjdk.org/~nbenalla/javadocGlobalPR/pkg1/package-summary.html)
>> to the generated docs.
>> Thanks in
> Can I please get a review for this change, that aims to add support for
> Global HTML tags.
> Here is the
> [link](https://cr.openjdk.org/~nbenalla/javadocGlobalPR/pkg1/package-summary.html)
> to the generated docs.
> Thanks in advance.
Nizar Benalla has updated the pull request
Please review a change to slightly adjust the proportion between the side bar
and the main content area in API documentation pages, making the side bar a bit
slimmer and leaving more space for the main content area. The ratio between
side bar and main content used to be 1 : 2.6, it is now 1 :
On Fri, 14 Jun 2024 12:15:45 GMT, Nizar Benalla wrote:
>> Can I please get a review for this change, that aims to add support for
>> Global HTML tags.
>> Here is the
>> [link](https://cr.openjdk.org/~nbenalla/javadocGlobalPR/pkg1/package-summary.html)
>> to the generated docs.
>> Thanks in
> The scrollable element `` with non-interactive content is not tabbable.
> Grid columns in the javadoc stylesheet has overflow: auto, which is failing
> Accessibility checks.
> https://bugs.openjdk.org/browse/JDK-8325690
psoujany has updated the pull request with a new target base due to a
On Tue, 11 Jun 2024 13:03:47 GMT, Pavel Rappo wrote:
>> Nizar Benalla has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains ten additional
>> commits
On Tue, 11 Jun 2024 22:12:42 GMT, Jonathan Gibbons wrote:
>> Nizar Benalla has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains ten additional
>>
On Fri, 14 Jun 2024 12:12:51 GMT, Nizar Benalla wrote:
>> Can I please get a review for this change, that aims to add support for
>> Global HTML tags.
>> Here is the
>> [link](https://cr.openjdk.org/~nbenalla/javadocGlobalPR/pkg1/package-summary.html)
>> to the generated docs.
>> Thanks in
On Wed, 12 Jun 2024 09:03:38 GMT, Hannes Wallnöfer wrote:
>> Wouldn't the easiest solution be to add a boolean `global`/`isGlobal` field
>> and getter to `Attr`?
>>
>> That would give use some more opportunities to simplify the code: We could
>> get rid of the `GLOBAL_ATTRS` map here *and*
On Wed, 12 Jun 2024 09:02:20 GMT, Hannes Wallnöfer wrote:
>> I suggest one of
>>
>> 1. Add a `boolean global;` member to the enum, and provide a second
>> constructor that allows that boolean member to be set true. For example,
>>
>> WIDTH,
>> STYLE(true),
>>
>>
>> 2. Make the `Set` be
On Tue, 11 Jun 2024 21:58:46 GMT, Jonathan Gibbons wrote:
>>> If we are ordering the entries, we can use comparable to check that an attr
>>> is greater than the start of the global attr, something like
>>>
>>> ```java
>>> private static boolean isGlobalAttr(Attr value) {
>>> return
On Tue, 11 Jun 2024 13:41:58 GMT, Pavel Rappo wrote:
>> If we are ordering the entries, we can use comparable to check that an attr
>> is greater than the start of the global attr, something like
>>
>> private static boolean isGlobalAttr(Attr value) {
>> return
On Tue, 11 Jun 2024 12:40:23 GMT, Nizar Benalla wrote:
>> Either what Chen has suggested or re-sort the complete list alphabetically
>> as it was prior to this change, well, almost.
>
>> Either what Chen has suggested or re-sort the complete list alphabetically
>> as it was prior to this
On Wed, 12 Jun 2024 09:26:25 GMT, Pavel Rappo wrote:
>> I admit to being "lazy" when I wrote `startsWith("on")`. In general, I don't
>> think folk should be writing many if any event handlers in doc comments, and
>> it was easier to give forward compatibility with a weak test for `"on"`.
>>
On Tue, 11 Jun 2024 23:49:15 GMT, Jonathan Gibbons wrote:
>> As @jonathan-gibbons likes to point out, javadoc is not an HTML validation
>> tool. So I think, it's okay to leave the code simple. Maybe this would be
>> even simpler?
>>
>> data-[a-z][-a-z0-9]*
>
> I admit to being "lazy" when
On Tue, 11 Jun 2024 15:09:03 GMT, Pavel Rappo wrote:
>> I think like being slightly restrictive and safe.
>
> As @jonathan-gibbons likes to point out, javadoc is not an HTML validation
> tool. So I think, it's okay to leave the code simple. Maybe this would be
> even simpler?
>
>
On Tue, 11 Jun 2024 14:42:48 GMT, Nizar Benalla wrote:
>> src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclint/Checker.java line
>> 695:
>>
>>> 693: // custom "data-*" attributes are also accepted
>>> 694: var attrName=name.toString();
>>> 695: if
On Tue, 11 Jun 2024 13:24:48 GMT, Nizar Benalla wrote:
>> Can I please get a review for this change, that aims to add support for
>> Global HTML tags.
>> Here is the
>> [link](https://cr.openjdk.org/~nbenalla/javadocGlobalPR/pkg1/package-summary.html)
>> to the generated docs.
>> Thanks in
On Tue, 11 Jun 2024 13:26:28 GMT, Pavel Rappo wrote:
>> Nizar Benalla has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains ten additional
>> commits
On Fri, 14 Jun 2024 12:12:51 GMT, Nizar Benalla wrote:
>> Can I please get a review for this change, that aims to add support for
>> Global HTML tags.
>> Here is the
>> [link](https://cr.openjdk.org/~nbenalla/javadocGlobalPR/pkg1/package-summary.html)
>> to the generated docs.
>> Thanks in
> Can I please get a review for this change, that aims to add support for
> Global HTML tags.
> Here is the
> [link](https://cr.openjdk.org/~nbenalla/javadocGlobalPR/pkg1/package-summary.html)
> to the generated docs.
> Thanks in advance.
Nizar Benalla has updated the pull request with a new
On Fri, 14 Jun 2024 11:31:30 GMT, psoujany wrote:
>> The scrollable element `` with non-interactive content is not tabbable.
>> Grid columns in the javadoc stylesheet has overflow: auto, which is failing
>> Accessibility checks.
>> https://bugs.openjdk.org/browse/JDK-8325690
>
> psoujany has
> The scrollable element `` with non-interactive content is not tabbable.
> Grid columns in the javadoc stylesheet has overflow: auto, which is failing
> Accessibility checks.
> https://bugs.openjdk.org/browse/JDK-8325690
psoujany has updated the pull request incrementally with one additional
> The scrollable element `` with non-interactive content is not tabbable.
> Grid columns in the javadoc stylesheet has overflow: auto, which is failing
> Accessibility checks.
> https://bugs.openjdk.org/browse/JDK-8325690
psoujany has updated the pull request incrementally with one additional
> The scrollable element `` with non-interactive content is not tabbable.
> Grid columns in the javadoc stylesheet has overflow: auto, which is failing
> Accessibility checks.
> https://bugs.openjdk.org/browse/JDK-8325690
psoujany has updated the pull request incrementally with one additional
> The scrollable element `` with non-interactive content is not tabbable.
> Grid columns in the javadoc stylesheet has overflow: auto, which is failing
> Accessibility checks.
> https://bugs.openjdk.org/browse/JDK-8325690
psoujany has updated the pull request incrementally with one additional
28 matches
Mail list logo