Page MenuHomePhabricator

[web] fix button tertiary colors
ClosedPublic

Authored by ginsu on Jan 7 2024, 11:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 23, 7:37 AM
Unknown Object (File)
Sat, Nov 23, 4:30 AM
Unknown Object (File)
Sat, Nov 23, 1:21 AM
Unknown Object (File)
Sun, Nov 3, 5:10 AM
Unknown Object (File)
Sun, Nov 3, 5:10 AM
Unknown Object (File)
Sun, Nov 3, 5:10 AM
Unknown Object (File)
Sun, Nov 3, 5:10 AM
Unknown Object (File)
Sun, Nov 3, 5:10 AM
Subscribers

Details

Summary

I made some small errors with the button tertiary colors that got caught in D10549 and ENG-6353, where I created two identical color variables for --button-label-tertiary-default + --button-label-secondary-defaul and I misspelled tertiary for --button-background-tetirary-hover. This diff addresses both of those mistakes

Test Plan

flow + confirmed there were no regressions

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

ginsu added reviewers: atul, inka.
Harbormaster returned this revision to the author for changes because remote builds failed.Jan 8 2024, 12:11 AM
Harbormaster failed remote builds in B25570: Diff 35337!
ginsu requested review of this revision.Jan 8 2024, 12:19 AM

will make sure ci passes before landing

This revision is now accepted and ready to land.Jan 9 2024, 5:14 AM

rebase to make sure ci passes before landing

This revision was automatically updated to reflect the committed changes.