Skip to content

fix responsiveness for graph layout#910

Open
ishaanxgupta wants to merge 2 commits intomainfrom
ishaan/responsiveness
Open

fix responsiveness for graph layout#910
ishaanxgupta wants to merge 2 commits intomainfrom
ishaan/responsiveness

Conversation

@ishaanxgupta
Copy link
Copy Markdown
Contributor

@ishaanxgupta ishaanxgupta commented May 8, 2026

fix responsiveness on graph layout and document section in add memory modal

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

cloudflare-workers-and-pages Bot commented May 8, 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 46555bb May 08 2026, 03:37 PM

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

cloudflare-workers-and-pages Bot commented May 8, 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 46555bb Commit Preview URL

Branch Preview URL
May 08 2026, 03:37 PM

@ishaanxgupta ishaanxgupta marked this pull request as ready for review May 8, 2026 16:06
@claude
Copy link
Copy Markdown

claude Bot commented May 8, 2026

Claude finished @ishaanxgupta's task in 1m 3s —— View job


Review Complete

  • Fetch PR diff and existing review comments
  • Analyze changed files for bugs/security issues
  • Submit review

Summary: Reviewed this responsiveness PR and found no bugs, security issues, or logic errors. The changes enable graph view on mobile and add proper responsive styling to the add-memory modal. Clean code — ready to merge.

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: Enables graph view on mobile devices and improves responsiveness for the add-memory modal.

Issues found: None — this looks good to ship.

The changes are straightforward UI/responsiveness improvements:

  • Graph view is now accessible on mobile (removed desktop-only restriction)
  • Add-memory modal has proper mobile layout with fixed header/footer, grid tabs, and usage meters
  • TextEditor gains a debounceMs prop to disable debouncing when needed
  • Share button on graph view is responsive with appropriate mobile sizing

The UsageMeter component correctly clamps percentage values, the close button respects isSubmitting state, and the debounce bypass logic is clean.

Score: 10/10

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