On Fri, 20 Oct 2023 03:26:51 GMT, Sai Pradeep Dandem wrote:
>> modules/javafx.graphics/src/test/java/test/javafx/scene/Node_lookup_Test.java
>> line 202:
>>
>>> 200: */
>>> 201: @Test
>>> 202: public void lookupPseudoTest3() {
>>
>> Consider naming the test methods according to wh
On Fri, 20 Oct 2023 02:50:28 GMT, Michael Strauß wrote:
>> Sai Pradeep Dandem has updated the pull request incrementally with two
>> additional commits since the last revision:
>>
>> - 8185831: Minor change to remove extra line
>> - 8185831: Added testcase to verify when the selector ignores
On Fri, 20 Oct 2023 00:26: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
On Fri, 20 Oct 2023 00:22:27 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
> **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
