Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(gestor-agendas): agrega aria-label en botoneras #2208

Merged
merged 2 commits into from
Apr 15, 2021
Merged

fix(gestor-agendas): agrega aria-label en botoneras #2208

merged 2 commits into from
Apr 15, 2021

Conversation

JSantarelli
Copy link
Contributor

@JSantarelli JSantarelli commented Apr 12, 2021 •
edited
Loading

Requerimiento

https://proyectos.andes.gob.ar/browse/CIT-109

Contexto

Imposibilidad de percibir con lector de pantalla, determinados elementos de la interfaz del gestor de agendas, especialmente en botones sin texto (acciones iconográficas al seleccionar una agenda).

Necesario para testear

  1. Descargar el lector de pantalla NVDA (sólo windows, sino alguno para Linux: se sugiere ORCA)
  2. Instalar plex v7.15.0 (npm i @andes/plex@latest)

Funcionalidad desarrollada

  1. Se agrega atributo ariaLabel a todos los <plex-button> con atributo 'icon' de modo que al pasar el screen-reader describa la acción que representa el icono en cuestión.

UserStory llegó a completarse

  • Si
  • No
  • No corresponde

Requiere actualizaciones en la base de datos

  • Si
  • No

Requiere actualizaciones en la API

  • Si
  • No

Requiere actualizaciones en andes-test-integracion

  • Si
  • No

@JSantarelli JSantarelli requested review from a team as code owners April 12, 2021 02:25
@sychus
Copy link

sychus commented Apr 13, 2021

@JSantarelli copadisimo!! una sola duda ¿cómo se sete el orden de navegación para cada control?. Es decir, más allá que el programa ahora lea estos labels como el usuario podría saber el orden de los controles para la navegación de los mismos?

@liquid36
Copy link
Contributor

liquid36 commented Apr 13, 2021 •
edited
Loading

@JSantarelli copadisimo!! una sola duda ¿cómo se sete el orden de navegación para cada control?. Es decir, más allá que el programa ahora lea estos labels como el usuario podría saber el orden de los controles para la navegación de los mismos?

De izquierda a derecha, de arriba para abajo. Salvo que alguien setee un tabindex.

@liquid36
Copy link
Contributor

USUARIO: mbotta
BUILD NUMBER: 3756
CYPRESS RUN: 2822
TEST START: 2021-04-13T17:09:38.418Z
TOTAL: 363
SUCCESS: 363
FAIL: 0
SKIPPED: 0

@liquid36 liquid36 added the test ok Los test estan ok label Apr 13, 2021
@JSantarelli
Copy link
Contributor Author

@JSantarelli copadisimo!! una sola duda ¿cómo se sete el orden de navegación para cada control?. Es decir, más allá que el programa ahora lea estos labels como el usuario podría saber el orden de los controles para la navegación de los mismos?

De izquierda a derecha, de arriba para abajo. Salvo que alguien setee un tabindex.

Buena preguna @sychus. Exactamente @liquid36, por eso en teoría, una pantalla bien jerarquizada debería poder prescindir de los tabindex. Cabe aclarar que en el caso del gestor de agendas, el usuario salta del listado (situado debajo) a la botonera (en la parte superior), por lo que al tabear en retroceso (shift + tab), la botonera se recorre de der. a izq.

@liquid36 liquid36 merged commit 1fb92a1 into master Apr 15, 2021
@liquid36 liquid36 deleted the CIT-109 branch April 15, 2021 16:16
liquid36 pushed a commit that referenced this pull request Apr 21, 2021
@semantic-release-bot
chore(release): 5.43.0 [skip ci]
82f61b9 
# [5.43.0](v5.42.0...v5.43.0) (2021-04-21)

### Bug Fixes

* **EP:** nombre tipo de ficha en buscador ([0040d54](0040d54))
* **gestor-agendas:** agrega aria-label en botoneras ([#2208](#2208)) ([1fb92a1](1fb92a1))
* **gestor-usuarios:** ajustes de perfiles ([#2218](#2218)) ([10a0c35](10a0c35))
* **inscripcion:** cambia formato de fecha ([#2212](#2212)) ([01f253d](01f253d))
* **mpi:**  en formulario contactos cambia patrón email ([#2227](#2227)) ([c6c6c49](c6c6c49))

### Features

* **BI:** biquery ficha epidemio ([c56c404](c56c404))
* **COM:** crea derivación como prestación ([05857da](05857da))
* **epidemiologia:** buscador de ficha epidemiologica ([1974b9c](1974b9c))
* **ficha-epdemiologica:** actualizacion ([6337034](6337034))
* **ficha-epidemilogica:** Actualizacion nuevos campos ficha covid ([6f331c2](6f331c2))
* **ficha-epidemio:** clear dependencias ([5d962fd](5d962fd))
* **ficha-epidemiologica:** buscador de fichas ([099fd26](099fd26))
* **ficha-epidemiologica:** modifica localidades ([843069e](843069e))
* **HUDS:** visualiza fichas epidemiologicas ([f22261f](f22261f))
* **vac:** agregar nota en inscripcion ([#2222](#2222)) ([237f9ce](237f9ce))
* **vac:** filtro por certificado ([#2213](#2213)) ([b3575eb](b3575eb))
* **vac:** poder editar datos basicos si notiene paciente asociado ([#2216](#2216)) ([8edd80f](8edd80f))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@palita1991 palita1991 palita1991 approved these changes

@liquid36 liquid36 liquid36 approved these changes

@andrrrl andrrrl Awaiting requested review from andrrrl

@martinebucarey martinebucarey Awaiting requested review from martinebucarey

Assignees
No one assigned
Labels
Aprobado test ok Los test estan ok
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@JSantarelli @sychus @liquid36 @palita1991