Skip to content

Updated Styling of Switch to Match Visual Design Spec#2832

Merged
aangerhofer merged 9 commits into
mainfrom
ADarrowSwitchStyling
Feb 5, 2026
Merged

Updated Styling of Switch to Match Visual Design Spec#2832
aangerhofer merged 9 commits into
mainfrom
ADarrowSwitchStyling

Conversation

@aangerhofer

@aangerhofer aangerhofer commented Jan 27, 2026

Copy link
Copy Markdown
Contributor

Pull Request

🤨 Rationale

The switch didn't look like our figma file. We are going to use it in the AI chat so I wanted to update it.

👩‍💻 Implementation

Looked at Brandon's styling & made the colors match

🧪 Testing

Ran a linter.

✅ Checklist

  • I have updated the project documentation to reflect my changes or determined no changes are needed. purely visual changes, no function

Comment thread packages/nimble-components/src/switch/styles.ts Outdated
Comment thread packages/nimble-components/src/switch/styles.ts
Comment thread packages/nimble-components/src/switch/styles.ts Outdated
Comment thread packages/nimble-components/src/switch/styles.ts Outdated
Comment thread packages/nimble-components/src/switch/styles.ts Outdated
Comment thread packages/nimble-components/src/switch/styles.ts Outdated

@jattasNI jattasNI 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.

You don't need to get Milan's review for this PR, my review is sufficient.

Once you've addressed / rejected my last comments and pushed any final changes you'll need to approve the visual diffs. Assuming you can access them, you can do this yourself but let me know if you need help with it.

Comment thread packages/nimble-components/src/switch/styles.ts
Comment thread packages/nimble-components/src/switch/styles.ts Outdated
@aangerhofer aangerhofer enabled auto-merge (squash) February 3, 2026 19:39
@aangerhofer aangerhofer merged commit fb39fd3 into main Feb 5, 2026
9 checks passed
@aangerhofer aangerhofer deleted the ADarrowSwitchStyling branch February 5, 2026 07:04
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.

3 participants