Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Increase Powerline glyph overlap #1419

Merged
merged 2 commits into from
Nov 16, 2023
Merged

Conversation

Finii
Copy link
Collaborator

@Finii Finii commented Nov 16, 2023

Description

[why]
We still fight with the faint lines on the big side of the powerline
glyphs. They come from the LCD antialiasing mode that has problems with
the borders.

Other fonts use far more overlap. We use only a modest amount of overlap
(2% of the width).

[how]
As the other fonts do, increase the overlap (to 7% now). We could not do
that before, because we had no full-height 'landing platforms' on the
outsides and so the triangle would look cut off at not 100% height.

Now with the landing platforms we can increase the overlap and still
have nice looking triangles that visibly reach the top and bottom of the
line.

Implement this only for the filled triangles to see how it goes.

Requirements / Checklist

What does this Pull Request (PR) do?

How should this be manually tested?

Any background context you can provide?

What are the relevant tickets (if any)?

Fixes #1245 (maybe, at least partially)

Screenshots (if appropriate or helpful)

image
left: before, right: after

[why]
The waveform glyphs E0C8 and E0CA have an incredable amount of points
and rendering is rather expensive.

It turned out that it has a hairline ourline, that will be invisible
anyhow.

[how]
Remove the Hairline, reducing the number of points to 1/3.

[note]
See #1245

Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
[why]
We still fight with the faint lines on the big side of the powerline
glyphs. They come from the LCD antialiasing mode that has problems with
the borders.

Other fonts use far more overlap. We use only a modest amount of overlap
(2% of the width).

[how]
As the other fonts do, increase the overlap (to 7% now). We could not do
that before, because we had no full-hight 'landing platforms' on the
outsides and so the triangle would look cut off at not 100% hight.

Now with the landing platforms we can increase the overlap and still
have nice looking triangles that visibly reach the top and bottom of the
line.

Implement this only for the filled triangles to see how it goes.

Related: #1245

Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
@Finii Finii added this to the v3.1.0 milestone Nov 16, 2023
@Finii Finii merged commit 307e2ad into master Nov 16, 2023
5 checks passed
@Finii Finii deleted the bugfix/powerline-glyph-overlap branch November 16, 2023 17:49
@Finii
Copy link
Collaborator Author

Finii commented Nov 16, 2023

I wrote 7% in the commit message and in the comment above, but in reality it's only 6% 😬

LNKLEO pushed a commit to LNKLEO/Nerd that referenced this pull request Nov 24, 2023
…ph-overlap

Increase Powerline glyph overlap
@Finii Finii mentioned this pull request Jan 22, 2024
2 tasks
Finii added a commit that referenced this pull request Oct 28, 2024
[why]
There are still the annoying vertical colored lines sometimes that turn
up due to subpixel rendering.

[how]
Add "landing platforms" to the big triangular glyphs (E0B8, E0BA, E0BC,
E0BE) and flames (E0C0, E0C2).
The landing platform is approx 7% wide based on one-cell width, assuming
the glyphs are all rendered 2 cells wide.

Increase the overlap width for patching to 5%.

See also:
Merge request #1551
Merge request #1419
Commit 5e28586
Commit a8b9e1d

Fixes: #1629 (well, not the top problem which is unfixable)

Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
Finii added a commit that referenced this pull request Oct 28, 2024
[why]
There are still the annoying vertical colored lines sometimes that turn
up due to subpixel rendering.

[how]
Add "landing platforms" to the big triangular glyphs (E0B8, E0BA, E0BC,
E0BE) and flames (E0C0, E0C2).
The landing platform is approx 7% wide (based on one-cell width, assuming
the glyphs are all rendered 2 cells wide for the "xy2" ones).

Increase the overlap width for patching to 5%.

See also:
Merge request #1551
Merge request #1419
Commit 5e28586
Commit a8b9e1d

Fixes: #1629 (well, not the top problem which is unfixable)

Signed-off-by: Fini Jastrow <ulf.fini.jastrow@desy.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

nf-hack behaves differently than the original hack-font
1 participant