On Wed, 8 Jun 2022 18:10:21 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:

>> I'll remove the button then as it's not necessary for the test. I mainly 
>> added it to demonstrate that HTML contents are rendering correctly. I'll 
>> make this change and update the doc for Parser to more explicitly explain 
>> what happens when its editable vs non-editable.
>
> It looks good overall.
> 
> It started with `<script>` tag; the text in javadoc refers to other unknown 
> or unsupported tags. Shall the test be expanded with an unknown tag?
> 
> Displaying two instances of `JEditorPane` where one's editable and the other 
> isn't could be too confusing. Only editable is enough, I guess.

Sure, I added a foo tag with text inside. Behavior still displays as expected 
and stated in the doc.

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

PR: https://git.openjdk.java.net/jdk/pull/7446

Reply via email to