-
-
Notifications
You must be signed in to change notification settings - Fork 666
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Navigation switch #7132
feat: Navigation switch #7132
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
paddingTop: theme.spacing(2), | ||
paddingBottom: theme.spacing(6), | ||
height: '100%', | ||
position: 'absolute', |
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.
removes sidebar from regular flow
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'm not sure about this change, since I'd like the content area to have the same padding on all sides when the sidebar is active. But let's see how it plays out.
About the changes
Simple show/hide switch to have keyboard navigation
![Screenshot 2024-05-23 at 17 50 41](https://private-user-images.githubusercontent.com/1394682/333255516-f841a5d7-2da5-4f50-9385-a0ba4b8a22fb.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjAzNjY2MTksIm5iZiI6MTcyMDM2NjMxOSwicGF0aCI6Ii8xMzk0NjgyLzMzMzI1NTUxNi1mODQxYTVkNy0yZGE1LTRmNTAtOTM4NS1hMGJhNGI4YTIyZmIucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDcwNyUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA3MDdUMTUzMTU5WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9MTc0NjFiMGU1NDBjNzY3ZTg4ODIxM2M1MTcwMzk3NzhhYmJjYTQ1ZTQ0YjQyODY4MzM2ZjdjMGFhMmZhMWEyMSZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.eVzFVkxVv5YvQ3IOyQ6kTnYW7_LNI99KBCTAcNRlge4)
![Screenshot 2024-05-23 at 17 50 50](https://private-user-images.githubusercontent.com/1394682/333255576-9964759c-9899-4841-8098-d16c7412d439.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjAzNjY2MTksIm5iZiI6MTcyMDM2NjMxOSwicGF0aCI6Ii8xMzk0NjgyLzMzMzI1NTU3Ni05OTY0NzU5Yy05ODk5LTQ4NDEtODA5OC1kMTZjNzQxMmQ0MzkucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDcwNyUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA3MDdUMTUzMTU5WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9ZWQwMzdmM2M5ZDkwODdmMzZkNDEzMTNmYTU1NDNjMjE2Nzg4NWMyNzg5ZTYxNDYwYzYzN2QyOGNhMDNhYjFkNSZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.bvSd9Gt9-DwFV6nBBpkykkGCJrfaoxv6PfyHO4xDinw)
Also changed positioning to absolute so that the project search content doesn't escape the parent and the show main content is not shifted on resizing of the sidebar. As a result we need a way to overlay the sidebar over the content on smaller screens.
Important files
Discussion points