Integrated: 8185831: Pseudo selectors do not appear to work in Node.lookupAll()
Sai Pradeep Dandem
duke at openjdk.org
Wed Nov 1 22:49:15 UTC 2023
On Tue, 19 Sep 2023 12:17:16 GMT, Sai Pradeep Dandem <duke at openjdk.org> wrote:
> **Issue:**
> Using pseudo classes in programmatic query using Node.lookupAll() or Node.lookup() gives unexpected results.
>
> **Cause:**
> There is no check for checking the psuedo states matching in the applies() method of SimpleSelector.java. So checking for "applies()" alone is not sufficient in lookup() method.
>
> **Fix:**
> Included an extra check for the psuedo states to match.
This pull request has now been integrated.
Changeset: bb06b64c
Author: Dandem, Sai Pradeep <saipradeep.dandem at gmail.com>
Committer: Andy Goryachev <angorya at openjdk.org>
URL: https://git.openjdk.org/jfx/commit/bb06b64c9e1cda9390a84704a75b530c1d1d4577
Stats: 185 lines in 2 files changed: 170 ins; 0 del; 15 mod
8185831: Pseudo selectors do not appear to work in Node.lookupAll()
Reviewed-by: kcr, angorya, jhendrikx
-------------
PR: https://git.openjdk.org/jfx/pull/1245
More information about the openjfx-dev
mailing list