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

Add inverse flag to contents list component #1037

Merged
merged 2 commits into from
Aug 9, 2019

Conversation

vanitabarrett
Copy link
Contributor

@vanitabarrett vanitabarrett commented Aug 9, 2019

What

Screen Shot 2019-08-09 at 12 09 58

Why

We have a need for using the contents list component within a topic page header, which is blue.

https://govuk-publishing-compo-pr-1037.herokuapp.com/component-guide/contents_list/inverse

Vanita Barrett added 2 commits August 8, 2019 01:46
We have a need to use the contents list component within a topic page header, which has a blue background.
@bevanloon bevanloon temporarily deployed to govuk-publishing-compo-pr-1037 August 9, 2019 11:11 Inactive
@bevanloon bevanloon temporarily deployed to govuk-publishing-compo-pr-1037 August 9, 2019 11:12 Inactive
@vanitabarrett vanitabarrett marked this pull request as ready for review August 9, 2019 11:14
@vanitabarrett vanitabarrett force-pushed the add-inverse-contents-list branch from b3833fc to 6ceac65 Compare August 9, 2019 11:20
@alex-ju alex-ju added this to the 18.0.0 milestone Aug 9, 2019
@vanitabarrett vanitabarrett merged commit c6955ce into master Aug 9, 2019
@vanitabarrett vanitabarrett deleted the add-inverse-contents-list branch August 9, 2019 11:30
JonathanHallam added a commit that referenced this pull request Sep 5, 2019
Reverts #1037
Trello - https://trello.com/c/d9KRrVMb/207-remove-the-contents-list-inverse-option

We no longer need to use the contents list component in a topic page header

Reverts #1037
JonathanHallam added a commit that referenced this pull request Sep 5, 2019
Reverts #1037
Trello - https://trello.com/c/d9KRrVMb/207-remove-the-contents-list-inverse-option

We no longer need to use the contents list component in a topic page header

Reverts #1037
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.

4 participants