Skip to content

Conversation

@jfversluis
Copy link
Member

@jfversluis jfversluis commented Feb 6, 2025

Description of Change

Resolve TODOs in code to obsolete the iOS compatibility AccessibilityExtensions.

Issues Fixed

Contributes to #8989

@jfversluis jfversluis added platform/ios area-controls-general General issues that span multiple controls, or common base classes such as View or Element t/a11y Relates to accessibility labels Feb 6, 2025
@jfversluis jfversluis added this to the .NET 10.0-preview2 milestone Feb 6, 2025
@jfversluis jfversluis requested a review from a team as a code owner February 6, 2025 07:34
@jfversluis jfversluis requested review from mattleibow and removed request for a team February 6, 2025 07:34
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

src/Controls/src/Core/Compatibility/iOS/Extensions/AccessibilityExtensions.cs:94

  • [nitpick] The parameter name '_defaultAccessibilityLabel' is not consistent with naming conventions. It should be renamed to 'defaultAccessibilityLabel'.
public static string SetAccessibilityLabel(this UIBarItem Control, Element Element, string _defaultAccessibilityLabel = null)

@jfversluis jfversluis merged commit 8a3cfce into net10.0 Feb 7, 2025
20 checks passed
@jfversluis jfversluis deleted the obsolete-compat-a11yext branch February 7, 2025 07:27
@github-actions github-actions bot locked and limited conversation to collaborators Mar 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-controls-general General issues that span multiple controls, or common base classes such as View or Element platform/ios t/a11y Relates to accessibility

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants