RFR: 8355012: JavaFX modena.css -fx-highlight-text-fill bug [v2]
Andy Goryachev
angorya at openjdk.org
Mon May 5 14:48:53 UTC 2025
On Mon, 5 May 2025 14:27:17 GMT, Ziad El Midaoui <zelmidaoui at openjdk.org> wrote:
>> The issue was happening because `-fx-highlight-text-fill` was set to white , so when the background color is bright (>60% brightness ) the text is not visible , the solution is to change the value to be dynamically set depending on the level of brightness of the background this is done by `-fx-focused-text-base-color` .
>
> Ziad El Midaoui has updated the pull request incrementally with one additional commit since the last revision:
>
> Created ModenaTest
modules/javafx.controls/src/test/java/test/com/sun/javafx/scene/control/skin/modena/ModenaTest.java line 23:
> 21: import static org.junit.jupiter.api.Assertions.fail;
> 22:
> 23: public class ModenaTest {
perhaps add a javadoc describing what this file is supposed to test?
modules/javafx.controls/src/test/java/test/com/sun/javafx/scene/control/skin/modena/ModenaTest.java line 33:
> 31: }
> 32:
> 33: private TextInputControl textInput;
Since this test tests `modena.css` aspects of the `TextInputControl` class hierarchy, should this test be renamed then (`TextInputControlModena` or something like that)?
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1801#discussion_r2073598355
PR Review Comment: https://git.openjdk.org/jfx/pull/1801#discussion_r2073597590
More information about the openjfx-dev
mailing list