selection background drawn with the line spacing for multiline selection

[tobiasmelcher]

Line selection background is drawn without the line spacing value in the StyledText control. For a multiline selection, a zebra effect occurs which looks questionable from my point of view. In this change, the line spacing height is included when drawing the selection background.

Before with line spacing value 60%:

After with line spacing value 60%:

[github-actions]

Test Results

   486 files  ±0     486 suites  ±0   7m 52s ⏱️ -33s
 4 151 tests ±0   4 143 ✅ ±0   8 💤 ±0  0 ❌ ±0 
16 358 runs  ±0  16 266 ✅ ±0  92 💤 ±0  0 ❌ ±0 

Results for commit d97c17a. ± Comparison against base commit 2374dcf.

♻️ This comment has been updated with latest results.

[BeckerWdf]

contributes to #1286

[BeckerWdf]

if nobody objects I will merge this today EoB.

[BeckerWdf]

@sratz: Do you have more insights into that? Do you see problems / side effects with this change?

[sratz]

@sratz: Do you have more insights into that? Do you see problems / side effects with this change?

This change looks good.

By not extending the selection of the last line to the bottom, this avoids many other minor visual issues (e.g. if there is also an other highlight applied on the last line, such as a marked occurrence or a search result).

However: This change only fixes it for Win32. What about macOS and Linux? Can the same fix be applied there as well?

[tobiasmelcher]

the latest patch request contains implementations for windows, linux and mac.

Windows:

Linux:

Mac:

[sratz]

I missed to merge this change for the M3 deadline :(

Any chance this could go in for RC1?

The coding is rather safe, especially in the default 'line spacing 0' case, in which no behavior is changed at all.

[merks]

@sratz

It is fixing a bug and I trust in your good judgement, so I will defer the decision to you with a +1 from me (the PMC).

[sratz]

@sratz

It is fixing a bug and I trust in your good judgement, so I will defer the decision to you with a +1 from me (the PMC).

Thanks!