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

HMRC header component #829

Merged
merged 7 commits into from
Oct 18, 2017
Merged

HMRC header component #829

merged 7 commits into from
Oct 18, 2017

Conversation

csabascript
Copy link
Contributor

@csabascript csabascript commented Oct 18, 2017

As the new design system is available, we are porting components from component library.

Problem

Header component is not available in design system.

Solution

Port header component from component library.

Example Screenshot

screen shot 2017-10-18 at 14 10 17

@rpowis
Copy link
Contributor

rpowis commented Oct 18, 2017

@csabascript Can you add a screenshot please?

@gavinwye gavinwye changed the title Hmrc header pattern HMRC header component Oct 18, 2017
@gavinwye
Copy link
Contributor

The screenshot doesn't match what we will be putting in to the Design System

@rpowis
Copy link
Contributor

rpowis commented Oct 18, 2017

@csabascript e.g. The .lang-select styles are missing whereas they're in this PR.

refined and clarified some content
@rpowis rpowis merged commit be35cfb into master Oct 18, 2017
@rpowis rpowis deleted the HMRC-header-pattern branch October 18, 2017 14:38
hmrc-web-operations pushed a commit that referenced this pull request Oct 18, 2017
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