Skip to content

Conversation

@ndeshev
Copy link
Contributor

@ndeshev ndeshev commented Nov 25, 2025

add aria-controls and aria-autocomplete attributes to the ui5-search-field
JIRA: BGSOFUIRILA-4136

@ui5-webcomponents-bot
Copy link
Collaborator

ui5-webcomponents-bot commented Dec 5, 2025

🧹 Preview deployment cleaned up: https://pr-12717--ui5-webcomponents.netlify.app

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview December 5, 2025 07:55 Inactive
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview December 5, 2025 08:06 Inactive
@ndeshev ndeshev marked this pull request as ready for review December 5, 2025 08:40
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview December 5, 2025 08:46 Inactive
@ndeshev ndeshev requested a review from ivoplashkov December 5, 2025 09:00
@ivoplashkov
Copy link
Contributor

ivoplashkov commented Dec 5, 2025

aria-label is applied on ui5-button, however it's not propageted properly to the native button, could you please check if there is something incorrect on our side ? Screenshot 2025-12-05 at 11 13 47

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview December 5, 2025 11:26 Inactive
@ndeshev ndeshev requested a review from ivoplashkov December 8, 2025 13:07
remove aria-label to prevent double announcements by screen readers
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview December 9, 2025 14:03 Inactive
@ndeshev ndeshev merged commit e97c302 into main Dec 17, 2025
14 checks passed
@ndeshev ndeshev deleted the search-acc branch December 17, 2025 08:00
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview December 17, 2025 08:00 Inactive
@ui5-webcomponents-bot
Copy link
Collaborator

🎉 This PR is included in version v2.18.0-rc.1 🎉

The release is available on v2.18.0-rc.1

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants