From cefafce5bdef0a5eb3c51fd1c605177e5779341c Mon Sep 17 00:00:00 2001
From: ehaines <ehaines@mitre.org>
Date: Sun, 24 Nov 2024 05:10:00 -0500
Subject: [PATCH] changed report colors to conform to  WCAG 2 AA thresholds

---
 .../Modules/CreateReport/CreateReport.psm1    |   2 +-
 .../Modules/CreateReport/scripts/main.js      |   1 +
 .../Modules/CreateReport/styles/main.css      |  21 ++++++++++++++++--
 .../IndividualReports/AADReport.html          | Bin 97766 -> 97214 bytes
 4 files changed, 21 insertions(+), 3 deletions(-)

diff --git a/PowerShell/ScubaGear/Modules/CreateReport/CreateReport.psm1 b/PowerShell/ScubaGear/Modules/CreateReport/CreateReport.psm1
index 06dd3d2e7f..7e1eda07f4 100644
--- a/PowerShell/ScubaGear/Modules/CreateReport/CreateReport.psm1
+++ b/PowerShell/ScubaGear/Modules/CreateReport/CreateReport.psm1
@@ -226,7 +226,7 @@ function New-Report {
     # Finish building the html report
     $Title = "$($FullName) Baseline Report"
     $AADWarning = "Exclusions must only be used if they are approved within an organization's security risk acceptance process.
-    Please reference <a href=`"$($ScubaGitHubUrl)/blob/v$($SettingsExport.module_version)/docs/configuration/configuration.md#entra-id-configuration`" target=`"_blank`">this section in the README file</a>
+    Please reference <a href=`"$($ScubaGitHubUrl)/blob/main/docs/configuration/configuration.md#entra-id-configuration`" target=`"_blank`" class=`" aad-link`">this section in the README file</a>
     file for a list of the policies that accept exclusions and the instructions for setting up exclusions in the configuration file.
     <i>Exclusions can introduce grave risks to your system and must be managed carefully.</i>"
     $NoWarning = "<br/>"
diff --git a/PowerShell/ScubaGear/Modules/CreateReport/scripts/main.js b/PowerShell/ScubaGear/Modules/CreateReport/scripts/main.js
index e3131e24dd..ac917c360f 100644
--- a/PowerShell/ScubaGear/Modules/CreateReport/scripts/main.js
+++ b/PowerShell/ScubaGear/Modules/CreateReport/scripts/main.js
@@ -18,6 +18,7 @@ const colorRows = () => {
             }
             else if (rows[i].children[statusCol].innerHTML === "Fail") {
                 rows[i].style.background = "var(--test-fail)";
+                rows[i].style.color = "var(--test-error-color)";
             }
             else if (rows[i].children[statusCol].innerHTML === "Warning") {
                 rows[i].style.background = "var(--test-warning)";
diff --git a/PowerShell/ScubaGear/Modules/CreateReport/styles/main.css b/PowerShell/ScubaGear/Modules/CreateReport/styles/main.css
index 5821d16ea7..440af37c49 100644
--- a/PowerShell/ScubaGear/Modules/CreateReport/styles/main.css
+++ b/PowerShell/ScubaGear/Modules/CreateReport/styles/main.css
@@ -38,7 +38,7 @@ html[data-theme='dark'] {
     --note-color: #ee4e04;
     --header-bottom: rgb(221, 221, 221);
     --link-color: #85B065;
-    --unvisited-link-color: #0092CC;
+    --unvisited-link-color: #E6F598;
     --text-color: #bdbdbd;
     --uuid-color: #999999;
     --border-color: #7b7b7b;
@@ -76,6 +76,23 @@ h4 {
     margin-bottom: 0.313em;
 }
 
+h4 .aad-link {
+    font-family: Arial, Helvetica, sans-serif;
+    color: var(--unvisited-link-color);
+    text-decoration: underline;
+}
+
+h4 .aad-link:visited {
+    font-family: Arial, Helvetica, sans-serif;
+    color: var(--link-color);
+}
+
+h4 .aad-link:hover {
+    font-family: Arial, Helvetica, sans-serif;
+    color: var(--link-color);
+    text-decoration: none;
+}
+
 
 .links {
     display: flex;
@@ -232,7 +249,7 @@ th.conditions {
 }
 
 #toggle-text {
-    color: #404040;
+    color: var(--text-color);
     margin: 0.313em 3.125em;
     font-family: Arial, Helvetica, sans-serif;
     font-size: 0.875em;
diff --git a/PowerShell/ScubaGear/Sample-Reports/IndividualReports/AADReport.html b/PowerShell/ScubaGear/Sample-Reports/IndividualReports/AADReport.html
index f15449705aca428a1f64dacac70d77a47da8126e..ce0f3ab8b8613ce5f2af5059d30308fb93647273 100644
GIT binary patch
delta 1681
zcmaF%n|0rJ)(PJxuQ3+d%*EJdG<lM-&GZs(Mxn_Ci#aC0GFF;=;jqwT1rr%|E(Qe<
zoE$hqd~!hp%VY&BxycPCPLn4&XiS!C;+T9WT4QohQ`F?5(5T5qWm=OT`I${N3QL=O
zE{kijQ?|$CfO0t?oi^Dh++wm&h0)|i@p+S#DorL&vhbU{$<A)Fl4Zx_ONn-(P<w!O
z)iUTXVA@(lH5)(KEh56k15pZ-wH(rbwmL!m7%(}&QHf-KIt7(X{^!yJ_Uj?n03vKw
zuo9X)%gqbTX1U3S+%@pnE;4yT3>Tx$WXJ4ujIai|p~5?k1UDRra+oaV8-eBqbl;av
zp5)&Kw@oR+Z}PivqsdAU6X3Dz6tspUTfHVbg!oZx>m&=O$>-{2NV0hk#kP_Y({htj
zl2VAUIcu_CsscQuVR?wO&;{k(u#7wcwh_*`q^BkYD}l-9s`#cy$TLPTD>3Lyp6H-S
zX5o=Gc~Maqv;ZMBk|E9zn!KX~6y)R;UrG^HlN%}w;0c`cqy}+C4Mi?kS7%1C3-~4r
zG(I8Q6*!V0H&QXaIk0_a^L8F?#t)p+|A{b8V$WqrWXNR5n?75V@!|G5F-9)N?Oo!G
J5ot_Z3;?&%P9XpQ

delta 2272
zcmdn@o%Pvo)(PJjc_$w<72eFj*k%M|_8Qv)Nyc~Dll^|MP5x!f1L7?doqXUW+hhe3
z5q@3<E(QfCtY+Yyd@;>*a*2r(NL8Kq<PEt}K-NMxO(6L(HyTJ9R)NTarrJPqV~IJC
z6pTrqY+%C$WKA?n2a+E>K%`@VC6H_kGX|23wfR7DqM7033Dzt?mY|(KNVLu#NG1lk
zPqwiH*|#ysULM&swG288$b6XVxU53Jy6NS*Cw4I;xbBR72Gn&Ad~|>;NZ2Wc6a(2*
z2|JhXVDlki7YJk%7jBx9#oWBWI>jfWx-5UPg}Vkp*Tqgwu;yY0#t0Gd8<>5WEH|1?
z4)CeM>PB3lR}NG?(ae6bgEhzG56LQ%9sFg0BFJfPVvY5r05_1b8tcgo0bwBH(UYS~
z>Et(o79bgDk|aJ@K~7ehoDduV){P#tv~{+f^W=MlAper->`75I1K?^h)351dtHcxv
zoa+MAi<~Z~nFtn><StT6OLB9Q`ecLLEU3$<RgR}m7l~sOnXFa_DzK3Ybn=rT$eDc8
zbrc!7Cd(DaQPsVmEDtIfE|h`FH;fpkApbxdU8BL6#i|6Vrl?q1POK@UiDNGqFg~EV
zV==-MS5b=6&ZuSJ-E7{vvzd{1`+E+?kDQFW)9(v0PM)5?%g8sqMwsy)P$E)<k(&`n
NCyO#hrm=w90RS?}R)YWl