Skip to content

Reverting the change to remove accessibility traits on the AttributedLabel#597

Merged
johnnewman-square merged 1 commit intomainfrom
johnnewman/bug/traits-fix
Nov 4, 2025
Merged

Reverting the change to remove accessibility traits on the AttributedLabel#597
johnnewman-square merged 1 commit intomainfrom
johnnewman/bug/traits-fix

Conversation

@johnnewman-square
Copy link
Contributor

@johnnewman-square johnnewman-square commented Nov 3, 2025

This reverts #590. During integration testing, the original change caused KIF failures while searching for labels with the staticText trait. The work to audit the failures and make necessary code changes has been ticketed here.

This revert allows the changes in #596 to be fully tested.

@johnnewman-square johnnewman-square marked this pull request as ready for review November 3, 2025 15:45
@johnnewman-square johnnewman-square requested a review from a team as a code owner November 3, 2025 15:45
@RoyalPineapple
Copy link
Collaborator

Any idea why this was changed in the first place?

@johnnewman-square
Copy link
Contributor Author

Any idea why this was changed in the first place?

I believe the code was originally changed to fix a case where setting accessibilityTraits to nil was not removing the traits from the underlying UILabel. I found this Slack thread that mentions the fix. I've added this info to the ticket.

@johnnewman-square johnnewman-square merged commit b455de6 into main Nov 4, 2025
8 checks passed
@johnnewman-square johnnewman-square deleted the johnnewman/bug/traits-fix branch November 4, 2025 18:08
johnnewman-square added a commit that referenced this pull request Nov 4, 2025
* main:
  Reverting the change to remove accessibility traits on the AttributedLabel (#597)
johnnewman-square added a commit that referenced this pull request Nov 13, 2025
## What's Changed
* Reverting the change to remove accessibility traits on the
AttributedLabel by @johnnewman-square in
#597
* Adding safe area edge detection for the scrollableAxes behavior by
@johnnewman-square in #596


**Full Changelog**:
6.3.1...6.4.0
maxg-square added a commit that referenced this pull request Nov 27, 2025
* main:
  chore: Release 6.5.0 (#600)
  refactor: Migrate Accessibility Infrastructure from Market Design System (#593)
  chore: Release 6.4.0 (#599)
  Adding safe area edge detection for the scrollableAxes behavior (#596)
  Reverting the change to remove accessibility traits on the AttributedLabel (#597)
  Update changelog for 6.3.1 release (#591)
  Fix warnings in Xcode 26 (#589)
  Apply empty accessibility traits to AttributedLabel if supplied traits is nil
  Bump rexml from 3.3.9 to 3.4.2
maxg-square added a commit that referenced this pull request Nov 27, 2025
* maxg/cache_1_equivalency:
  chore: Release 6.5.0 (#600)
  refactor: Migrate Accessibility Infrastructure from Market Design System (#593)
  chore: Release 6.4.0 (#599)
  Adding safe area edge detection for the scrollableAxes behavior (#596)
  Reverting the change to remove accessibility traits on the AttributedLabel (#597)
  Update changelog for 6.3.1 release (#591)
  Fix warnings in Xcode 26 (#589)
  Apply empty accessibility traits to AttributedLabel if supplied traits is nil
  Bump rexml from 3.3.9 to 3.4.2
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.

2 participants