- Notifications
You must be signed in to change notification settings - Fork928
fix: Page header width in the workspace page#2125
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
@Kira-Pilot what do you think about the way this calculates the page header width? |
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.
LGTM
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 think this looks good! Eventually I would love to move away from hardcoded widths entirely (that 300px is my crime, i know) but I feel like that is a change to be handled later down the road.
Fix bug introduced on#2097