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 bottom border option to breadcrumb #1952

Merged
merged 1 commit into from
Mar 2, 2021
Merged

Conversation

injms
Copy link
Contributor

@injms injms commented Mar 1, 2021

What

Adds a parameter so a border can be added to the bottom of a breadcrumb.

Why

This is needed so this component can be used in alphagov/manuals-frontend.

Visual Changes

Desktop:

image

Mobile:

@bevanloon bevanloon temporarily deployed to govuk-publis-add-border-5fcgvl March 1, 2021 17:31 Inactive
@injms injms requested review from chris-gds and owenatgov March 1, 2021 17:35
@chris-gds
Copy link
Contributor

chris-gds commented Mar 1, 2021

All good 👍🏼

Screenshot 2021-03-01 at 18 49 21

Screenshot 2021-03-01 at 18 47 57

(the visuals here have collapse_on_mobile: true active - which alters the padding)

eg without:

image

Adding a border on the bottom of a breadcrumb is needed so this component can be
used in alphagov/manuals-frontend.
@injms injms force-pushed the add-border-to-breadcrumb branch from f80cfbc to 7d6d9a7 Compare March 2, 2021 09:41
@bevanloon bevanloon temporarily deployed to govuk-publis-add-border-5fcgvl March 2, 2021 09:41 Inactive
@injms injms merged commit 77496db into master Mar 2, 2021
@injms injms deleted the add-border-to-breadcrumb branch March 2, 2021 09:51
@danacotoran danacotoran mentioned this pull request Mar 2, 2021
This was referenced Mar 11, 2021
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