diff --git a/lms/static/coffee/src/calculator.coffee b/lms/static/coffee/src/calculator.coffee index ce881e08b4..3342aee9dd 100644 --- a/lms/static/coffee/src/calculator.coffee +++ b/lms/static/coffee/src/calculator.coffee @@ -82,6 +82,7 @@ class @Calculator .addClass('shown') .attr('aria-hidden', false) + $(document).on('click', @handleClickOnDocument) hideHint: -> @@ -185,11 +186,14 @@ class @Calculator @hideHint() handleClickOnHintButton: (e) -> + e.preventDefault() e.stopPropagation() if @hintPopup.hasClass 'shown' @hideHint() + @hintButton.attr('aria-expanded', false) else @showHint() + @hintButton.attr('aria-expanded', true) @activeHint.focus() handleClickOnHintPopup: (e) -> diff --git a/lms/static/sass/course/modules/_calculator.scss b/lms/static/sass/course/modules/_calculator.scss index 23f6d807e9..24d6aff666 100644 --- a/lms/static/sass/course/modules/_calculator.scss +++ b/lms/static/sass/course/modules/_calculator.scss @@ -62,7 +62,7 @@ div.calc-main { @include box-sizing(border-box); padding: lh(); - input#calculator_button { + button#calculator_button { background: #111; border: 1px solid $black; border-radius: 0; @@ -131,9 +131,12 @@ div.calc-main { @include hide-text; width: 35px; display: block; + border: none; &:focus { outline: 5px auto #5b9dd9; + box-shadow: none; + border: none; } } diff --git a/lms/templates/calculator/toggle_calculator.html b/lms/templates/calculator/toggle_calculator.html index 7a244e07b0..073e7d2feb 100644 --- a/lms/templates/calculator/toggle_calculator.html +++ b/lms/templates/calculator/toggle_calculator.html @@ -9,15 +9,16 @@