Skip to content
This repository was archived by the owner on Mar 17, 2026. It is now read-only.

Add header top bar section - #10

Merged
srgzlb merged 5 commits into
mainfrom
feature/sc-1857-header-section-top-bar
Oct 28, 2025
Merged

srgzlb merged 5 commits into
mainfrom
feature/sc-1857-header-section-top-bar

Conversation

@srgzlb

@srgzlb srgzlb commented Oct 23, 2025 •

Copy link
Copy Markdown
Contributor

This PR adds a new top bar section to the header and refactors some styles

Arc 2025-10-23 18 51 06

@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

  • SCSS Linting: ✅ Passed
  • JavaScript Linting: ✅ Passed
  • Build Process: ✅ Passed

🎉 All checks passed! Ready for review.

@srgzlb srgzlb added review Please look at my code shop-team Shop and commerce labels Oct 23, 2025
@booqable booqable deleted a comment from github-actions Bot Oct 24, 2025
@booqable booqable deleted a comment from github-actions Bot Oct 24, 2025
@JKinnez
JKinnez self-requested a review October 24, 2025 08:02
@dabrorius
dabrorius self-requested a review October 27, 2025 09:01
Comment thread src/scss/_helpers/_mixins.scss Outdated
}

// Flex center mixin
@mixin flex-center {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Do we really need a mixin for 2 lines of CSS?

@srgzlb srgzlb removed review Please look at my code shop-team Shop and commerce labels Oct 28, 2025
@github-actions

Copy link
Copy Markdown

🔍 Code Quality Report

  • SCSS Linting: ✅ Passed
  • JavaScript Linting: ✅ Passed
  • Build Process: ✅ Passed

🎉 All checks passed! Ready for review.

@srgzlb
srgzlb merged commit 72930ca into main Oct 28, 2025
3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants