Skip to content

Fix: paused project modal mobile upgrade action#3024

Merged
HarshMN2345 merged 2 commits intomainfrom
fix-paused-project-modal-mobile-upgrade-link
May 5, 2026
Merged

Fix: paused project modal mobile upgrade action#3024
HarshMN2345 merged 2 commits intomainfrom
fix-paused-project-modal-mobile-upgrade-link

Conversation

@HarshMN2345
Copy link
Copy Markdown
Member

What does this PR do?

before:

image

after:

image

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 5, 2026

Greptile Summary

  • Adds mobile-responsive behaviour to the paused project modal: on small viewports (≤767px), the Upgrade button is hidden and replaced with an inline Link.Anchor inside the descriptive text, pointing to the change-plan page via a $derived upgradeHref.
  • handleUpgrade (used by the desktop button) still independently calls resolve(...) rather than reusing the already-computed upgradeHref; the two stay in sync only by convention.
  • No functional regressions introduced; the SSR initial value of isSmallViewport is false (desktop branch), which matches the server render and avoids hydration mismatches.

Confidence Score: 5/5

Safe to merge — the change is a targeted UI fix with no logic regressions.

Only P2-level observations remain (the handleUpgrade/upgradeHref duplication was already flagged in a previous review). No P0 or P1 issues found.

No files require special attention.

Important Files Changed

Filename Overview
src/routes/(console)/project-[region]-[project]/pausedProjectModal.svelte Adds mobile-responsive behaviour: hides the Upgrade button on small viewports and replaces it with an inline Link.Anchor in the text. Uses $derived for upgradeHref and imports isSmallViewport store correctly.

Reviews (2): Last reviewed commit: "Fix paused project modal upgrade href re..." | Re-trigger Greptile

Comment thread src/routes/(console)/project-[region]-[project]/pausedProjectModal.svelte Outdated
@HarshMN2345 HarshMN2345 merged commit 53d3c55 into main May 5, 2026
3 of 4 checks passed
@HarshMN2345 HarshMN2345 deleted the fix-paused-project-modal-mobile-upgrade-link branch May 5, 2026 13:29
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