On Wed, 15 Jul 2026 15:11:52 GMT, Florian Kirmaier <[email protected]> wrote:
>> ### Fix >> The PR replaces applyCss() with reapplyCSS in the nodeOrientation code of >> Scene. >> applyCss() reapplied styles eagerly - and didn't rebuild the "style maps" >> resulting in wrong css. >> reapplyCSS() rematches correctly - and also defers it to the next pulse. >> >> I've added a unit test to: Node_effectiveOrientation_Css_Test. >> >> ### Test improvements >> Because the whole test class was disabled, I've also investigated which >> tests are working - and reenabled the working tests. >> This PR also fixes 2 of the previously failing tests in >> Node_effectiveOrientation_Css_Test - which are now enabled. >> >> Which are the following tests: >> >> Node_effectiveOrientation_Css_Test.test_dir_pseudoClass_functions_on_scene_effective_orientation_not_node >> Node_effectiveOrientation_Css_Test.test_SimpleSelector_dir_pseudoClass_with_scene_effective_orientation_rtl >> >> >> >> >> >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > I could imagine, that some parts of the code-base rely on the forced CSS > computation. > But otherwise i have a good feeling that its correct. FYI @FlorianKirmaier this looks good to me. But you have still outstanding errors/warnings, see your PR body. You probably should also merge in master again, while here. ------------- PR Comment: https://git.openjdk.org/jfx/pull/2213#issuecomment-5636570575
