pharmaR / riskassessment

Collaborative Deployment: https://app.pharmar.org/riskassessment/ Risk Assessment Demo App: https://rinpharma.shinyapps.io/riskassessment
https://pharmar.github.io/riskassessment/
Other
101 stars 28 forks source link

Create new, light color-blind friendly palette #670

Closed AARON-CLARK closed 11 months ago

AARON-CLARK commented 11 months ago

Seeks to close #654

codecov[bot] commented 11 months ago

Codecov Report

Merging #670 (866a454) into dev (d94a024) will increase coverage by 0.01%. Report is 1 commits behind head on dev. The diff coverage is 87.50%.

:exclamation: Current head 866a454 differs from pull request most recent head 32fdd93. Consider uploading reports for the commit 32fdd93 to get more accurate results

@@            Coverage Diff             @@
##              dev     #670      +/-   ##
==========================================
+ Coverage   72.96%   72.97%   +0.01%     
==========================================
  Files          33       33              
  Lines        4834     4829       -5     
==========================================
- Hits         3527     3524       -3     
+ Misses       1307     1305       -2     
Files Coverage Δ
R/app_ui.R 100.00% <100.00%> (ø)
R/mod_databaseView.R 89.77% <100.00%> (-0.12%) :arrow_down:
R/mod_uploadPackage.R 76.75% <100.00%> (-0.12%) :arrow_down:
R/utils_build_cards.R 68.10% <100.00%> (ø)
R/utils_config_db.R 67.76% <100.00%> (ø)
R/utils_startup.R 63.95% <100.00%> (ø)
R/mod_decision_automation.R 78.17% <80.00%> (+0.50%) :arrow_up:
R/mod_packageDependencies.R 18.85% <0.00%> (+0.07%) :arrow_up:

... and 1 file with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Jeff-Thompson12 commented 11 months ago

Database table error should be fixed with my last commit. I will look at the other items.

Jeff-Thompson12 commented 11 months ago

I think it's the bold font. Changing the color to grey doesn't look quite right. Here's what it looks like without the bold: image

Jeff-Thompson12 commented 11 months ago

@AARON-CLARK ready for another look