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 null variables for nav-link #31035

Merged
merged 2 commits into from
Jul 11, 2020
Merged

Add null variables for nav-link #31035

merged 2 commits into from
Jul 11, 2020

Conversation

mdo
Copy link
Member

@mdo mdo commented Jun 15, 2020

Replaces #30498 by adding four new null default variables for .nav-link. Doesn't carry over font-style from the original PR though since that's rarely used, at least by default Bootstrap. Nullifies all values from that PR, too, since we count on some basic inheritance here and don't need color by default.

Fixes #30847.

Replaces #30498 by adding four new null default variables for .nav-link. Doesn't carry over font-style from the original PR though since that's rarely used, at least by default Bootstrap. Nullifies all values from that PR, too, since we count on some basic inheritance here and don't need color by default.
@mdo
Copy link
Member Author

mdo commented Jun 15, 2020

Do we backport? Thinking maybe we could actually.

@mdo mdo changed the base branch from master to main June 16, 2020 19:30
@mdo mdo added this to Inbox in v5.0.0-alpha2 via automation Jun 16, 2020
@mdo mdo added this to Inbox in v4.5.1 via automation Jun 16, 2020
@mdo
Copy link
Member Author

mdo commented Jul 11, 2020

Nixing the backport for now assuming we're doing v4.5.x and not a v4.6.x release.

@mdo mdo removed this from Inbox in v4.5.1 Jul 11, 2020
@mdo mdo merged commit 2b4db02 into main Jul 11, 2020
v5.0.0-alpha2 automation moved this from Inbox to Shipped Jul 11, 2020
@mdo mdo deleted the nav-vars branch July 11, 2020 18:56
mdo added a commit that referenced this pull request Jul 11, 2020
XhmikosR added a commit that referenced this pull request Jul 13, 2020
* Clarify screen reader changes

* Add some docs and reboot notes to migration guide

* Add mention of docs renaming of screen reader helper page

* Mention null vars from navs PR at #31035

* Update migration.md

Co-authored-by: Patrick H. Lauke <redux@splintered.co.uk>
Co-authored-by: XhmikosR <xhmikosr@gmail.com>
olsza pushed a commit to olsza/bootstrap that referenced this pull request Oct 3, 2020
* Clarify screen reader changes

* Add some docs and reboot notes to migration guide

* Add mention of docs renaming of screen reader helper page

* Mention null vars from navs PR at twbs#31035

* Update migration.md

Co-authored-by: Patrick H. Lauke <redux@splintered.co.uk>
Co-authored-by: XhmikosR <xhmikosr@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
v5.0.0-alpha2
  
Shipped
Development

Successfully merging this pull request may close these issues.

No scss variable to change color for a .nav-item inside a .nav
2 participants