Welcome to Codidact Meta!

Codidact Meta is the "town hall" (meta-discussion site) for the Codidact community network and the Codidact software. Whether you have bug reports or feature requests, support questions or rule discussions that touch the whole network – this is the site for you.

Comments on The keyboard shortcut navigation is flipped

Post

The keyboard shortcut navigation is flipped

−0

The keyboard shortcut j should navigate to the next post (down). Instead it navigates to the previous post (up).

The keyboard shortcut k should navigate to the previous post (up). Instead it navigates to the next post (down).

This is the opposite to the help menu (?):

j Move one item down
k Move one item up

As well the opposite to how Vim uses these two (the shortcuts are based on it).

This bug applies both in question lists as well as Q&As.

bug status-completed keyboard-shortcuts

posted 9 months ago

CC BY-SA 4.0

9mo ago by Monica Cellio‭

VLAZ‭

1 0 4 0

Raw

Markdown

History

1 comment thread

Oops! We're on it. (I am puzzled by how this happened.) (4 comments)

VLAZ‭ wrote 9 months ago

copy link

No idea, but the deployed code seems different than the pre-build version. In the GitHub repo, the code in keyboard_tools.js is correct - defines that j will go to the next item (via .nextAll()) and k would go to the previous (via .prevAll()).

The deployed code is the opposite. I don't know why the discrepancy exists, but seems there was another source which produced the deployed code.

VLAZ‭ wrote 9 months ago · edited 9 months ago

copy link

The deployed code (slightly un-minified):

else if ('k' === t.key) null == _CodidactKeyboard.selectedItem ? _CodidactKeyboard.selectedItem = $('[data-ckb-list-item]:first-of-type') [0] : _CodidactKeyboard.selectedItem = $(_CodidactKeyboard.selectedItem).nextAll('[data-ckb-list-item]') [0] ||
          _CodidactKeyboard.selectedItem,
          _CodidactKeyboard.updateSelected();
else if ('j' === t.key) null == _CodidactKeyboard.selectedItem ? _CodidactKeyboard.selectedItem = $('[data-ckb-list-item]:first-of-type') [0] : _CodidactKeyboard.selectedItem = $(_CodidactKeyboard.selectedItem).prevAll('[data-ckb-list-item]') [0] ||
          _CodidactKeyboard.selectedItem,
          _CodidactKeyboard.updateSelected();

Communities

Comments on The keyboard shortcut navigation is flipped

The keyboard shortcut navigation is flipped

1 comment thread