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

Added a new Variant in Header Component #3459

Merged
merged 10 commits into from
Dec 17, 2024
Merged

Conversation

SriHV
Copy link
Contributor

@SriHV SriHV commented Dec 16, 2024

What is the context of this PR?

ONSDESYS-208

Added a new variant in the header as per figma and prototype

How to review this PR

Check the new example created matches the design in figma and prototype

Checklist

This needs to be completed by the person raising the PR.

  • I have selected the correct Assignee
  • I have linked the correct Issue

@SriHV SriHV self-assigned this Dec 16, 2024
Copy link

netlify bot commented Dec 16, 2024

Deploy Preview for ons-design-system-preview ready!

Name Link
🔨 Latest commit 6155e1e
🔍 Latest deploy log https://app.netlify.com/sites/ons-design-system-preview/deploys/6761a9837698310008bb32df
😎 Deploy Preview https://deploy-preview-3459--ons-design-system-preview.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 site configuration.

@SriHV SriHV added the Enhancement Change of existing feature label Dec 16, 2024
@SriHV SriHV requested a review from a team December 16, 2024 09:18
@SriHV SriHV marked this pull request as ready for review December 16, 2024 09:18
Copy link
Contributor

@rmccar rmccar left a comment

Choose a reason for hiding this comment

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

I don't think nav-only is a good name for this because this variant can be used without navigation, like the example does. Maybe something like basic would be better

src/components/header/_macro-options.md Outdated Show resolved Hide resolved
rmccar
rmccar previously approved these changes Dec 17, 2024
@rmccar rmccar dismissed their stale review December 17, 2024 16:26

Testing

@SriHV SriHV merged commit f32bd96 into main Dec 17, 2024
25 checks passed
@SriHV SriHV deleted the enhancement/208/new-header-variant branch December 17, 2024 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Change of existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants