Skip to content

feat(ui): Add inline volume split sliders#273

Merged
prajjwalkumar17 merged 4 commits intomainfrom
feat/volume-split-inline-sliders
Apr 29, 2026
Merged

feat(ui): Add inline volume split sliders#273
prajjwalkumar17 merged 4 commits intomainfrom
feat/volume-split-inline-sliders

Conversation

@bot4pk
Copy link
Copy Markdown
Collaborator

@bot4pk bot4pk commented Apr 29, 2026

Summary

  • Move volume split allocation sliders into the per-gateway Split % row instead of rendering a separate allocation panel.
  • Keep the inferred last gateway behavior visible inline with an Auto badge.
  • Add the missing useRef import in the Euclid rules page so the frontend build from current main succeeds.
  • Regenerate committed dashboard assets from npm run build.

Impact

This keeps gateway name, gateway ID, numeric split, and slider adjustment in one row, matching the requested side-by-side editing flow and reducing duplicate slider UI.

Validation

  • npm run build
  • curl http://localhost:8080/health
  • Local dashboard started at http://localhost:5174/ because port 5173 was already occupied.

@prajjwalkumar17 prajjwalkumar17 changed the title [codex] Add inline volume split sliders feat(ui): Add inline volume split sliders Apr 29, 2026
@prajjwalkumar17 prajjwalkumar17 marked this pull request as ready for review April 29, 2026 12:57
Copilot AI review requested due to automatic review settings April 29, 2026 12:57
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR refactors the dashboard UI to support more inline editing flows, notably moving volume split allocation sliders into the per-gateway row and updating supporting UI/asset changes.

Changes:

  • Inline volume split allocation sliders into the per-gateway “Split %” row (and remove the separate allocation panel).
  • Update Analytics custom time-range UI to use an inline popover that can be dismissed via outside-click / Escape.
  • Fix a missing React useRef import and regenerate committed website/dist build assets.

Reviewed changes

Copilot reviewed 4 out of 14 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
website/src/components/pages/VolumeSplitPage.tsx Moves allocation sliders inline into each gateway row; responsive layout adjustments.
website/src/components/pages/AnalyticsPage.tsx Adds a dismissible popover for custom time range selection.
website/src/components/pages/PaymentAuditPage.tsx Updates JSON inspector <pre> styling for improved readability.
website/src/components/pages/EuclidRulesPage.tsx Adds missing useRef import to fix build.
website/dist/index.html Updates hashed asset references after rebuild.
website/dist/decision-engine/index.html Updates hashed asset references after rebuild.
website/dist/assets/index-cq4Fupgx.css Removes old hashed CSS asset from build output.
website/dist/decision-engine/assets/index-cq4Fupgx.css Removes old hashed CSS asset from build output.
Comments suppressed due to low confidence (1)

website/src/components/pages/VolumeSplitPage.tsx:291

  • The gateway name input has no associated label or aria-label. Since the column headers are hidden on small screens, this field (and the gateway ID field below) ends up without an accessible name and with unclear context on mobile. Add an explicit label (visible on mobile) or at least an aria-label tied to the row’s gateway label.
                <input
                  value={g.gatewayName}
                  onChange={e => updateGateway(g.id, 'gatewayName', e.target.value)}
                  placeholder="e.g. stripe"
                  className="border border-slate-200 dark:border-[#222226] bg-transparent rounded-lg px-3 py-1.5 text-sm focus:outline-none focus:ring-1 focus:ring-brand-500"
                />

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 292 to 297
<input
value={g.gatewayId}
onChange={e => updateGateway(g.id, 'gatewayId', e.target.value)}
placeholder="optional gateway_id"
className="border border-slate-200 dark:border-[#222226] bg-transparent rounded-lg px-3 py-1.5 text-sm focus:outline-none focus:ring-1 focus:ring-brand-500"
/>
@prajjwalkumar17 prajjwalkumar17 merged commit 493510f into main Apr 29, 2026
13 checks passed
@prajjwalkumar17 prajjwalkumar17 deleted the feat/volume-split-inline-sliders branch April 29, 2026 13:03
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.

4 participants