RFR: 8273986: JEditorPane HTML Demo - Accessibility issues [v2]

Sergey Bylokhov serb at openjdk.org
Mon Feb 27 21:00:43 UTC 2023


On Mon, 27 Feb 2023 04:51:50 GMT, Abhishek Kumar <abhiscxk at openjdk.org> wrote:

>> src/java.desktop/share/classes/javax/swing/text/html/HTMLEditorKit.java line 690:
>> 
>>> 688:         if (accessibleContext == null
>>> 689:             || doc != theEditor.getDocument()) {
>>> 690:             AccessibleHTML a = new AccessibleHTML(theEditor);
>> 
>> The "AccessibleHTML" internally registers the listener on the editor to update the state/doc. Why that notification does not work and it is necessary to recreate the "AccessibleHTML" here?
>
> The "AccessibleHTML" does internally register the listener on the editor to update the state/doc.
> 
> But the `rootHTMLAccessibleContext` is not updated/created for the new root element on doc change and when  `getAccessibleChildrenCount` is called it returns the child count 0, because the API is get called on the old object.

if the "rootHTMLAccessibleContext" depends on the doc then I think it should be updated in that listener at the same moment the doc is replaced. especially taking into account that the "rootElementInfo" is updated already.

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

PR: https://git.openjdk.org/jdk/pull/12707



More information about the client-libs-dev mailing list