-
Notifications
You must be signed in to change notification settings - Fork 142
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 padding on buy screen within modal #4253
Comments
I can fix this and also this issue: #4423 But we don't have visual designs for a lot of screens across all views so we have improvised here. It would be helpful to have pixel perfect designs for screens in at least full screen and extension view so we can set padding etc. correctly |
Thanks for linking this design @mica000 . I've made some changes:
When resizing the browser, when I move from full tab the changes don't kick in until we are in a narrow screen. Let me know if you want me to further adjust heading size / padding along the way Kapture.2023-11-01.at.11.34.18.mp4 |
@pete-watters Sorry, I see that you just closed the issue but didnt add the feedback yet. My bad! Here is the issue I talked about: Wondering if we could:
|
@mica000 not your bad at all, I closed it as I fixed the other problems. I can see the issue too and it looks bad in dark mode also: It seems:
I'll fix it in the containers issue if that's OK? I will be working on that next week |
We seem to have too much in a few places:
The text was updated successfully, but these errors were encountered: