On Sat, 25 Oct 2025 07:19:09 GMT, Alexander Zuev <[email protected]> wrote:
>> GlyphView.paint() draws background bounding the passed Shape, while the span >> reserved for the superscripted text is taller than the height of the glyphs >> so it is better to use the painter.getHeight() instead of alloc.height to >> fill the actual glyphs boundary >> >> Before fix >> <img width="103" height="93" alt="image" >> src="https://github.com/user-attachments/assets/e673d214-13c7-4a42-8f55-cfd654e5a978"; >> /> >> >> >> With fix >> <img width="110" height="86" alt="image" >> src="https://github.com/user-attachments/assets/c13377a6-b4ac-406c-a153-a55c0b9a917b"; >> /> >> >> No regressions is observed in CI..A manual verification test is provided.. > > Ok, i have experimented with the partial selection and now convinced that > with the fix it looks better. @azuev-java I have automated the test..Can you please review and reapprove? ------------- PR Comment: https://git.openjdk.org/jdk/pull/27947#issuecomment-3450403340
