On Fri, 2 Apr 2021 19:47:30 GMT, mstr2 <github.com+43553916+ms...@openjdk.org> wrote:
> This PR fixes incorrect parsing of mnemonic symbols in the following cases: > 1. an escaped double underscore incorrectly shows up as a double underscore > in displayed text > 2. an extended mnemonic incorrectly removes a part of adjacent text > > As a side effect, this PR also fixes another undocumented issue for multiline > text where text metrics calculations incorrectly operate on text for which > mnemonic symbols have not been processed. > > This can lead to a glitch where the mnemonic underscore makes a text just > long enough to wrap to a new line, but when the text is rendered, the second > line shows up entirely empty. The reason for this is that for rendering, the > text is laid out _after_ mnemonic symbols have been removed. > > The solution is to _always_ use processed text ("_clean text_") when laying > out text. This pull request has now been integrated. Changeset: affb1084 Author: mstr2 <43553916+ms...@users.noreply.github.com> Committer: Kevin Rushforth <k...@openjdk.org> URL: https://git.openjdk.java.net/jfx/commit/affb10845bdf9a20f7e44c5b6a62883a892e4076 Stats: 251 lines in 3 files changed: 120 ins; 51 del; 80 mod 8137323: Incorrect parsing of mnemonic in controls text Reviewed-by: aghaisas, kcr ------------- PR: https://git.openjdk.java.net/jfx/pull/453