Skip to content

fix(frontend): move plan page CTAs below pricing cards - #2942

Merged
riderx merged 1 commit into
mainfrom
cursor/move-plans-ctas-below-pricing-f347
Aug 8, 2026
Merged

fix(frontend): move plan page CTAs below pricing cards#2942
riderx merged 1 commit into
mainfrom
cursor/move-plans-ctas-below-pricing-f347

Conversation

@riderx

@riderx riderx commented Aug 8, 2026

Copy link
Copy Markdown
Member

Summary (AI generated)

  • Moved the Credits CTA and Expert-as-a-Service CTA on the organization Plans/pricing page from above the plan cards to below them
  • Plan prices are visible immediately without scrolling past those banners

Motivation (AI generated)

The two CTAs sat above the pricing grid and pushed plan cards down. Users had to scroll just to see prices, which blocked the core decision UI.

Business Impact (AI generated)

Pricing is the conversion surface. Showing plan prices first reduces friction and keeps secondary upsells (credits / expert support) available without hiding the plans.

Test Plan (AI generated)

  • Open /settings/organization/plans on desktop
  • Confirm plan cards/prices appear first without scrolling past CTAs
  • Confirm Credits CTA and Expert CTA still render under the plan grid
  • Confirm monthly/yearly toggle and plan action buttons still work
  • Confirm mobile still hides those CTAs as before

Generated with AI

Open in Web Open in Cursor 

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Review in cubic

Summary by CodeRabbit

  • Style
    • Moved the Credits and Expert as a Service calls to action below the plans grid.
    • Preserved existing styling and visibility behavior while adjusting spacing for the new placement.

Credits and expert-service banners sat above the plan grid and forced users to scroll before seeing prices. Place them under the pricing cards instead.

Co-authored-by: Martin DONADIEU <martindonadieu@gmail.com>
@coderabbitai

coderabbitai Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: fc1a4a26-ea5d-4498-adf4-12cadf84868d

📥 Commits

Reviewing files that changed from the base of the PR and between 6c172f0 and d03ca9d.

📒 Files selected for processing (1)
  • src/pages/settings/organization/Plans.vue

📝 Walkthrough

Walkthrough

The Plans page moves the Credits and Expert as a Service CTAs from above the plans grid to below it. Their conditional behavior and styling remain unchanged.

Changes

Plans CTA layout

Layer / File(s) Summary
Move CTAs below plans
src/pages/settings/organization/Plans.vue
The Credits and Expert as a Service CTAs now render below the plans grid with top-margin spacing. Their existing styling and conditional behavior remain unchanged.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested labels: codex

Suggested reviewers: wcaleniewolny


Comment @coderabbitai help to get the list of available commands.

@riderx
riderx marked this pull request as ready for review August 8, 2026 17:09
@cursor

cursor Bot commented Aug 8, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_0a31fb03-0fc2-4fd6-89f1-5c44fd76b914)

@riderx
riderx merged commit 10f1f4a into main Aug 8, 2026
47 of 53 checks passed
@riderx
riderx deleted the cursor/move-plans-ctas-below-pricing-f347 branch August 8, 2026 17:09
@coderabbitai coderabbitai Bot added the codex label Aug 8, 2026
@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Visual diff passed

Visual changes

Generated at 2026-08-08T17:16:29.815Z. Threshold: 0.1% pixel difference.

Route Diff % Status
login 0.000 unchanged
dashboard 0.681 changed
apps 0.040 unchanged
app-overview 0.105 changed
app-settings 0.000 unchanged
app-settings-access 2.407 changed
channels 0.025 unchanged
devices 0.000 unchanged
observe 0.000 unchanged
observe-logs 0.000 unchanged
observe-native 0.000 unchanged
observe-compatibility 0.000 unchanged
observe-plugins 0.000 unchanged
api-keys-app-preview 2.211 changed

Commit: d03ca9dd98fc446e71baaade948ea9446cb820bc
Download the HTML report from workflow artifacts (artifact: visual-diff-report-d03ca9dd98fc446e71baaade948ea9446cb820bc).

Open index.html from the artifact for side-by-side before/after/diff screenshots.

@codspeed-hq

codspeed-hq Bot commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Merging this PR will not alter performance

✅ 43 untouched benchmarks
⏩ 2 skipped benchmarks1


Comparing cursor/move-plans-ctas-below-pricing-f347 (d03ca9d) with main (6c172f0)

Open in CodSpeed

Footnotes

  1. 2 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@sonarqubecloud

sonarqubecloud Bot commented Aug 8, 2026

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants