Skip to content
This repository was archived by the owner on Mar 4, 2020. It is now read-only.

chore(Button): Adding aria-disabled when button has loading state#1789

Merged
kolaps33 merged 4 commits intomasterfrom
mituron-loader-button-acc
Aug 13, 2019
Merged

chore(Button): Adding aria-disabled when button has loading state#1789
kolaps33 merged 4 commits intomasterfrom
mituron-loader-button-acc

Conversation

@kolaps33
Copy link
Collaborator

Based on the feedback from the users, we should set button like disabled, during button is in progress/unavailable state...

@codecov
Copy link

codecov bot commented Aug 13, 2019

Codecov Report

Merging #1789 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1789      +/-   ##
==========================================
+ Coverage   69.75%   69.76%   +<.01%     
==========================================
  Files         868      868              
  Lines        7483     7484       +1     
  Branches     2175     2176       +1     
==========================================
+ Hits         5220     5221       +1     
  Misses       2255     2255              
  Partials        8        8
Impacted Files Coverage Δ
...b/accessibility/Behaviors/Button/buttonBehavior.ts 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 25f8353...ec7be94. Read the comment docs.

@kolaps33 kolaps33 merged commit f3485b5 into master Aug 13, 2019
@delete-merged-branch delete-merged-branch bot deleted the mituron-loader-button-acc branch August 13, 2019 14:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants