-
-
Notifications
You must be signed in to change notification settings - Fork 234
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: now search input is no longer squised on non-macOS devices #3487
fix: now search input is no longer squised on non-macOS devices #3487
Conversation
used ternary operator to switch width, didnot use w-72 as the header logo gets very small when approaching sm breakpoint fix open-sauced#3481
✅ Deploy Preview for oss-insights ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for design-insights ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
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.
PR Compliance Checks Passed!
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.
Looks great! Thanks for the fox @msaadg! 🚢
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.
Looks great! Thanks for the fox @msaadg! 🚢
@nickytonline if it's okay to ask, what is the tool you used to show the design across different platforms? |
## [2.32.0](v2.31.0...v2.32.0) (2024-05-28) ### 🐛 Bug Fixes * added more padding between footer and highlights ([#3483](#3483)) ([352cde5](352cde5)) * now if a dynamic widget errors out, StarSearch continues to work ([#3474](#3474)) ([f840d6b](f840d6b)) * now search input is no longer squised on non-macOS devices ([#3487](#3487)) ([15d70ce](15d70ce)) * now StarSearch suggestion boxes have uniform widths ([#3471](#3471)) ([1302d33](1302d33)) * now the outline button shows button text when disabled ([#3485](#3485)) ([cb8bd3a](cb8bd3a)) * **search:** only show one not found message ([#3477](#3477)) ([7714523](7714523)) * update feed highlight to StarSearch ([#3489](#3489)) ([c99921a](c99921a)) ### 🍕 Features * Adds URL parameter validators to fetch requests ([#3467](#3467)) ([4c056d9](4c056d9)) * Updates error copy StarSearch ([#3494](#3494)) ([735c8c6](735c8c6))
Description
Used ternary operator to switch between w-56 and w-64, did not use w-72 as the header logo gets very small when approaching sm breakpoint.
Related Tickets & Documents
Fixes #3481
Mobile & Desktop Screenshots/Recordings
Before
Non-macOS
macOS
After
Non-macOS
macOS
Steps to QA
Tier (staff will fill in)
[optional] What gif best describes this PR or how it makes you feel?