-
-
Notifications
You must be signed in to change notification settings - Fork 9.8k
Make theme desktop responsive breakpoint configurable #9603
Copy link
Copy link
Open
Labels
proposalThis issue is a proposal, usually non-trivial changeThis issue is a proposal, usually non-trivial change
Description
Have you read the Contributing Guidelines on issues?
- I have read the Contributing Guidelines on issues.
Motivation
See
- feat(theme-common): Get desktopThresholdWidth from docusaurus config #9261
- refactor(theme-common): allow optional desktopBreakpoint param in useWindowSize #9335
People want to be able to configure easily the 996px desktop breakpoint.
It's not easy currently because the value is hardcoded in both the JS side, but also CSS side (inside Infima directly), cf my comment #9261 (comment)
We should make this easier to achieve without requiring swizzling.
Self-service
- I'd be willing to do some initial work on this proposal myself.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
proposalThis issue is a proposal, usually non-trivial changeThis issue is a proposal, usually non-trivial change