-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
fix: add overlay for sidebar on mobile #3197
Conversation
the overlay in frontend\src\components\header\HeaderBar.vue Add a display condition “layoutStore.currentPromptName == 'sidebar'” |
True, I just dont like the gray look of the overlay there and i didnt wanna change the style: |
I agree that it doesn't look good in the light theme. Do you have any ideas about how we can improve it? |
Sure, ill test some ideas over weekend. |
Should I merge this one, or are you going to revisit it? |
Yes please, so at least we can close the related issue. |
Description
Fixes #3190
🚨 Before submitting your PR, please indicate which issues are either fixed or closed by this PR. See GitHub Help: Closing issues using keywords.