Skip to content

feat(web): redesign login with split layout and context network panel#1010

Merged
graphite-app[bot] merged 1 commit into
mainfrom
MaheshtheDev/redesign-login-split-context-panel
May 27, 2026
Merged

feat(web): redesign login with split layout and context network panel#1010
graphite-app[bot] merged 1 commit into
mainfrom
MaheshtheDev/redesign-login-split-context-panel

Conversation

@MaheshtheDev
Copy link
Copy Markdown
Member

Consolidate login routes, add animated tool context visualization on the hero panel, and polish loading states across onboarding, settings, and workspace bootstrap.

Copy link
Copy Markdown
Member Author


How to use the Graphite Merge Queue

Add the label Main to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@claude
Copy link
Copy Markdown

claude Bot commented May 26, 2026

Claude finished @MaheshtheDev's task in 1m 39s —— View job


Review Complete

  • Review the latest changes (commits: a35c162, c2b6ddd)
  • Check for bugs, security issues, and logic errors
  • Submit review summary

Overview: This PR consolidates login routes (/login/new/login), adds an animated tool context visualization panel on the login hero, and polishes loading states across onboarding, settings, and workspace bootstrap.

Issues found: None — this looks good to ship.

The changes are well-structured:

  • Route consolidation properly preserves query params when redirecting from /login/new to /login
  • EnsureWorkspace loading state logic correctly handles all edge cases (session restoring, no session, orgs loading, empty orgs redirecting to onboarding)
  • Social auth handlers correctly only reset isLoading in the .catch() block, since successful OAuth initiates a redirect (no .finally() that would run during redirect)
  • LoginToolsPanel animation logic is clean with proper timeout cleanup in useEffect
  • Tooltip additions on disabled buttons improve UX by explaining why actions are blocked
  • Latest commits (a35c162, c2b6ddd) polish the desktop split layout and mobile responsive styling

Note: Socket flagged a critical CVE in next@16.0.0 but this is in packages/tools/test/chatapp/package.json (a test app), not the main web application. Consider updating that dependency separately.

Score: 10/10

@socket-security
Copy link
Copy Markdown

socket-security Bot commented May 26, 2026

@socket-security
Copy link
Copy Markdown

socket-security Bot commented May 26, 2026

All alerts resolved. Learn more about Socket for GitHub.

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

View full report

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: This PR consolidates login routes (/login/new/login), adds an animated tool context visualization panel on the login hero, and polishes loading states across onboarding, settings, and workspace bootstrap.

Issues found: None — this looks good to ship.

The changes are well-structured:

  • Route consolidation properly preserves query params when redirecting from /login/new to /login
  • EnsureWorkspace loading state logic correctly handles all edge cases (session restoring, no session, orgs loading, empty orgs)
  • Social auth handlers correctly only reset isLoading in the .catch() block, since successful OAuth initiates a redirect
  • LoginToolsPanel animation logic is clean with proper cleanup in useEffect
  • Tooltip additions on disabled buttons improve UX by explaining why actions are blocked

Score: 10/10

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

cloudflare-workers-and-pages Bot commented May 26, 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 31d416b Commit Preview URL

Branch Preview URL
May 27 2026, 01:01 AM

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

cloudflare-workers-and-pages Bot commented May 26, 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 31d416b May 27 2026, 01:00 AM

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: This PR consolidates login routes (/login/new/login), adds an animated tool context visualization panel on the login hero, and polishes loading states across onboarding, settings, and workspace bootstrap.

Issues found: None — this looks good to ship.

The changes are well-structured:

  • Route consolidation properly preserves query params when redirecting from /login/new to /login
  • EnsureWorkspace loading state logic correctly handles all edge cases (session restoring, no session, orgs loading, empty orgs)
  • Social auth handlers correctly only reset isLoading in the .catch() block, since successful OAuth initiates a redirect
  • LoginToolsPanel animation logic is clean with proper cleanup in useEffect
  • Tooltip additions on disabled buttons improve UX by explaining why actions are blocked

Note: Socket flagged a critical CVE in next@16.0.0 but this is in packages/tools/test/chatapp/package.json (a test app), not the main web application. Consider updating that dependency separately.

Score: 10/10

Prasanna721
Prasanna721 approved these changes May 27, 2026
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented May 27, 2026

Merge activity

…#1010)

Consolidate login routes, add animated tool context visualization on the hero panel, and polish loading states across onboarding, settings, and workspace bootstrap.
@graphite-app graphite-app Bot force-pushed the MaheshtheDev/redesign-login-split-context-panel branch from 5335791 to 31d416b Compare May 27, 2026 00:59
@graphite-app graphite-app Bot merged commit 31d416b into main May 27, 2026
6 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