Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AXCustomContent Support #471

Merged
merged 3 commits into from
Jan 18, 2024
Merged

AXCustomContent Support #471

merged 3 commits into from
Jan 18, 2024

Conversation

RoyalPineapple
Copy link
Collaborator

@RoyalPineapple RoyalPineapple commented Nov 28, 2023

This change allows AccessibilityElement to provide arbitrary strings to assistive technologies using the AXCustomContent protocol.

@@ -3,62 +3,8 @@ import UIKit

public struct AccessibilityElement: Element {

/// Used to provide additional functionality to assistive technologies beyond your accessible UI.
public struct CustomAction: Equatable, Hashable {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Moving nested types to the bottom of the file.

@RoyalPineapple RoyalPineapple marked this pull request as ready for review December 5, 2023 17:42
@RoyalPineapple RoyalPineapple requested a review from a team as a code owner December 5, 2023 17:42

extension AccessibilityElement {
public struct CustomContent {
public enum Importance: Equatable { case regular, high }
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you document this a bit? Also, maybe split onto multiple lines like regular enums

Copy link
Collaborator

@kyleve kyleve left a comment

Choose a reason for hiding this comment

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

Still LGTM

@RoyalPineapple RoyalPineapple merged commit cd8c0eb into main Jan 18, 2024
5 checks passed
@RoyalPineapple RoyalPineapple deleted the alex/AXCustomContent branch January 18, 2024 23:08
kyleve added a commit that referenced this pull request Mar 16, 2024
…ller

* origin/main: (62 commits)
  AccessibilityBlocker aggressively blocking. (#483)
  Bumping version to 3.0.0 (#482)
  Allow for customization of the preview name (#478)
  Update CHANGELOG for AttributedLabel fixes (#480)
  Fix link detection for stretched labels (#476)
  chore: Updated minimum deployment target from iOS 14 to iOS 15 [UI-5185] (#479)
  chore(ios): Bump to Xcode 15.1 and Ruby 3.2.2 [UI-5184] (#477)
  AXCustomContent Support (#471)
  Bumping versions to 2.2.0 (#470)
  Update concatenation logic and unit tests
  update changelog
  never cache subelements
  optionally do not cache subelements
  Feature: add TintAdjustmentMode, modifiers, and tests
  Add to CHANGELOG
  Add tintAdjustmentMode to Image
  Bumping versions to 2.1.0 (#466)
  Resolved a Swift 5.9 compilation warning (#465)
  Update KeyboardObserver (#463)
  Bump activesupport from 7.0.4.3 to 7.0.7.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.

None yet

2 participants