This repository has been archived by the owner on Feb 20, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 272
Make the h key behave like a real back button #689
Labels
Comments
goggle
added a commit
to goggle/rtv
that referenced
this issue
May 18, 2019
This PR implements a proper return key as proposed in issue michael-lazar#689. As a consequence of that, the former (IMO rather complicated) approach of using the methods `loop` and `handle_selected_page` is replaced by using a simple page stack to keep track of the currently selected page. This page stack also defines a maximal size: If the page stack is full and an additional page is added to the page stack, the **oldest** entry in the page stack will be removed, so that the page stack does not exceed the maximal size. For this new approach, the tests had to be adjusted. Here is a summary: Removed tests (no longer needed or replaced by the newly added tests): - test_inbox_close - test_submission_exit - test_subreddit_handle_selected_page - test_subreddit_page_loop_pre_select - test_subreddit_page_loop - test_subscription_close Modified tests: - test_inbox_view_content - test_inbox_open_submission - test_submission_propmpt - test_subreddit_prompt - test_subreddit_open - test_subreddit_post - test_subreddit_open_subscriptions - test_subreddit_open_multireddits - test_subreddit_frontpage_toggle - test_subscription_prompt - test_subscription_close - test_page_unauthenticated Added tests: - test_page_stack - test_page_back_button
I see that you've got several pull requests open here that won't get pulled in. I have forked RTV with the intention of maintaining it under the new name 'TUIR' in the Gitlab repository here. Please feel free to open your pull requests there, I'd love to have these changes in the program. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Is your feature request related to a problem? Please describe.
Currently the behavior of the back key
h
is quite restricted. It can only go back to the previous page once.Describe the solution you'd like
Make the key
h
behave like a real back button, i.e. it should the user allow to go back multiple times instead of just once.Describe alternatives you've considered
The alternative is to keep the current behavior which is very limited.
The text was updated successfully, but these errors were encountered: