Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bugfix: keys/hotkeys: Revert minor changes to improve documentation. #941

Merged
merged 1 commit into from
Mar 4, 2021

Conversation

neiljp
Copy link
Collaborator

@neiljp neiljp commented Mar 4, 2021

A few changes in fb9f20b had no apparent issue with UI navigation, but
unintended consequences with regard to triggering custom actions. This
commit therefore reverts a subset of that commit to allow those to
proceed as before.

In particular, scrolling beyond the current list of messages was not
possible with the renamed keys, as the generated key was 'down' but the
listed entry was 'Down'. Similarly with other directions, page up/down,
and End.

docs/hotkeys.md regenerated.

A few changes in fb9f20b had no apparent issue with UI navigation, but
unintended consequences with regard to triggering custom actions. This
commit therefore reverts a subset of that commit to allow those to
proceed as before.

In particular, scrolling beyond the current list of messages was not
possible with the renamed keys, as the generated key was 'down' but the
listed entry was 'Down'. Similarly with other directions, page up/down,
and End.

docs/hotkeys.md regenerated.
@neiljp neiljp added bug Something isn't working PR soon to be merged PR has been reviewed & is ready to be merged labels Mar 4, 2021
@neiljp neiljp added this to the Next Release milestone Mar 4, 2021
@zulipbot zulipbot added the size: M [Automatic label added by zulipbot] label Mar 4, 2021
@neiljp neiljp merged commit e846e39 into zulip:main Mar 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working PR soon to be merged PR has been reviewed & is ready to be merged size: M [Automatic label added by zulipbot]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants