RFR: 8017266: Background is painted taller than needed for styled text.

Alexander Zuev kizune at openjdk.org
Sat Oct 25 07:22:00 UTC 2025


On Thu, 23 Oct 2025 03:19:06 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> 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..

Marked as reviewed by kizune (Reviewer).

Ok, i have experimented with the partial selection and now convinced that with the fix it looks better.

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

PR Review: https://git.openjdk.org/jdk/pull/27947#pullrequestreview-3379797965
PR Comment: https://git.openjdk.org/jdk/pull/27947#issuecomment-3446049893


More information about the client-libs-dev mailing list