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

Fix rendering issue on WASM with AsymmetricRoundedRectangle #1292

Merged
merged 1 commit into from
Jul 4, 2024

Conversation

chriadam
Copy link
Contributor

@chriadam chriadam commented Jul 2, 2024

In WASM, AsymmetricRoundedRectangle (used in TabBar for example on the Levels page) sometimes shows a line on the edge of the clip item (between the rounded background and the non-rounded background section). The issue doesn't reproduce on desktop or on device, but it can be resolved by overlapping the rounded background section on top of the normal background section by 1 pixel (can do this now, since colours are opaque).

@chriadam chriadam requested review from blammit and DanielMcInnes July 2, 2024 08:03
@DanielMcInnes
Copy link
Contributor

This fixes the line, but now the tab fade in / out looks funny. Main branch isn't perfect wrt tab fade in & out, but these changes make it a bit worse

In WASM, AsymmetricRoundedRectangle (used in TabBar for example
on the Levels page) sometimes shows a line on the edge of the
clip item (between the rounded background and the non-rounded
background section).  The issue doesn't reproduce on desktop or
on device, but it can be resolved by overlapping the rounded
background section on top of the normal background section by
1 pixel (can do this now, since colours are opaque).
@chriadam chriadam force-pushed the chriadam/tabbar-pixel-clipping branch from deeab0b to f331d0c Compare July 4, 2024 04:09
@chriadam
Copy link
Contributor Author

chriadam commented Jul 4, 2024

Thanks, fixed, now use a layer to ensure the clip can't mess with the z order etc.

Copy link
Contributor

@DanielMcInnes DanielMcInnes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@chriadam chriadam merged commit 647b1ef into main Jul 4, 2024
1 check passed
@chriadam chriadam deleted the chriadam/tabbar-pixel-clipping branch July 4, 2024 05:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants