Skip to content

Conversation

@pixelspark
Copy link
Contributor

This PR by @Lubrsi was automatically closed, but it fixes an issue that affects my personal website so I thought let's fix it :-)

My PR adds the requested 'test flakiness' fix (it uses an external SVG file instead of a data URI and has an img tag to load the image) to the previous work done in #3646.

Note that (at least on my mac) there seems to be an issue updating the mask position now (it only does so after hovering over one of the masked items. On my website this triggers a change in CSS transform). Any pointers on how to fix this would be appreciated!

mask-image-scroll-offset-3

This makes icon SVGs not disappear when scrolling on
https://www.fangamer.com/. Icons on this site are boxes with background
colours, with the SVG of the icon masking it with mask-image.
@ladybird-bot
Copy link
Collaborator

Hello!

One or more of the commit messages in this PR do not match the Ladybird code submission policy, please check the lint_commits CI job for more details on which commits were flagged and why.
Please do not close this PR and open another, instead modify your commit message(s) with git commit --amend and force push those changes to update this PR.

@pixelspark pixelspark force-pushed the mask-image-scroll-offset branch from 6d106c2 to c16dd5c Compare April 10, 2025 18:08
@pixelspark pixelspark marked this pull request as draft April 10, 2025 18:08
@pixelspark pixelspark force-pushed the mask-image-scroll-offset branch from c16dd5c to 7252b7e Compare April 10, 2025 18:31
@pixelspark pixelspark force-pushed the mask-image-scroll-offset branch from 7252b7e to 1056c17 Compare April 10, 2025 19:32
@github-actions
Copy link

github-actions bot commented May 2, 2025

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions!

@github-actions github-actions bot added the stale label May 2, 2025
@github-actions
Copy link

github-actions bot commented May 9, 2025

This pull request has been closed because it has not had recent activity. Feel free to open a new pull request if you wish to still contribute these changes. Thank you for your contributions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants