Skip to content

Adjust spacing around 'Add Router' button.#13184

Merged
szgupta merged 2 commits into
masterfrom
suraj/adjust-model-router-ui
Jul 1, 2026
Merged

Adjust spacing around 'Add Router' button.#13184
szgupta merged 2 commits into
masterfrom
suraj/adjust-model-router-ui

Conversation

@szgupta

@szgupta szgupta commented Jun 29, 2026

Copy link
Copy Markdown
Member

Before:
Screenshot 2026-06-29 at 1 08 33 PM

After:
Screenshot 2026-06-29 at 1 07 04 PM

@szgupta szgupta requested a review from coolcom200 June 29, 2026 17:08
@cla-bot cla-bot Bot added the cla-signed label Jun 29, 2026
@oz-for-oss

oz-for-oss Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

@szgupta

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@szgupta szgupta enabled auto-merge (squash) June 29, 2026 17:09

@oz-for-oss oz-for-oss Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Overview

This PR adjusts the layout around the Custom Routers header and Add Router button in the AI settings page. The change is limited to spacing/alignment in app/src/settings_view/ai_page.rs, and the PR description includes before/after screenshots for the user-facing update.

Concerns

  • No blocking correctness, security, or spec-alignment concerns found in the annotated diff.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@szgupta szgupta merged commit 887c458 into master Jul 1, 2026
39 of 41 checks passed
@szgupta szgupta deleted the suraj/adjust-model-router-ui branch July 1, 2026 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants