Skip to content

chore: replace play icon with sync icon on connections list#983

Merged
ved015 merged 3 commits into
mainfrom
vorflux/sync-icon-replace-play
May 21, 2026
Merged

chore: replace play icon with sync icon on connections list#983
ved015 merged 3 commits into
mainfrom
vorflux/sync-icon-replace-play

Conversation

@vorflux
Copy link
Copy Markdown
Contributor

@vorflux vorflux Bot commented May 21, 2026

Summary

Two fixes in this PR:

1. Replace play icon with sync icon on connections list

Replaces the Play icon with RefreshCw (sync/rotate arrows) on the manual sync button in the connections list. This makes the button's intent clearer — it triggers a sync, not playback.

File changed: apps/web/components/settings/connections-mcp.tsx

2. Fix login page background — top area showing grainy texture

The two gradient divs on the login page both started at top-[20%], leaving the top 20% uncovered and showing just the grainy bg-rectangle.png overlay against the bare page background. Fixed by:

  • Moving gradients to top-0 and anchoring them at bg-bottom so they fill upward
  • Adding bg-[#030912] as a dark base on the wrapper

File changed: apps/web/app/(auth)/login/new/page.tsx

Testing

  • Icon change: purely a visual swap, no logic changes — verified import updated from Play to RefreshCw
  • Background fix: no JS logic changes; CSS-only fix verified by reviewing the Cloudflare preview deployment

Session Details

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 21, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
supermemory-mcp 1ed6467 May 21 2026, 11:16 AM

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 21, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
supermemory-app 1ed6467 Commit Preview URL

Branch Preview URL
May 21 2026, 11:17 AM

@ved015 ved015 marked this pull request as ready for review May 21, 2026 11:07
@graphite-app graphite-app Bot requested a review from Dhravya May 21, 2026 11:07
@claude
Copy link
Copy Markdown

claude Bot commented May 21, 2026

Claude finished @ved015's task in 42s —— View job


Review Complete

  • Analyze PR context
  • Get and review the diff
  • Check for bugs, security issues, and logic errors
  • Submit review

Summary: This PR looks good to ship. Both changes are purely visual — a simple icon swap from Play to RefreshCw, and a CSS fix to ensure the login page gradients cover the full viewport (changed top-[20%] to top-0 and bg-top to bg-bottom). No logic changes, no security concerns, no bugs found.

Score: 10/10

Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Overview: Replaces the Play icon with a RefreshCw (sync) icon on the connections list and fixes the login page background coverage by anchoring gradients to the top edge.

Issues found: None — this looks good to ship.

Both changes are purely visual:

  • Icon swap is a straightforward import/component change with no logic impact
  • CSS fix correctly addresses the gap by setting top-0, changing bg-top to bg-bottom, and adding a dark base color

Score: 10/10

@ved015
Copy link
Copy Markdown
Member

ved015 commented May 21, 2026

Checked on local both changes looks fine

@ved015 ved015 enabled auto-merge (squash) May 21, 2026 11:23
@ved015 ved015 disabled auto-merge May 21, 2026 11:59
@ved015 ved015 merged commit bdf23d2 into main May 21, 2026
7 of 8 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.

2 participants