On Mon, 27 Oct 2025 12:23:35 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..
>
> Prasanta Sadhukhan has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Use width, height

Marked as reviewed by kizune (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/27947#pullrequestreview-3384454334

Reply via email to