[#69] Fix content overlap and improve design of ClientLogos component#81
[#69] Fix content overlap and improve design of ClientLogos component#81jbampton merged 5 commits intometropolis-retro:mainfrom
Conversation
|
@hamza-bousalih is attempting to deploy a commit to the retro Team on Vercel. A member of the Team first needs to authorize it. |
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the .gitignore to exclude .DS_Store and refines the marquee animation in globals.css by increasing its duration and organizing it within the components layer. The ClientLogos component has been significantly redesigned with improved layout, gradient masks, and a new pill-based styling for community names. Feedback highlights potential CSS class conflicts in the SectionTitle component and provides a suggestion to fix a visual jump in the marquee animation by adjusting how spacing is applied between items.
DeepSource Code ReviewWe reviewed changes in See full review on DeepSource ↗ Important Some issues found as part of this review are outside of the diff in this pull request and aren't shown in the inline review comments due to GitHub's API limitations. You can see those issues on the DeepSource dashboard. Code Review Summary
|
![deepsource-io[bot]](https://avatars.githubusercontent.com/in/16372?s=60&v=4){kind=small}
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
2 participants
No description provided.