vaadin / expense-manager-demo

Progressive Web App (PWA) demo using Vaadin components
https://vaadin.com/components
Apache License 2.0
360 stars 142 forks source link

Add aria-label="Add expense" for #add-button #92

Closed limonte closed 6 years ago

limonte commented 6 years ago

Fixes #91


This change is Reviewable

manolo commented 6 years ago

:lgtm: needs also to be applied to the pro version


Reviewed 1 of 1 files at r1. Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

limonte commented 6 years ago

Done: https://github.com/vaadin/expense-manager-demo-pro/pull/16


Review status: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from Reviewable

yuriy-fix commented 6 years ago
:lgtm:

Review status: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from Reviewable

yuriy-fix commented 6 years ago

Reviewed 1 of 1 files at r1. Review status: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from Reviewable