Skip to content
This repository has been archived by the owner on Dec 8, 2022. It is now read-only.

Added watches for when radio buttons change and the ability to set an aria label #26

Merged
merged 2 commits into from
Mar 19, 2019

Conversation

Blackbaud-TrevorBurch
Copy link
Member

@Blackbaud-TrevorBurch Blackbaud-TrevorBurch commented Mar 14, 2019

#27 and #28 are addressed.

@codecov
Copy link

codecov bot commented Mar 14, 2019

Codecov Report

Merging #26 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #26   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          13     13           
  Lines         515    527   +12     
  Branches       87     87           
=====================================
+ Hits          515    527   +12
Impacted Files Coverage Δ
.../app/public/modules/radio/radio-group.component.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 4d6ac4c...4b652ab. Read the comment docs.

@blackbaud-johnly
Copy link
Contributor

I created blackbaud/skyux2-docs#356 to document this and have the docs set up in blackbaud/skyux2-docs#357.

@Blackbaud-TrevorBurch Blackbaud-TrevorBurch merged commit 1fd2798 into master Mar 19, 2019
@Blackbaud-SteveBrush Blackbaud-SteveBrush deleted the radio-group-improvements branch July 25, 2019 13:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants