Skip to content

Conversation

@michaelmkraus
Copy link
Contributor

@michaelmkraus michaelmkraus commented Aug 28, 2025

Proposed changes

  • enable in/valid state for switch
  • add accent styling
  • configurable label position
  • remove emphasis property from switch

Types of changes

  • Bugfix (non-breaking change that fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Refactoring (improvements to existing components or architectural decisions)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices apply)

Further comments

closes https://github.com/db-ux-design-system/core-team/issues/851

🔭🐙🐈 Test this branch here: https://db-ux-design-system.github.io/core-web/review/851-switch-stable

michaelmkraus and others added 19 commits May 23, 2025 11:18
…olor

# Conflicts:
#	__snapshots__/textarea/showcase/chromium/DBTextarea-should-match-screenshot-1/DBTextarea-should-match-screenshot.png
#	__snapshots__/textarea/showcase/firefox/DBTextarea-should-match-screenshot-1/DBTextarea-should-match-screenshot.png
#	__snapshots__/textarea/showcase/mobile-chrome/DBTextarea-should-match-screenshot-1/DBTextarea-should-match-screenshot.png
…olor

# Conflicts:
#	__snapshots__/textarea/showcase/mobile-chrome/DBTextarea-should-match-screenshot-1/DBTextarea-should-match-screenshot.png
@changeset-bot
Copy link

changeset-bot bot commented Aug 28, 2025

🦋 Changeset detected

Latest commit: c199799

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 9 packages
Name Type
@db-ux/core-components Major
@db-ux/ngx-core-components Major
@db-ux/react-core-components Major
@db-ux/v-core-components Major
@db-ux/wc-core-components Major
@db-ux/core-foundations Major
@db-ux/core-stylelint Major
@db-ux/core-migration Major
@db-ux/agent-cli Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added 📺showcases Changes to 1-n showcases 🏘components labels Aug 28, 2025
@github-actions
Copy link
Contributor

🔭🐙🐈 Test this branch here: https://db-ux-design-system.github.io/core-web/review/851-switch-stable

@michaelmkraus michaelmkraus self-assigned this Aug 28, 2025
@michaelmkraus michaelmkraus marked this pull request as draft August 28, 2025 11:30
@michaelmkraus michaelmkraus moved this to 🕋 In progress in UX Engineering Team Backlog Aug 28, 2025
@nmerget nmerget marked this pull request as ready for review October 13, 2025 10:29
@nmerget nmerget enabled auto-merge (squash) October 13, 2025 10:29
@nmerget nmerget moved this from 🏗 In progress to 🎁 Ready for review in UX Engineering Team Backlog Oct 13, 2025
mfranzke and others added 14 commits October 13, 2025 12:47
Removed the `emphasis` property from the switch component and updated migration documentation.
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
[showMessage]="exampleProps?.showMessage"
[label]="exampleProps?.label"
>{{
exampleProps?.label ? exampleProps?.label : exampleName
Copy link
Collaborator

@mfranzke mfranzke Oct 13, 2025

Choose a reason for hiding this comment

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

@nmerget those innerText differ from the other JS frameworks (showcases/vue-showcase/src/components/switch/Switch.vue and showcases/react-showcase/src/components/switch/index.tsx), is this intentionally?

@mfranzke mfranzke disabled auto-merge October 13, 2025 15:01
@mfranzke mfranzke merged commit cb2deb0 into main Oct 13, 2025
8 checks passed
@mfranzke mfranzke deleted the 851-switch-stable branch October 13, 2025 15:01
@github-project-automation github-project-automation bot moved this from 🎁 Ready for review to ✅ Done in UX Engineering Team Backlog Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏘components 📕documentation Improvements or additions to documentation 📺showcases Changes to 1-n showcases

Projects

Status: No status
Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

6 participants