RFR: 8336097: UserAgent Styles using lookups are promoted to Author level if look-up is defined in Author stylesheet [v2]
John Hendrikx
jhendrikx at openjdk.org
Mon Jul 22 14:16:37 UTC 2024
On Mon, 22 Jul 2024 10:01:44 GMT, John Hendrikx <jhendrikx at openjdk.org> wrote:
>> Another question:
>> do we want to go through all the cases of Scene/SubScene/Parent/Region user agent style sheet, for completeness sake, or would that be a different test file?
>
>> Another question: do we want to go through all the cases of Scene/SubScene/Parent/Region user agent style sheet, for completeness sake, or would that be a different test file?
>
> In my opinion, this does seem a bit of out of scope for this change. It would not add any new coverage to the code that was changed. Also the origin of the user agent stylesheet is not a factor in the resolution of lookups. The user agent stylesheets, their replacement (via Scene/Subscene) or addition (via Region) should (already?) be tested elsewhere.
I added a few cases where user agent is `null`. I've not added all combinations as that would lead to 20-30 extra cases that add little of value. If I already verified that INLINE will override the lower priority sources individually, then it's safe to say that all possible other combinations of those lower priority sources will also get overridden.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1503#discussion_r1686628470
More information about the openjfx-dev
mailing list