From e789c554c73b2fc5e54fcbfbc20cc31010769aab Mon Sep 17 00:00:00 2001 From: Chris Rodriguez Date: Tue, 24 Mar 2015 14:56:06 -0400 Subject: [PATCH] Updating blue color variables for accessibility in the LMS --- lms/static/sass/base/_variables.scss | 8 ++++---- lms/static/sass/course/_info.scss | 5 +++++ lms/static/sass/course/wiki/_wiki.scss | 1 + lms/static/sass/shared/_footer.scss | 3 +-- 4 files changed, 11 insertions(+), 6 deletions(-) diff --git a/lms/static/sass/base/_variables.scss b/lms/static/sass/base/_variables.scss index f29b1921c1..6928590944 100644 --- a/lms/static/sass/base/_variables.scss +++ b/lms/static/sass/base/_variables.scss @@ -139,7 +139,7 @@ $facebook-blue: #3B5998; $linkedin-blue: #0077B5; // TODO: both blue and yellow variables differ from CMS rgb value, need to confirm change to CMS variable is ok in current platform uses before switching. -$blue: rgb(29,157,217); +$blue: rgb(0, 120, 176); $yellow: rgb(255, 252, 221); // COLORS: old variables @@ -183,7 +183,7 @@ $m-blue-l4: tint($m-blue,90%); $m-blue-l5: tint($m-blue,95%); $m-blue-l6: #4bb4fb; $m-blue-d1: rgb(23,144,199); // #1790C7 -$m-blue-d2: rgb(21,128,176); // #1580B0 +$m-blue-d2: $blue; $m-blue-d3: rgb(18,111,154); // #126F9A $m-blue-d4: rgb(10,74,103); // #0A4A67 $m-blue-d5: rgb(0,158,231); // #009EE7 @@ -252,7 +252,7 @@ $courseware-footer-margin: 0px; $courseware-border-bottom-color: rgb(68,162,222); // #44a2de $courseware-button-border-color: rgb(230,230,230); // #e6e6e6 $courseware-hover-color: rgb(51,52,53); // #333435 -$courseware-navigation-color: rgb(0,158,231); // #009ee7 +$courseware-navigation-color: $blue; // ==================== @@ -386,7 +386,7 @@ $border-color-l4: $m-gray-l4; // MISC: links and buttons $link-color: $blue; -$link-color-d1: $m-blue-d2; +$link-color-d1: $blue; $link-hover: $pink; $site-status-color: $pink; $button-color: $blue; diff --git a/lms/static/sass/course/_info.scss b/lms/static/sass/course/_info.scss index e16dc78ef5..70bae30ad6 100644 --- a/lms/static/sass/course/_info.scss +++ b/lms/static/sass/course/_info.scss @@ -84,6 +84,10 @@ div.info-wrapper { box-shadow: none; font-size: 14px; + a { + color: $link-color; + } + &:after { left: -1px; right: auto; @@ -104,6 +108,7 @@ div.info-wrapper { a { display: block; padding: 0; + color: $link-color; &:hover, &:focus { background: transparent; diff --git a/lms/static/sass/course/wiki/_wiki.scss b/lms/static/sass/course/wiki/_wiki.scss index ded79da90d..22347d7346 100644 --- a/lms/static/sass/course/wiki/_wiki.scss +++ b/lms/static/sass/course/wiki/_wiki.scss @@ -49,6 +49,7 @@ section.wiki { height: 100%; text-overflow: ellipsis; white-space: nowrap; + color: $link-color; } &:after { diff --git a/lms/static/sass/shared/_footer.scss b/lms/static/sass/shared/_footer.scss index e40c21f1f1..89dac2cafa 100644 --- a/lms/static/sass/shared/_footer.scss +++ b/lms/static/sass/shared/_footer.scss @@ -35,7 +35,6 @@ &:hover, &:focus, &:active { border-bottom: 1px dotted $link-color; - color: $link-color; } } @@ -208,7 +207,7 @@ // edX theme: LMS Footer // ==================== $edx-footer-spacing: ($baseline*0.75); -$edx-footer-link-color: rgb(0, 158, 231); +$edx-footer-link-color: $link-color; $edx-footer-bg-color: rgb(252,252,252); %edx-footer-reset {