text icon indicating copy to clipboard operation
text copied to clipboard

fix(FloatingButtons): Add missing `aria-label` to NcButton components

Open mejo- opened this issue 1 month ago • 3 comments

Fixes: #8031

🏁 Checklist

  • [x] Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • [x] Sign-off message is added to all commits

mejo- avatar Dec 10 '25 15:12 mejo-

/backport to stable32

mejo- avatar Dec 10 '25 15:12 mejo-

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests.

:loudspeaker: Thoughts on this report? Let us know!

codecov[bot] avatar Dec 10 '25 15:12 codecov[bot]

I tried to reproduce the problem in main branch but could not see the warnings so could not test this.

max-nextcloud avatar Dec 12 '25 09:12 max-nextcloud

@max-nextcloud the funny thing is I did not see this in the text app but in the tables app which seems to have a text integration?

susnux avatar Dec 12 '25 20:12 susnux

(anyways missing aria labels when there is no visual text are always a reason to fail WCAG so good that this is fixed now :) )

susnux avatar Dec 12 '25 20:12 susnux

/backport to stable32

mejo- avatar Dec 13 '25 15:12 mejo-