Alpha grainier and warmer than {alacr,k}itty #249

Closed
opened 10 months ago by JorwLNKwpH · 4 comments

Not sure if this can be fixed, as alacritty and kitty calls OpenGL to make this much better-looking, but the alpha compositing in foot is grainier and warmer compared to the other two terminals. The text looks "fuzzier" and this decreases readability as a result. Attached a picture to demonstrate, foot on the left and kitty on the right with alpha at 0.75 for both terminals. Note that kitty and alacritty basically look the same for similar alpha levels, so I guess they are doing something similar.

Not sure if this can be fixed, as alacritty and kitty calls OpenGL to make this much better-looking, but the alpha compositing in foot is grainier and warmer compared to the other two terminals. The text looks "fuzzier" and this decreases readability as a result. Attached a picture to demonstrate, foot on the left and kitty on the right with alpha at 0.75 for both terminals. Note that kitty and alacritty basically look the same for similar alpha levels, so I guess they are doing something similar.
Owner

Interresting. Might be an issue with pre-alpha-multiplied color components, or maybe gamma related...

Interresting. Might be an issue with pre-alpha-multiplied color components, or maybe gamma related...
dnkl closed this issue 10 months ago
Owner

@JorwLNKwpH it was a rounding error when calculating the pre-multiplied color channels. I intended to create a PR for you to test, but accidentaly pushed to master...

But it looked right to me. Just re-open this issue if the problem persists.

@JorwLNKwpH it was a rounding error when calculating the pre-multiplied color channels. I intended to create a PR for you to test, but accidentaly pushed to master... But it looked right to me. Just re-open this issue if the problem persists.
Poster

Just tried out master and it works great!

Glad to see this fixed on your other projects as well.

Just tried out master and it works great! Glad to see this fixed on your other projects as well.
Owner

Just tried out master and it works great!

Great, thanks!

Glad to see this fixed on your other projects as well.

Seems kind of dumb not to... :)

> Just tried out master and it works great! Great, thanks! > Glad to see this fixed on your other projects as well. Seems kind of dumb not to... :)
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.