[Popover] Don't hide scrollbar
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 99.1k
- Forks
- 32.5k
- Avg merge
- 2d 17h
- Merged PRs (30d)
- 106
Description
- I have searched the issues of this repository and believe that this is not a duplicate.
Summary 💡
In the documentation, it is written: "The scroll and click away are blocked unlike with the Popper component.". But why? Why not allowing to unlock the scrollbar as it is the case for Select, Menu, Dialog, etc...?
You will say me: "use the popper". I can do it, but the popper misses a lot of features that the popover has (escape key & click away).
Moreover, Google alreadys uses Popover that doesn't block the scrollbar:
So it would be nice if there is a way to not block the scrollbar on Popover (this is the only that prevents me to use it everytime I want to use it).
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the Popover documentation entry point and compare its scrollbar behavior with Select, Menu, and Dialog. Determine how Popover could expose the requested option without losing escape-key and click-away behavior; done means scrollbar blocking can be disabled and the documentation explains the behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, react
- Domain
- frontend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100