-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
fixed category slider in integrations page #5270
Conversation
Signed-off-by: prabhjeevnijjar <[email protected]>
🚀 Preview for commit 2d68a6e at: https://659837064804832d8bfabe93--layer5.netlify.app |
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.
@prabhjeevnijjar thank you.
Is there anything that we can do about this? This happens in Safari (and maybe other browsers).
@prabhjeevnijjar, we have another issue with respect to maintaining aspect ration of the icons. I wonder if you might have a look into this as well. Screenshot from Chrome. |
Just checked, Logo is too wide hence the compression
let me know which one i go with. |
Its safari only problem 😆 |
@prabhjeevnijjar |
@prabhjeevnijjar, while we won't be able to change this logo, we can use content-fit or something similar to control it's display. Let me know if you get stuck and can't quite figure this one out. |
We can merge your progress, if you open up a new issue to track the other item discussed in the comments. |
Sure |
|
Created new issue for stretching of some logos: #5308 |
Thank you @prabhjeevnijjar |
Description
This PR fixes #5269
Notes for Reviewers
Signed commits