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

V4 dev backports #30592

Merged
merged 12 commits into from
Apr 28, 2020
Merged

V4 dev backports #30592

merged 12 commits into from
Apr 28, 2020

Conversation

XhmikosR
Copy link
Member

@XhmikosR XhmikosR commented Apr 15, 2020

Preview: https://deploy-preview-30592--twbs-bootstrap.netlify.com/

TODO:

  • Verify the dashboard changes are OK
  • Verify the a11y color changes in docs

XhmikosR and others added 3 commits April 18, 2020 09:27
Cleans up some column classes to ensure things line up across all viewports. Also fixes some broken padding on the .navbar-brand.

Co-authored-by: XhmikosR <xhmikosr@gmail.com>
@XhmikosR XhmikosR force-pushed the v4-dev-xmr branch 2 times, most recently from 2947ea0 to e13db00 Compare April 18, 2020 06:48
zalog and others added 3 commits April 21, 2020 15:14
Co-authored-by: Martijn Cuppens <martijn.cuppens@gmail.com>
checks for `$grid-row-columns`
grid: checks for `$grid-columns > 0`
@XhmikosR XhmikosR force-pushed the v4-dev-xmr branch 2 times, most recently from cb15aaa to e351318 Compare April 21, 2020 12:19
Remove caret element
@XhmikosR XhmikosR marked this pull request as ready for review April 28, 2020 06:35
@XhmikosR XhmikosR requested review from a team as code owners April 28, 2020 06:35
@XhmikosR XhmikosR force-pushed the v4-dev-xmr branch 2 times, most recently from 9692fb8 to f9bd7d7 Compare April 28, 2020 06:55
@XhmikosR
Copy link
Member Author

@MartijnCuppens can you confirm the dashboard and the a11y changes are OK and if so approve?

@MartijnCuppens
Copy link
Member

Spotted 1 missing color change (<!doctype html> color), also fixed the commit message of this commit:
image

Copy link
Member

@MartijnCuppens MartijnCuppens left a comment

Choose a reason for hiding this comment

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

:shipit:

@XhmikosR
Copy link
Member Author

I actually didn't backport the whole #29208 only 9b97dc6

@MartijnCuppens
Copy link
Member

MartijnCuppens commented Apr 28, 2020

Ow, ok, shouldn't we just ditch the # than?

Edit: I removed the # and pushed the change

XhmikosR and others added 3 commits April 28, 2020 19:03
Tweak syntax highlighting colors to be WCAG2AA valid.

Co-authored-by: Martijn Cuppens <martijn.cuppens@gmail.com>
This is so that bundlewatch compares against v4-dev and not master.
The latest `actions/setup-node` prints this info by default
@XhmikosR
Copy link
Member Author

XhmikosR commented Apr 28, 2020 via email

@MartijnCuppens
Copy link
Member

It was referring to the commit hash, not sure why it wasn't highlighted.

It wasn't highlighted because it was prefixed with #, fixed that in the latest push.

@XhmikosR
Copy link
Member Author

It wasn't highlighted because it was prefixed with #, fixed that in the latest push.

Duh! Thanks!

@XhmikosR XhmikosR merged commit 748b1b9 into v4-dev Apr 28, 2020
@XhmikosR XhmikosR deleted the v4-dev-xmr branch April 28, 2020 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants