RFR: JDK-8292276 : Missing color names in CSS [v23]
ScientificWare
duke at openjdk.org
Wed Sep 28 22:43:45 UTC 2022
On Wed, 28 Sep 2022 17:13:42 GMT, ScientificWare <duke at openjdk.org> wrote:
>> This is referenced in Java Bug Database as
>> - [JDK-8292276 : Missing color names in CSS](https://bugs.java.com/bugdatabase/view_bug.do?bug_id=8292276)
>>
>> This is tracked in JBS as
>> - [JDK-8292276 : Missing color names in CSS](https://bugs.openjdk.java.net/browse/JDK-8292276)
>>
>> Adds missing color names, defined by CSS Level 4, in CSS.java :
>> CSS Color Module Level 4
>> W3C Candidate Recommendation Snapshot, 5 July 2022
>> [7.1 Named Colors](https://www.w3.org/TR/css-color-4/#named-color)
>>
>> Designed from : [ScientificWare JDK-8292276 : Missing color names in CSS](https://github.com/scientificware/jdk/issues/12)
>
> ScientificWare has updated the pull request incrementally with one additional commit since the last revision:
>
> Moves jtreg tags to the class declaration
>
> Comment with tags isn't collapsed when viewed in an IDE if it's placed before the class declaration.
> Are you talking about https://github.com/openjdk/jdk/pull/9825#issuecomment-1216011827?
@SWinxy Yes, only because the result is publicly exposed : `stringToColor` is also used by `javax.swing.text.html.StyleSheet stringToColor` method to publicly return the `Color` Object. It seems unique, I didn't see any other usages.
> also reminder that .contains...(Object) exists
I saw these methods but in our case we need a result anyway. Using `containsKey(Object)` implies a second request to get the value ?
-------------
PR: https://git.openjdk.org/jdk/pull/9825
More information about the client-libs-dev
mailing list