fix: Scroll metadata sidebar tab content #7499
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.
An attempt at addressing #7390 which looks like,
Chrome
![Screenshot from 2024-06-05 14-24-13](https://cdn.statically.io/img/private-user-images.githubusercontent.com/620580/336663834-57dfe4f6-f8bc-4238-8550-719be71cf437.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjE4MjExODEsIm5iZiI6MTcyMTgyMDg4MSwicGF0aCI6Ii82MjA1ODAvMzM2NjYzODM0LTU3ZGZlNGY2LWY4YmMtNDIzOC04NTUwLTcxOWJlNzFjZjQzNy5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzI0JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcyNFQxMTM0NDFaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT03NzVjZWViYjFjMDI3ZTJjNGNlYzI1YWI4OGYxM2RkMDgwMmE4NjkwZTNjNDQ3NWE2ZGVjYzU3ZjZjYTY1MjdhJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.bMA07kWGm0SwtqD91IgRHc_iHdsc3kdiFgie1V9KCwI)
Firefox
![Screenshot from 2024-06-05 14-24-50](https://cdn.statically.io/img/private-user-images.githubusercontent.com/620580/336663856-40528431-8673-4d0f-ae1e-6666a0fde057.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjE4MjExODEsIm5iZiI6MTcyMTgyMDg4MSwicGF0aCI6Ii82MjA1ODAvMzM2NjYzODU2LTQwNTI4NDMxLTg2NzMtNGQwZi1hZTFlLTY2NjZhMGZkZTA1Ny5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzI0JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcyNFQxMTM0NDFaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT0zMWZiYzc5ZjFjMjY5MmNlY2E4ZjNlMzZhYmQ4ODAxMGIzYWUzZDBiOGEyYmE0MzRhZWQ0MGY2YmJjZDAwYzNmJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.k-FcocR5c0cLqMHoM2DZVqVU1pJglesjFbv5EGprY0I)
For UX reasons just the tab content in the sidebar has been made scrollable. Is that ok? The UX reason are that if the sidebar had a full scrollbar it would look like the page itself had 2 scrollbars, whereas this is more clearly just making the tab content scrollable. This does mean that very short screens might not have enough visible scrolling area but such a small screen is probably mobile/tablet which needs a different responsive approach anyway.
Thoughts?
@larseggert @martinthomson