Skip to content

[Color Modes] Remove presentational utilities/variables#1233

Merged
simurai merged 3 commits intomkt/color-modes-wheefrom
color_modes/deprecate
Mar 4, 2021
Merged

[Color Modes] Remove presentational utilities/variables#1233
simurai merged 3 commits intomkt/color-modes-wheefrom
color_modes/deprecate

Conversation

@simurai
Copy link
Copy Markdown
Contributor

@simurai simurai commented Mar 4, 2021

This removes the presentational utilities/variables that got replaced with functional names. See migration guide.

Just a few examples:

  • .link-gray
  • $border-gray-dark
  • $bg-green
  • $text-blue
  • $box-shadow-medium
  • .box-shadow-extra-large
  • .bg-gray-dark
  • .text-yellow
  • .border-blue
  • etc.

Edit: Some of the utilities got added back so that they still work for ViewComponents: 376e0d4

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 4, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/primer/primer-css/2Z4LHTGc8ZEfJ8TpvRNCRpG5Qd87
✅ Preview: https://primer-css-git-colormodes-deprecate-primer.vercel.app

@vercel vercel bot temporarily deployed to Preview March 4, 2021 13:04 Inactive
@simurai simurai marked this pull request as ready for review March 4, 2021 13:24
@simurai
Copy link
Copy Markdown
Contributor Author

simurai commented Mar 4, 2021

Ok, I tested this in https://github.com/github/github/pull/172506 and seems good. So I'll merge this into mkt/color-modes-whee.

@simurai simurai merged commit 89d022b into mkt/color-modes-whee Mar 4, 2021
@simurai simurai deleted the color_modes/deprecate branch March 4, 2021 13:25
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.

1 participant