RFR: 8017266: Background is painted taller than needed for styled text.
Prasanta Sadhukhan
psadhukhan at openjdk.org
Mon Oct 27 09:55:17 UTC 2025
On Sat, 25 Oct 2025 07:19:09 GMT, Alexander Zuev <kizune 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..
>
> Ok, i have experimented with the partial selection and now convinced that with the fix it looks better.
@azuev-java I have automated the test..Can you please review and reapprove?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/27947#issuecomment-3450403340
More information about the client-libs-dev
mailing list