Integrated: 8340580: Characters in Document diagrams not in the middle

Alexey Ivanov aivanov at openjdk.org
Wed Dec 4 19:00:58 UTC 2024


On Wed, 4 Dec 2024 15:40:31 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

> This is a continuation to #20376, it moves the text in `Document-{insert,remove}.svg` down by 0.5 pixel.
> 
> This change makes the text centred in the boxes.
> 
> Here are quotes from https://github.com/openjdk/jdk/pull/20376#issuecomment-2353728939 and https://github.com/openjdk/jdk/pull/20376#issuecomment-2353743533:
> 
>> <q cite="https://github.com/openjdk/jdk/pull/20376#issuecomment-2353728939">At 200% scale and Arial font, there are 6 pixels on top of ‘T’ and ‘h’, but there are 8 pixels on the bottom of ‘q’.</q>
>> 
>> <q cite="https://github.com/openjdk/jdk/pull/20376#issuecomment-2353743533">Moving the text position by 0.5 seems to help, and the values become equal: 7 on either side.</q>

This pull request has now been integrated.

Changeset: faa109e8
Author:    Alexey Ivanov <aivanov at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/faa109e8d40dbe8c6beb26bd8f9e5f6f63e4cab1
Stats:     39 lines in 2 files changed: 0 ins; 0 del; 39 mod

8340580: Characters in Document diagrams not in the middle

Move text in Document-{insert,remove}.svg by 0.5 of pixel

Reviewed-by: azvegint, kizune

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

PR: https://git.openjdk.org/jdk/pull/22549


More information about the client-libs-dev mailing list