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

label, alert, status text updates #1509

Merged
merged 6 commits into from
Mar 12, 2018

Conversation

Blackbaud-ToddRoberts
Copy link
Contributor

@Blackbaud-ToddRoberts Blackbaud-ToddRoberts commented Mar 1, 2018

change icon on Info version of alert
add icons to labels
add info version of status text

#1462

change icon on Info version of alert
add icons to labels
add info version of status text
@codecov-io
Copy link

codecov-io commented Mar 1, 2018

Codecov Report

Merging #1509 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1509   +/-   ##
======================================
  Coverage     100%    100%           
======================================
  Files         387     387           
  Lines        7568    7568           
  Branches      970     970           
======================================
  Hits         7568    7568
Impacted Files Coverage Δ
src/modules/card/card-actions.component.ts 100% <0%> (ø) ⬆️

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 23e4452...38bbd03. Read the comment docs.

@Blackbaud-ToddRoberts Blackbaud-ToddRoberts changed the title label, alert, status text updates [HOLD] label, alert, status text updates Mar 7, 2018
@Blackbaud-ToddRoberts
Copy link
Contributor Author

Holding this pending a review of which icon we use for Info status.

@Blackbaud-ToddRoberts Blackbaud-ToddRoberts changed the title [HOLD] label, alert, status text updates label, alert, status text updates Mar 8, 2018
@Blackbaud-ToddRoberts
Copy link
Contributor Author

Icon is ok as is, this is ready to go

@Blackbaud-SteveBrush Blackbaud-SteveBrush merged commit 6d0fd19 into master Mar 12, 2018
@Blackbaud-SteveBrush Blackbaud-SteveBrush deleted the label-status-text-update branch March 12, 2018 21:18
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.

3 participants