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..

I am not sure I understand. 
I tried selection in the original test. It seems same to me..
WIth fix
<img width="274" height="157" alt="image" 
src="https://github.com/user-attachments/assets/fea2d1f1-08f0-4d7a-a32b-8de3fac32ee6";
 />

Before fix
<img width="289" height="129" alt="image" 
src="https://github.com/user-attachments/assets/1919aa19-7a05-4ba5-a698-57a8452cca8f";
 />

Part selection
WIth fix
<img width="228" height="94" alt="image" 
src="https://github.com/user-attachments/assets/310c0400-6908-4212-a7d3-b83b2685119d";
 />

Before fix
<img width="251" height="95" alt="image" 
src="https://github.com/user-attachments/assets/397c0874-78c3-455c-a7b0-9ae5bf963bb6";
 />

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

PR Comment: https://git.openjdk.org/jdk/pull/27947#issuecomment-3440852177

Reply via email to