Skip to content

fix: Add accessible name to emojis - EXO-87562 - #5897

Open
akhanfir wants to merge 10 commits into
feature/maintenancefrom
TASK-87562
Open

fix: Add accessible name to emojis - EXO-87562#5897
akhanfir wants to merge 10 commits into
feature/maintenancefrom
TASK-87562

Conversation

@akhanfir

Copy link
Copy Markdown
Contributor

Rendered emojis had no accessible name, making them invisible to screen readers.

@akhanfir
akhanfir enabled auto-merge (squash) July 20, 2026 08:56
@github-actions github-actions Bot added the partialCIBuild Perform Partial CI Build label Jul 20, 2026
@akhanfir
akhanfir requested a review from ahamdi July 20, 2026 09:10

@ahamdi ahamdi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This file is for purifying the content, the emoji names handling should not go here

@exo-swf
exo-swf force-pushed the feature/maintenance branch from 142f5bd to 19924d1 Compare July 20, 2026 23:18
@akhanfir
akhanfir force-pushed the TASK-87562 branch 2 times, most recently from 1de44e0 to e413a44 Compare July 21, 2026 09:27
@exo-swf
exo-swf force-pushed the feature/maintenance branch 2 times, most recently from 27ac8ff to cb9b1a9 Compare July 22, 2026 23:18
@akhanfir
akhanfir force-pushed the TASK-87562 branch 2 times, most recently from e5541eb to ada43a7 Compare July 23, 2026 16:26
@akhanfir
akhanfir requested a review from ahamdi July 23, 2026 16:26
halaya and others added 8 commits July 23, 2026 23:17
…eview in the spaces directory - EXO-87925 (#5885)

The avatar becomes a non-focusable decorative element, and only the name
(the text) remains the clickable/accessible link to the profile.
…n - EXO-88253 (#5887)

The two focusable elements (card + name) are Vuetify v-cards rather than divs, to benefit from the outline: none already integrated into the .v-card class (avoids the browser's default focus outline which was hiding our custom border) — without adding any CSS/style specific to this component.
Before this change, when accessing the space directory and clicking the filter button, the filter field had no aria-label/title, making it inaccessible to screen readers. To fix this problem, add aria-label/title=`Filter by name or description to the filter field` and update its placeholder to `Search for a space`. After this change, the filter field is properly labeled for accessibility
Before this change, when accessing the user directory and clicking the filter button, the filter field had no aria-label/title, making it inaccessible to screen readers. To fix this problem, add aria-label/title=`Filter by name to the filter field` and update its placeholder to `Search for a user`. After this change, the filter field is properly labeled for accessibility.
Before this change, when access to note application and click on the fiter button, the filter field hasn't a label. After this change, add aria-label/title=`Search for content to the search field` and update its placeholder to `Start searching content...`, and add aria-label/title=`Filter by tag name to the tag filter field`.
Before this change, when userA make spaceX notes page as him own home page and USerA user an url that generates page-not-found then userA clicks on Back Home button, /dw page is displayed. To fix this problem, change the default link to home link in the PageNotFound. After this change, the home page is displayed.
…5908)

Before this change, when sending a URL in chat or tasks and the link is displayed correctly, then clicking on the link, the URL opened is one that doesn't exist and a 404 page is displayed. To fix this problem, the encodeURI call is removed; the raw URL, already correctly encoded as returned by match.getUrl(), is used, consistent with Utils.js:toLinkUrl, which already works without this additional encoding. After this change, the link opens the correct URL.
akhanfir added 2 commits July 24, 2026 09:45
Before this change, the filter field showed a custom Vuetify tooltip in addition to the native title attribute already added for accessibility. To fix this, the custom tooltip has been removed from the shared toolbar component, along with its now-unused configuration. After this change, the filter field relies solely on the native tooltip.
Before this change, when userA make spaceX notes page as him own home page and USerA user an url that generates page-not-found then userA clicks on Back Home button, /dw page is displayed. To fix this problem, change the default link to home link in the PageNotFound. After this change, the home page is displayed.
@sonarqubecloud

Copy link
Copy Markdown

@exo-swf
exo-swf force-pushed the feature/maintenance branch 3 times, most recently from 699f324 to 99012c9 Compare July 26, 2026 23:18
@exo-swf
exo-swf force-pushed the feature/maintenance branch 2 times, most recently from 9f5bf25 to 0f6cd16 Compare July 29, 2026 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

partialCIBuild Perform Partial CI Build

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants