Skip to content

Make the (lazy) action menu spinner aria label translatable and reduce it's size to small - #369

Merged
HDinger merged 1 commit into
mainfrom
code-maintenance/68046-action-menu-refinements
Oct 9, 2025
Merged

Make the (lazy) action menu spinner aria label translatable and reduce it's size to small#369
HDinger merged 1 commit into
mainfrom
code-maintenance/68046-action-menu-refinements

Conversation

@akabiru

@akabiru akabiru commented Oct 9, 2025

Copy link
Copy Markdown
Member

What are you trying to accomplish?

Make the (lazy) action menu spinner aria label translatable and reduce it's size to small

List the issues that this change affects.

https://community.openproject.org/wp/68046

Risk Assessment

  • Low risk the change is small, highly observable, and easily rolled back.
  • Medium risk changes that are isolated, reduced in scope or could impact few users. The change will not impact library availability.
  • High risk changes are those that could impact customers and SLOs, low or no test coverage, low observability, or slow to rollback.

What approach did you choose and why?

Anything you want to highlight for special attention from reviewers?

Accessibility

  • Fixes axe scan violation - This change fixes an existing axe scan violation.
  • No new axe scan violation - This change does not introduce any new axe scan violations.
  • New axe violation - This change introduces a new axe scan violation. Please describe why the violation cannot be resolved below.

Merge checklist

  • Added/updated tests
  • Added/updated documentation
  • Added/updated previews (Lookbook)
  • Tested in Chrome
  • Tested in Firefox
  • Tested in Safari
  • Tested in Edge

…e it's size to small

https://community.openproject.org/wp/68046

Co-authored-by: Henriette Darge <h.darge@openproject.com>
@changeset-bot

changeset-bot Bot commented Oct 9, 2025

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: a202e63

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@openproject/primer-view-components Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@HDinger HDinger left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

👏

@HDinger
HDinger merged commit af264e8 into main Oct 9, 2025
27 of 28 checks passed
@openprojectci openprojectci mentioned this pull request Oct 9, 2025
@akabiru
akabiru deleted the code-maintenance/68046-action-menu-refinements branch October 9, 2025 13:40
@akabiru

akabiru commented Oct 9, 2025

Copy link
Copy Markdown
Member Author

fwiw @HDinger the reported failing test is passing locally- seems we have some flakiness?

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

Development

Successfully merging this pull request may close these issues.

2 participants