Updating case of shortcut on Discussion view fixes VICE-577
Test plan - Pull down change for testing - Open a course with announcement or discussions - Open an announcement or discussion - Tab through page to see Shortcut announcement with ALT + F8 shortcut - Confirm hidden text says ALT + F8 and not Shift + ? Change-Id: I919226707dddad23a6be262580f1140da25419be Reviewed-on: https://gerrit.instructure.com/c/canvas-lms/+/241560 Tested-by: Service Cloud Jenkins <svc.cloudjenkins@instructure.com> Reviewed-by: Drake Harper <drake.harper@instructure.com> QA-Review: Drake Harper <drake.harper@instructure.com> Product-Review: Drake Harper <drake.harper@instructure.com>
This commit is contained in:
parent
871ffb44e7
commit
88e7d48170
|
@ -42,7 +42,7 @@ describe(KeyboardShortcutModal, () => {
|
|||
component.unmount()
|
||||
})
|
||||
|
||||
test('appears when alt + f8 is pressed', async () => {
|
||||
test('appears when ALT + F8 is pressed', async () => {
|
||||
expect(document.querySelector('.keyboard_navigation')).toBeNull()
|
||||
const e = new Event('keydown')
|
||||
e.which = 119
|
||||
|
|
|
@ -64,7 +64,7 @@
|
|||
<div class="form-inline" style="overflow: visible;">
|
||||
<div id="keyboard-shortcut-modal-info" tabindex="0">
|
||||
<span class="accessibility-warning">
|
||||
<%= t 'Press alt + F8 to see a list of keyboard shortcuts' %>
|
||||
<%= t 'Press ALT + F8 to see a list of keyboard shortcuts' %>
|
||||
</span>
|
||||
</div>
|
||||
<div class="pull-right">
|
||||
|
|
|
@ -52,7 +52,7 @@ QUnit.module('DiscussionTopicKeyboardShortcutModal#render')
|
|||
test('renders shortcuts', async function() {
|
||||
const wrapper = mount(<DiscussionTopicKeyboardShortcutModal />)
|
||||
|
||||
// open the modal by pressing "alt + f8"
|
||||
// open the modal by pressing "ALT + f8"
|
||||
const e = new Event('keydown')
|
||||
e.which = 119
|
||||
e.altKey = true
|
||||
|
|
Loading…
Reference in New Issue