Skip to content

fix auto scroll bug and mobile responsiveness#965

Merged
graphite-app[bot] merged 1 commit into
mainfrom
fix/responsive
May 18, 2026
Merged

fix auto scroll bug and mobile responsiveness#965
graphite-app[bot] merged 1 commit into
mainfrom
fix/responsive

Conversation

@ved015

@ved015 ved015 commented May 18, 2026

Copy link
Copy Markdown
Member

This PR fixes a bug in auto scroll after the first msg auto scroll was not working properly it is fixed now

It also tightens Support & Help on small screens: slightly reduced card/title spacing and stacked-button gaps, and replaces fixed-height pills with responsive vertical padding so label text is not flush with the pill edges on mobile devices

Before

Screenshot 2026-05-18 at 6 56 48 PM

After

Screenshot 2026-05-18 at 6 57 04 PM

@cloudflare-workers-and-pages

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

Copy link
Copy Markdown

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 39f30f1 May 18 2026, 04:38 PM

@cloudflare-workers-and-pages

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

Copy link
Copy Markdown

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 39f30f1 Commit Preview URL

Branch Preview URL
May 18 2026, 04:39 PM

@graphite-app graphite-app Bot requested a review from Dhravya May 18, 2026 13:27
@claude

claude Bot commented May 18, 2026

Copy link
Copy Markdown

Claude finished @ved015's task in 1m 23s —— View job


PR Review: fix auto scroll bug and mobile responsiveness

  • Get PR diff and analyze changes
  • Review auto scroll fix in chat/index.tsx
  • Review mobile responsiveness changes in support.tsx
  • Submit review summary

Overview: This PR fixes auto-scroll behavior in chat by tracking when the user just sent a message, and improves mobile responsiveness for the Support page with responsive padding/spacing.

Issues found: None — this looks good to ship.

Analysis

Auto scroll fix (chat/index.tsx):
The fix introduces a userJustSentRef that tracks when the user has just sent a message. The auto-scroll logic now checks isScrolledToBottomRef.current || userJustSentRef.current, ensuring auto-scroll continues even if the scroll position check happens during the transition between states. The ref is properly reset when:

  • Status changes from streaming/submitted to ready
  • User manually scrolls away from the bottom

This is a well-designed fix — the previous implementation could fail if the user's scroll position was momentarily not at bottom during the state transitions after sending a message.

Mobile responsiveness (support.tsx):
Clean CSS-only changes using sm: breakpoints:

  • Reduced padding on mobile for cards and titles
  • Replaced fixed h-11 on buttons with responsive py-3 sm:py-2.5 vertical padding
  • Tighter gaps between stacked buttons on mobile

No logic changes, just improved layout on smaller screens.

Score: 10/10

@ved015 ved015 requested a review from MaheshtheDev May 18, 2026 13:31
@graphite-app

graphite-app Bot commented May 18, 2026

Copy link
Copy Markdown

Merge activity

This PR fixes a bug in auto scroll after the first msg auto scroll was not working properly it is fixed now

 It also tightens Support & Help on small screens: slightly reduced card/title spacing and stacked-button gaps, and replaces fixed-height pills with responsive vertical padding so label text is not flush with the pill edges on mobile devices

## Before
<img width="382" height="232" alt="Screenshot 2026-05-18 at 6 56 48 PM" src="https://github.com/user-attachments/assets/62c583a2-2417-44b6-8502-d6a7950eb136" />

## After
<img width="396" height="267" alt="Screenshot 2026-05-18 at 6 57 04 PM" src="https://github.com/user-attachments/assets/919dbf50-664a-44ec-aa95-287ca13fd77a" />
@graphite-app graphite-app Bot merged commit 39f30f1 into main May 18, 2026
6 of 7 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