Skip to content

feat: update client logo carousel#101

Merged
Gowiem merged 3 commits intomasterfrom
feat/update-client-logos
Mar 26, 2026
Merged

feat: update client logo carousel#101
Gowiem merged 3 commits intomasterfrom
feat/update-client-logos

Conversation

@Gowiem
Copy link
Copy Markdown
Member

@Gowiem Gowiem commented Mar 25, 2026

Summary

  • Add new client logos: Cursor, MarketSpark, MetaCTO (with links to their websites)
  • Hide logos: Makr, We The Collective, Watermark
  • Reorder: Formation Bio moved to Develop into master #3, Thinaer moved to end
  • Fix carousel autoplay: eliminate initial scroll delay, tune timing to reduce stutter

Test plan

  • Verify new logos render correctly in carousel
  • Verify logo links open correct websites in new tab
  • Check carousel scrolls immediately on page load without stutter

🤖 Generated with Claude Code

Summary by CodeRabbit

Release Notes

  • New Features

    • Added three new client logos (Cursor, MarketSpark, MetaCTO) to the carousel display.
  • Updates

    • Increased carousel animation speed for faster transitions between logos.
    • Adjusted logo visibility and reordered display sequence in carousel.
    • Hid three logos from active carousel display.

- Add Cursor, MarketSpark, and MetaCTO logos with links
- Hide Makr, We The Collective, and Watermark logos
- Move Thinaer to end, Formation Bio to #3
- Fix carousel: eliminate initial delay, tune autoplay timing

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 25, 2026

Deploy Preview for masterpoint ready!

Name Link
🔨 Latest commit 3e8eb75
🔍 Latest deploy log https://app.netlify.com/projects/masterpoint/deploys/69c563a51992f100083ab330
😎 Deploy Preview https://deploy-preview-101--masterpoint.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 29
Accessibility: 86
Best Practices: 83
SEO: 79
PWA: 70
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 25, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (3)
  • static/img/logos/cursor.svg is excluded by !**/*.svg
  • static/img/logos/marketspark.png is excluded by !**/*.png
  • static/img/logos/metacto.svg is excluded by !**/*.svg

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: b284cdf9-f5c4-4db2-8815-83a0a886b5a1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

The PR updates the client logos carousel timing configuration and manages logo visibility and ordering. The Owl Carousel initialization is refactored to reduce animation timeouts from 9000ms to 5000ms with immediate advancement on load. Three new logo entries are added, three existing logos are toggled hidden, and logo weights are adjusted for sorting.

Changes

Cohort / File(s) Summary
Carousel Timing
assets/js/plugins.js
Refactored Owl Carousel initialization to store jQuery element in variable. Reduced autoplayTimeout and autoplaySpeed from 9000ms to 5000ms. Set smartSpeed to 5000ms. Added immediate next.owl.carousel trigger to bypass initial autoplay delay.
New Logo Entries
content/logos/cursor.md, content/logos/marketspark.md, content/logos/metacto.md
Added three new visible logo entries with metadata including title, image path, external link, and weight for sorting.
Logo Visibility Changes
content/logos/makr.md, content/logos/watermark.md, content/logos/we_the_collective.md
Toggled visibility flag from true to false for three existing logo entries, effectively hiding them from display.
Logo Ordering
content/logos/formation-bio.md, content/logos/thinaer.md
Adjusted weight values: Formation Bio reduced from 10 to 3, Thinaer increased from 3 to 99, affecting carousel sorting/priority.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested reviewers

  • gberenice
  • oycyc
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title 'feat: update client logo carousel' accurately summarizes the main change—updating the client logos carousel through new additions, visibility changes, reordering, and autoplay fixes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/update-client-logos

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@oycyc oycyc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

trunk fmt needed it looks like

@Gowiem Gowiem requested a review from a team as a code owner March 26, 2026 16:46
@Gowiem Gowiem enabled auto-merge (squash) March 26, 2026 16:49
@Gowiem Gowiem merged commit 7fa35d2 into master Mar 26, 2026
7 of 8 checks passed
@Gowiem Gowiem deleted the feat/update-client-logos branch March 26, 2026 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants