From b2d534da610c8ef6d295d6d700ad84aecbe643a4 Mon Sep 17 00:00:00 2001 From: Michael Terry Date: Thu, 16 Nov 2017 16:33:50 -0500 Subject: [PATCH] Show current tab even in high contrast mode In the dashboard header and the course navigation bar, the current tab is supposed to be the only one with a bottom border. But in high contrast mode in Windows, all of them were assigned a bottom border color. So instead of toggling the color when active, toggle the border style, since Windows overrides our color choice anyway. LEARNER-3219 --- lms/static/sass/_header.scss | 4 ++-- lms/static/sass/course/layout/_courseware_header.scss | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/lms/static/sass/_header.scss b/lms/static/sass/_header.scss index d48d926a55..b6e77fce1f 100644 --- a/lms/static/sass/_header.scss +++ b/lms/static/sass/_header.scss @@ -102,12 +102,12 @@ font-weight: $font-weight-normal; display: inline-block; margin-bottom: -1*$baseline/2; - border-bottom: 4px solid transparent; + border-bottom: 4px hidden theme-color("dark"); cursor: pointer; &.active, &:hover { - border-bottom-color: theme-color("dark"); + border-bottom-style: solid; } &:hover { diff --git a/lms/static/sass/course/layout/_courseware_header.scss b/lms/static/sass/course/layout/_courseware_header.scss index 5f446ab492..2a28fa2f15 100644 --- a/lms/static/sass/course/layout/_courseware_header.scss +++ b/lms/static/sass/course/layout/_courseware_header.scss @@ -52,13 +52,13 @@ display: block; text-align: center; text-decoration: none; - border-bottom: 4px solid transparent; + border-bottom: 4px hidden theme-color("primary"); &:hover, &:focus, &.active { color: theme-color("primary"); - border-bottom-color: theme-color("primary"); + border-bottom-style: solid; background-color: transparent; } }