Skip to content

Fix missing blue gradient (Nova tint) on mobile and adjust position#911

Open
ved015 wants to merge 1 commit intomainfrom
fix/responsive
Open

Fix missing blue gradient (Nova tint) on mobile and adjust position#911
ved015 wants to merge 1 commit intomainfrom
fix/responsive

Conversation

@ved015
Copy link
Copy Markdown
Member

@ved015 ved015 commented May 8, 2026

The blue gradient background (Nova tint) was hidden on mobile devices because showNovaBackdrop required !isMobile. This made the app appear as plain black on phones.

Changes

  • Show gradient on mobile: showNovaBackdrop now includes all view modes (graph, list, dashboard) regardless of screen size
  • Move gradient higher: Adjusted pctNarrow from 70% to 55% so the blue tint appears in the middle-upper area on mobile instead of near the bottom
  • Cleanup: Removed unused isMemoriesDesktop and isHomeDesktop variables

Before

Screenshot 2026-05-08 at 9 40 25 PM

After

Screenshot 2026-05-08 at 9 39 08 PM

@cloudflare-workers-and-pages
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 6aa6568 May 08 2026, 03:51 PM

@cloudflare-workers-and-pages
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 6aa6568 Commit Preview URL

Branch Preview URL
May 08 2026, 03:52 PM

@claude
Copy link
Copy Markdown

claude Bot commented May 8, 2026

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


Review Complete

  • Fetch PR diff and existing review comments
  • Analyze changes for bugs, security issues, and logic errors
  • Submit review

Summary: Reviewed the changes to enable the Nova gradient backdrop on mobile. The logic is clean and correct — no bugs, security issues, or breaking changes found. This PR is good to ship.

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: Enables the Nova gradient backdrop on mobile devices and repositions it higher on narrow screens.

Issues found: None — this looks good to ship.

The logic change is straightforward and correct:

  • showNovaBackdrop now triggers for graph, list, and dashboard views regardless of screen size
  • The gradient position adjustment from 70% to 55% moves the visual effect to a better position on mobile
  • Removed variables (isMemoriesDesktop, isHomeDesktop) were only used for the old showNovaBackdrop computation, so cleanup is appropriate

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