RFR: JDK-8292276 : Add named colors from CSS Color Module Level 4 [v31]
ScientificWare
duke at openjdk.org
Mon Aug 7 22:26:40 UTC 2023
On Mon, 7 Aug 2023 18:00:38 GMT, ScientificWare <duke at openjdk.org> wrote:
>> This is referenced in Java Bug Database as
>> - [JDK-8292276 : Add named colors from CSS Color Module Level 4](https://bugs.java.com/bugdatabase/view_bug.do?bug_id=8292276)
>>
>> This is tracked in JBS as
>> - [JDK-8292276 : Add named colors from CSS Color Module Level 4](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 : Add named colors from CSS Color Module Level 4](https://github.com/scientificware/jdk/issues/12)
>
> ScientificWare has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 13 commits:
>
> - configure : linux file permission moves from 644=rw-r--r-- to 755=rwxr-xr-x.
> - Merge master
> - Merge master
> - Merge master
>
> # Conflicts:
> # src/java.desktop/share/classes/javax/swing/text/html/CSS.java
> # src/java.desktop/share/classes/javax/swing/text/html/StyleSheet.java
> - CSS.java
> - Updates copyright date to 2023.
> - Adds the missing color names defined by :
> 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)
> - Adds relative imports.
> - Replaces, if ... then ... else statements with a Map called "colorNamed".
>
> StyleSheet.java ;
> - Updates copyright date to 2023.
> - Updates stringToColor specifications to integrate a reference to CSS Color Module 4 for color names and hexadecimal notation.
>
> MissingColorNames.java :
> - Création d'un test pour les nouveaux noms de couleurs ajoutés.
> - MissingColorNames.java :
> 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.
> - JDK8292276MissingColorNamesInCSS.java -> MissingColorNames.java :
> - Changement de nom.
> - Déplacement vers un autre dossier.
> - Simplification d'une description.
> - Ajout d'espaces manquants.
> - Correction de l'alignement d'un délimiteur de commentaire.
>
> CSS.java :
> - Changement de nom pour la variable colorNamed.
> - Ajout du modificateur final à la variable ColorNames.
> - Uniformisation de l'utilisation de la variable strlc.
> - renvoi direct de la couleur trouvée si elle existe.
> - Problème de configuration de KWrite.
> Qui rajoutait un retour ligne à la fin de chaque fichier.
> - CSS.java
> - Corrige une grossière erreur de code :
> Modifie l'emplacement de la mise en minuscules pour le placer juste après les cas pour lequel cela n'a pas d'importance et surtout après avoir envisager le cas null.
> Sinon ce pouvait engendrer un NPE.
> - Modifie la déclaration des couleurs opaques. En choisissant la déclaration avec trois entiers.
>
> JDS8292276MissingColorNamesInCSS.java :
> - Ajout du cas null.
> - Ajout du test sur le
> - Erreur de nom.
> Un caractère de retour à la ligne s'est immiscer dans le nom du dossier.
> - ... and 3 more: https://git.openjdk.org/jdk/compare/1da82a34...63d6...
https://github.com/openjdk/jdk/pull/9825#discussion_r1120290326
> @aivanov-jdk The hex color notation and rgb() and rgba() functions should link to their specifications:
> https://www.w3.org/TR/css-color-4/#hex-notation
> https://www.w3.org/TR/css-color-4/#rgb-functions
> correspondingly.
My proposition :
* Note : This will only convert string colors using names listed in
* <a href="https://www.w3.org/TR/css-color-4/#named-colors">CSS-COLOR-4
* Named Colors</a>
* or RGB 3, 4, 6, 8 digit hexadecimal notations as specified by the
* <a href="https://www.w3.org/TR/css-color-4/#hex-notation">CSS-COLOR-4
* The RGB Hexadecimal Notations</a>
* and rgb() or rgba() function definitions described in
* <a href="https://www.w3.org/TR/css-color-4/#rgb-functions">CSS-COLOR-4
* The RGB functions</a>.
* Otherwise, it will return null.
* This method is case-insensitive.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/9825#issuecomment-1668652267
More information about the client-libs-dev
mailing list