-
Notifications
You must be signed in to change notification settings - Fork 4
feat: add new sponsor logos and update layout for better responsiveness #71
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -46,7 +46,9 @@ const SPONSOR_CATEGORIES = [ | |||||||||||||||||
| { name: 'Pisarv', src: '/sponsors/WhatsApp_Image_2026-04-03_at_13.02.15-removebg-preview.png' }, | ||||||||||||||||||
| { name: 'Universal Informatics', src: '/sponsors/Universal_Logo-removebg-preview.png' }, | ||||||||||||||||||
| { name: 'Study Metro', src: '/sponsors/sm_logo-removebg-preview.png' }, | ||||||||||||||||||
| { name: 'Encode AI', src: '/sponsors/image.png' } | ||||||||||||||||||
| { name: 'Encode AI', src: '/sponsors/image.png' }, | ||||||||||||||||||
| { name: 'Quelwebs', src: '/sponsors/WhatsApp_Image_2026-04-04_at_11.37.13-removebg-preview.png' }, | ||||||||||||||||||
| { name: 'study swift', src: '/sponsors/WhatsApp_Image_2026-04-04_at_11.37.49-removebg-preview.png' } | ||||||||||||||||||
|
||||||||||||||||||
| { name: 'study swift', src: '/sponsors/WhatsApp_Image_2026-04-04_at_11.37.49-removebg-preview.png' } | |
| { name: 'Study Swift', src: '/sponsors/WhatsApp_Image_2026-04-04_at_11.37.49-removebg-preview.png' } |
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.
Normalize sponsor name casing for consistent brand/alt text quality.
On Line 51, study swift is lowercased while other brand names are title-cased. Since this value is used in alt, keep naming consistent.
Suggested fix
- { name: 'study swift', src: '/sponsors/WhatsApp_Image_2026-04-04_at_11.37.49-removebg-preview.png' }
+ { name: 'Study Swift', src: '/sponsors/WhatsApp_Image_2026-04-04_at_11.37.49-removebg-preview.png' }📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| { name: 'Encode AI', src: '/sponsors/image.png' }, | |
| { name: 'Quelwebs', src: '/sponsors/WhatsApp_Image_2026-04-04_at_11.37.13-removebg-preview.png' }, | |
| { name: 'study swift', src: '/sponsors/WhatsApp_Image_2026-04-04_at_11.37.49-removebg-preview.png' } | |
| { name: 'Encode AI', src: '/sponsors/image.png' }, | |
| { name: 'Quelwebs', src: '/sponsors/WhatsApp_Image_2026-04-04_at_11.37.13-removebg-preview.png' }, | |
| { name: 'Study Swift', src: '/sponsors/WhatsApp_Image_2026-04-04_at_11.37.49-removebg-preview.png' } |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@src/views/home/SponsorsSection.js` around lines 49 - 51, Normalize the
sponsor name casing in the sponsors list used by the SponsorsSection component:
change the entry with name 'study swift' in the sponsors array to title-case
(e.g., 'Study Swift') so its alt text and branding match the other entries;
update the name string in the sponsors array where the objects are declared to
maintain consistent casing across renderings.
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.
Setting the carousel grid to
position: 'absolute'removes it from the document flow, so the parent’s height no longer matches the content and must be managed separately. If you keep absolute positioning for the slide animation, ensure the parent height is driven by the grid’s actual height (not only a breakpoint-basedminHeight) to prevent overlap/clipping on different content sizes.