Re: RFR: 8185831: Pseudo selectors do not appear to work in Node.lookupAll() [v11]

2023-10-28 Thread Kevin Rushforth
On Fri, 27 Oct 2023 16:39:33 GMT, Kevin Rushforth wrote: >> Sai Pradeep Dandem has updated the pull request incrementally with one >> additional commit since the last revision: >> >> 8185831: Changes to doc as per review comments > > modules/javafx.graphics/src/main/java/javafx/scene/Node.jav

Re: RFR: 8185831: Pseudo selectors do not appear to work in Node.lookupAll() [v11]

2023-10-27 Thread Kevin Rushforth
On Thu, 26 Oct 2023 23:27:08 GMT, Sai Pradeep Dandem 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

Re: RFR: 8185831: Pseudo selectors do not appear to work in Node.lookupAll() [v11]

2023-10-26 Thread Sai Pradeep Dandem
> **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 > s