Skip to content

Conversation

@giamir
Copy link
Contributor

@giamir giamir commented Dec 19, 2025

For navigation items there is no need to enforce a white static background for avatars. In fact we have a need for the transparency of the images to be respected. This PR simply specify that for avatar included in navigation items background should remain transparent.

See this slack thread for full context.

@changeset-bot
Copy link

changeset-bot bot commented Dec 19, 2025

🦋 Changeset detected

Latest commit: 26164bf

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@netlify
Copy link

netlify bot commented Dec 19, 2025

Deploy Preview for stacks ready!

Name Link
🔨 Latest commit 26164bf
🔍 Latest deploy log https://app.netlify.com/projects/stacks/deploys/69451ae80282a100086f7f53
😎 Deploy Preview https://deploy-preview-2108--stacks.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link

netlify bot commented Dec 19, 2025

Deploy Preview for stacks-svelte ready!

Name Link
🔨 Latest commit 26164bf
🔍 Latest deploy log https://app.netlify.com/projects/stacks-svelte/deploys/69451ae8c1a9c1000854f4ed
😎 Deploy Preview https://deploy-preview-2108--stacks-svelte.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

@dancormier dancormier left a comment

Choose a reason for hiding this comment

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

👍

@giamir giamir merged commit d91d8c8 into beta Dec 19, 2025
18 checks passed
@giamir giamir deleted the gbuoncristiani/navigation-avatar branch December 19, 2025 15:34
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