Skip to content

Conversation

yordan-st
Copy link
Contributor

@yordan-st yordan-st commented Sep 1, 2025

Pull request type

Bug fix (non-breaking change which fixes an issue)


Description

  • Added a visually hidden native to the Switch component for accessibility and label support.
  • Ensured the Mendix-rendered label with for attribute toggles the switch as expected.

Related branch: mendix/atlas#269

What should be covered while testing?

  • If clicking the label toggles the Switch

@yordan-st yordan-st requested a review from a team as a code owner September 1, 2025 13:22
@yordan-st yordan-st force-pushed the feat/WC-2552_switch-toggle-label branch from 4b1064f to 7d720a0 Compare September 3, 2025 15:03
r0b1n
r0b1n previously approved these changes Sep 3, 2025
@gjulivan gjulivan force-pushed the feat/WC-2552_switch-toggle-label branch from 7d720a0 to 6bdccea Compare September 4, 2025 10:07
@gjulivan
Copy link
Collaborator

gjulivan commented Sep 4, 2025

add changelog and increase minor version

@yordan-st yordan-st force-pushed the feat/WC-2552_switch-toggle-label branch 2 times, most recently from 3c586da to eb330d0 Compare September 8, 2025 09:56
r0b1n
r0b1n previously approved these changes Sep 9, 2025
@yordan-st yordan-st force-pushed the feat/WC-2552_switch-toggle-label branch from eb330d0 to 2273fc9 Compare September 10, 2025 10:08
@yordan-st yordan-st force-pushed the feat/WC-2552_switch-toggle-label branch 2 times, most recently from 0961878 to 4586379 Compare September 17, 2025 15:16
@yordan-st yordan-st force-pushed the feat/WC-2552_switch-toggle-label branch from 4586379 to 9239cdc Compare September 17, 2025 15:29
@yordan-st yordan-st force-pushed the feat/WC-2552_switch-toggle-label branch from 9239cdc to af94963 Compare September 18, 2025 11:52
@gjulivan gjulivan merged commit 932724b into main Sep 19, 2025
14 checks passed
@gjulivan gjulivan deleted the feat/WC-2552_switch-toggle-label branch September 19, 2025 07:30
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.

4 participants