RFR: 8017266: Background is painted taller than needed for styled text.
Prasanta Sadhukhan
psadhukhan at openjdk.org
Fri Oct 24 03:59: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..
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
More information about the client-libs-dev
mailing list