Skip to content

Add customer account menu web component#77

Merged
jessebreneman merged 1 commit intomainfrom
jb-add-customer-account-menu
Feb 26, 2026
Merged

Add customer account menu web component#77
jessebreneman merged 1 commit intomainfrom
jb-add-customer-account-menu

Conversation

@jessebreneman
Copy link
Contributor

Adds the new <shopify-account> web component to the header, replacing the existing account link.

@jessebreneman jessebreneman self-assigned this Feb 19, 2026
@karreiro karreiro self-requested a review February 19, 2026 17:10
Copy link
Contributor

@karreiro karreiro left a comment

Choose a reason for hiding this comment

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

Thank you for this PR, @jessebreneman!

Looks good to me. The only detail we might optimize is removing the settings in favor of the hard-coded menu value at sections/header.liquid:14.

That introduces a bit of ambiguity since merchants may reassign any kind of menu in that setting. To keep the Skeleton theme simple, we might lean on the hardcoded approach instead.

Thanks again for this PR!

Copy link
Contributor

@karreiro karreiro left a comment

Choose a reason for hiding this comment

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

Thank you for the extra context shared offline, @jessebreneman!

@jessebreneman
Copy link
Contributor Author

@karreiro Thank you for reviewing!

@jessebreneman jessebreneman merged commit a4f32d3 into main Feb 26, 2026
3 checks passed
@jessebreneman jessebreneman deleted the jb-add-customer-account-menu branch February 26, 2026 21:14
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