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

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

Reply via email to