Skip to content

fix(homepage): fix trusted by best-in-class apps carousel animation - #1249

Merged
dsanders11 merged 1 commit into
electron:mainfrom
daxter-army:main
Oct 22, 2026
Merged

dsanders11 merged 1 commit into
electron:mainfrom
daxter-army:main

Conversation

@daxter-army

@daxter-army daxter-army commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

Description of Change

The Trusted by best-in-class apps carousel, which displays brand logos, had an issue with infinite scrolling. Fixed the CSS issue, and the carousel now loops seamlessly.

Video Confirmation

before_carousel_fix.mov
after_carousel_fix.mov

Checklist

@daxter-army
daxter-army requested a review from a team as a code owner October 18, 2026 04:08
@daxter-army daxter-army changed the title fix(marquee-animation): fix trusted by best-in-class apps carousel animation fix(brands-carousel): fix trusted by best-in-class apps carousel animation Oct 18, 2026
@daxter-army daxter-army changed the title fix(brands-carousel): fix trusted by best-in-class apps carousel animation fix(homepage): fix trusted by best-in-class apps carousel animation Oct 18, 2026

@dsanders11 dsanders11 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks!

@dsanders11
dsanders11 enabled auto-merge (squash) October 21, 2026 22:00
@dsanders11

Copy link
Copy Markdown
Member

@daxter-army, we require signed commits, so we're unable to merge this PR until you update it to have those.

auto-merge was automatically disabled October 22, 2026 03:38

Head branch was pushed to by a user without write access

@daxter-army

Copy link
Copy Markdown
Contributor Author

Hi @dsanders11, thank you for reviewing my PR!
I've updated the PR with verified commits. Could you please take another look?

@dsanders11
dsanders11 merged commit 1fe95bd into electron:main Oct 22, 2026
6 checks passed
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.

3 participants