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