-
Notifications
You must be signed in to change notification settings - Fork 63
Refactor SCREEN_SIZES
to match z-indices.js
#1985
Conversation
Storybook and Tailwind configuration previews: Ready Storybook: https://wordpress.github.io/openverse-frontend/_preview/1985 Please note that GitHub pages takes a little time to deploy newly pushed code, if the links above don't work or you see old versions, wait 5 minutes and try again. You can check the GitHub pages deployment action list to see the current status of the deployments. |
Size Change: -2.88 kB (0%) Total Size: 821 kB
ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the ease with which we can use the screens. Approving, but would like to see more documentation about why xs
is not real 😆
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed with @obulat about the special handling of xs
being confusing (I sadly don't remember why it's an exception, so documentation would be useful) but this LGTM.
SCREEN_SIZES
a Record
instead of a Map
SCREEN_SIZES
to match z-indices.js
Description
This PR changes
SCREEN_SIZES
, which is currently aMap
into aRecord
, which makes it more ergonomic to use and improves type-safety.Checklist
Update index.md
).main
) or a parent feature branch.Developer Certificate of Origin
Developer Certificate of Origin