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 breadcrumb and back link modifiers and styles #3774

Merged
merged 3 commits into from
Jun 9, 2023

Conversation

querkmachine
Copy link
Member

@querkmachine querkmachine commented Jun 9, 2023

Because everyone loves a small story.

Adds inverted colour scheme variants of the Breadcrumb and Back Link components, for use on darker backgrounds, as is used on product landing pages and some sections of GOV.UK.

Related to #2266.

Thoughts

  • We don't have a mechanism for showing examples of these inverse coloured variants in the review app—currently it's white text on a white background. We should probably do something about that!

@querkmachine querkmachine self-assigned this Jun 9, 2023
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-3774 June 9, 2023 13:20 Inactive
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-3774 June 9, 2023 13:29 Inactive
@querkmachine querkmachine requested a review from a team June 9, 2023 13:40
@querkmachine querkmachine marked this pull request as ready for review June 9, 2023 13:41
@owenatgov owenatgov self-requested a review June 9, 2023 16:04
Copy link
Contributor

@owenatgov owenatgov left a comment

Choose a reason for hiding this comment

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

Does the job. Tested by applying the class and adding a black background locally.

You make a good point on not being able to see examples of this in the review app and I think we can get around this relatively easily by updating the showExamples macro. I'll put an issue together.

@owenatgov
Copy link
Contributor

An extra comment: I wonder if we should keep the linked issue open after this gets merged for 2 reasons:

  1. Technically speaking, the issue requires these examples get added to the review app and as we've illustrated that's not possible yet. Hopefully this can be solved with Allow for viewing of inverse modifications of components in the review app #3780
  2. Ollie makes a comment on the border at the bottom of the breadcrumbs in the govuk examples. I think it's worth using that issue to explore this more in future

@querkmachine
Copy link
Member Author

Good points, I've removed the closing comment.

@querkmachine querkmachine merged commit 1f39c1c into main Jun 9, 2023
21 checks passed
@querkmachine querkmachine deleted the kg-inverted-breadcrumbs-back-links branch June 9, 2023 16:45
peteryates added a commit to x-govuk/govuk-components that referenced this pull request Jun 25, 2023
Support was recently added for inverse breadcrumbs[0]. Like the recent
inverse button change[1] we're using the `inverse` keyword arg to allow
the mode to be toggled.

Refs #428

[0] #418
[1] alphagov/govuk-frontend#3774
peteryates added a commit to x-govuk/govuk-components that referenced this pull request Jun 25, 2023
Support was recently added for inverse breadcrumbs[0]. Like the recent
inverse button change[1] we're using the `inverse` keyword arg to allow
the mode to be toggled.

Refs #428

[0] #418
[1] alphagov/govuk-frontend#3774
peteryates added a commit to x-govuk/govuk-components that referenced this pull request Jun 25, 2023
Support was recently added for inverse breadcrumbs[0]. Like the recent
inverse button change[1] we're using the `inverse` keyword arg to allow
the mode to be toggled.

Refs #428

[0] #418
[1] alphagov/govuk-frontend#3774
peteryates added a commit to x-govuk/govuk-components that referenced this pull request Jul 8, 2023
Support was recently added for inverse breadcrumbs[0]. Like the recent
inverse button change[1] we're using the `inverse` keyword arg to allow
the mode to be toggled.

Refs #428

[0] #418
[1] alphagov/govuk-frontend#3774
peteryates added a commit to x-govuk/govuk-components that referenced this pull request Jul 8, 2023
Support was recently added for inverse breadcrumbs[0]. Like the recent
inverse button change[1] we're using the `inverse` keyword arg to allow
the mode to be toggled.

Refs #428

[0] #418
[1] alphagov/govuk-frontend#3774
peteryates added a commit to x-govuk/govuk-components that referenced this pull request Jul 8, 2023
Support was recently added for inverse breadcrumbs[0]. Like the recent
inverse button change[1] we're using the `inverse` keyword arg to allow
the mode to be toggled.

Refs #428

[0] #418
[1] alphagov/govuk-frontend#3774
peteryates added a commit to x-govuk/govuk-components that referenced this pull request Jul 8, 2023
Support was recently added for inverse breadcrumbs[0]. Like the recent
inverse button change[1] we're using the `inverse` keyword arg to allow
the mode to be toggled.

Refs #428

[0] #418
[1] alphagov/govuk-frontend#3774
colinrotherham pushed a commit that referenced this pull request Jul 19, 2023
…inks

Add inverse breadcrumb and back link modifiers and styles
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.

None yet

3 participants