MetaMask / MetaMask/metamask-extension
nit: Settings section should be scrollable, not entire page
Open
ux-enhancement
- Dominant language
- TypeScript
- Stars
- 13.2k
- Forks
- 5.6k
- Avg merge
- 2d 5h
- Merged PRs (30d)
- 451
Description
Steps:
- Go to settings > Advanced
- Scroll down to enable a setting
Problem:
- To go back, I now have to scroll to the top of the screen to reveal the settings menu & home buttons.
Solution:
- Only the long list of settings should be scrolling in this view. There's no reason to push the top bar out of view, nor the side menu of settings categories.
Contributor guide
Research direction
Reproduce the issue by opening Settings > Advanced and scrolling down to a setting. Inspect the settings view's layout and scroll container; done means only the settings list scrolls while the top bar, category menu, and home controls remain visible.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- frontend
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100