papercups-io/papercups

Accessibility - buttons should have discernible text

Open

#885 aperta il 17 giu 2021

Vedi su GitHub
 (7 commenti) (1 reazione) (0 assegnatari)Elixir (566 fork)batch import
buggood first issue

Metriche repository

Star
 (6032 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Describe the bug The button to toggle the chat widget doesn't have discernible text.

To Reproduce Steps to reproduce the behavior:

  1. Go to a page with the Papercups chat widget installed.
  2. Check accessibility with axe DevTools
  3. See Buttons must have discernible text issue on <button class="Papercups-toggleButton css-1lzvboz">

Expected behavior All buttons should have discernible text, so that screen readers can properly present the page to people with low or no vision.

Additional context Fixing this is basically following one of the solutions in https://dequeuniversity.com/rules/axe/4.2/button-name?application=rocketvalidator - basically adding a aria-label attribute with the text Open chat widget for example.

Guida contributor

Direzione di ricerca
Aggiungi un attributo aria label all'elemento del pulsante di toggle con testo descrittivo come 'Open chat widget'.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
HTMLAccessibility basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90