Visitar URL original
USWDS - Breadcrumb: Use margin to offset breadcrumb list padding. by mejiaj · Pull Request #3737 · uswds/uswds · GitHub
Skip to content

USWDS - Breadcrumb: Use margin to offset breadcrumb list padding. - #3737

Merged
thisisdano merged 1 commit into
developfrom
jm-breadcrumb-margins
Oct 9, 2020
Merged

thisisdano merged 1 commit into
developfrom
jm-breadcrumb-margins

Conversation

@mejiaj

@mejiaj mejiaj commented Oct 2, 2020 •

Copy link
Copy Markdown
Contributor

Preview →

Description

Fixes #3692

Replaced u-margin() mixin on Breadcrumb list with margin so we can
offset the padding with any value, instead of forcing an explicit token.

Additional information

Using #{units($theme-focus-width)} * -1 caused a compile error:

Error: Undefined operation "0.25rem * -1".

Before you hit Submit, make sure you’ve done whichever of these applies to you:

  • Follow the 18F Front End Coding Style Guide and Accessibility Guide.
  • Run npm test and make sure the tests for the files you have changed have passed.
  • Run your code through HTML_CodeSniffer and make sure it’s error free.
  • Title your pull request using this format: [Website] - [UI component]: Brief statement describing what this pull request solves.

#3692

Replaced `u-margin()` mixin on Breadcrumb list with `margin` so we can
offset the padding with any value, instead of forcing an explicit token.
@thisisdano
thisisdano merged commit 3dc296e into develop Oct 9, 2020
@thisisdano
thisisdano deleted the jm-breadcrumb-margins branch October 9, 2020 16:15
@thisisdano thisisdano mentioned this pull request Dec 17, 2020
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.

USWDS - Breadcrumb: Possible invalid margin token.

2 participants