RFR: 8204568: Relative CSS-Attributes don't work all time [v4]

Ambarish Rapte arapte at openjdk.java.net
Sat Mar 6 16:10:18 UTC 2021


> Issue is that the size of properties that are relatively(`em`) sized is not computed correctly when the reference `-fx-font-size` is also specified relatively and is nested.
> 
> Fix is a slight variation of an earlier suggestion in [the PR](https://github.com/javafxports/openjdk-jfx/pull/94).
> 
> Fix is very specific to this scenario and did not show any side effect nor any test failure.
> 
> There are 12 new unit tests added along with fix:
> - Two tests fail before and pass after this fix. These test verify the reported failing scenario.
> sameRelativeFontSizeNestedParentTest
> relativeFontSizeDeepNestedParentControlTest
> - Two other tests fail both before and after this fix. They are not related to the fix. These two tests are ignored. I shall file new JBS issues to track these cases and update PR with the IDs added to @Ignore.
> propertySizesRelativeToFontSizeOfControlTest
> propertySizesRelativeToFontSizeOfParentTest
> - Other 8 tests are sanity tests which pass both before and after this fix.

Ambarish Rapte has updated the pull request incrementally with one additional commit since the last revision:

  rephrasing comment statements and review corrections

-------------

Changes:
  - all: https://git.openjdk.java.net/jfx/pull/397/files
  - new: https://git.openjdk.java.net/jfx/pull/397/files/1b17d26a..ba8b4ba0

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jfx&pr=397&range=03
 - incr: https://webrevs.openjdk.java.net/?repo=jfx&pr=397&range=02-03

  Stats: 27 lines in 2 files changed: 4 ins; 1 del; 22 mod
  Patch: https://git.openjdk.java.net/jfx/pull/397.diff
  Fetch: git fetch https://git.openjdk.java.net/jfx pull/397/head:pull/397

PR: https://git.openjdk.java.net/jfx/pull/397


More information about the openjfx-dev mailing list