Updating semantics of census buttons #49526
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Real talk: how often do we use the census reviewer page? Seems worth auditing. It's also very very hard to repro locally.
CensusForm.jsx (only box shadow difference)
Before:
After:
I don't even want to talk about or admit HOW MANY HOURS it took me to create enough false data to actually see these forms. What matters? I got these screenshots and was able to verify that removing these lines doesn't change the buttons apart from the box shadows.
TableBefore:
TableAfter:
ReviewDetailsBefore:
ReviewDetailsAfter:
Links
Testing story
Deployment strategy
Follow-up work
Privacy
Security
Caching
PR Checklist: