RFR: JDK-8266779: Use <wbr> instead of ZERO_WIDTH_SPACE [v2]

[Hannes Wallnöfer]

On Mon, 10 May 2021 14:01:46 GMT, Hannes Wallnöfer <hannesw at openjdk.org> wrote:

>> This replaces usages of the zero-width space character (ZWSP, entity `​`) with the HTML5 `<wbr>` element. `<wbr>` acts as a word break opportunity without adding characters to the text content like ZWSP does. It is supported in all modern browsers, the only browser I  know of which doesn't support is are old versions of Internet Explorer. 
>> 
>> https://developer.mozilla.org/en-US/docs/Web/HTML/Element/wbr
>> https://caniuse.com/wbr-element
>> 
>> I have tested the output (both layout and copy-paste behaviour) on Firefox, Safari, and Chrome on Mac OS X.
>
> Hannes Wallnöfer has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision:
> 
>  - Merge branch 'master' into JDK-8266779
>  - JDK-8266779: Use <wbr> instead of ZERO_WIDTH_SPACE

I’d rather not. I don’t think it’s a huge improvement, there’s only 4 usages of this tag, and … it’s late and I’d like to close the issue :) 

I think there are some elements that are much more often used that don’t have a factory method. Maybe another cleanup task?

Hannes

> Am 10.05.2021 um 21:16 schrieb Pavel Rappo ***@***.***>:
> 
> 
> Come to think of it, consider obtaining an instance of a <wbr> tag indirectly through a static factory method HtmlTree.WBR() rather than through a direct allocation new HtmlTree(TagName.WBR).
> 
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or unsubscribe.
>

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

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