search

michael-lazar / rtv

Browse Reddit from your terminal
MIT License
4.64k stars 273 forks source link

Typo in Help #680

Closed MarkusPettersson98 closed 5 years ago

MarkusPettersson98 commented 5 years ago

Bug

The [Basic Commands] listing on the help page of rtv shows wrong information regarding moving the cursor up and down in rtv. Currently, it says the following: 

 j     : Move the cursor up
 k     : Move the cursor down

This is a clear typo, since rtv actually uses vim bindings for moving the cursor up and down (j for cursor down, k for cursor up). This is even more evident when looking at the following binding

m     : Move the cursor up one page

Which moves the cursor to the top of rtw, or in the same direction as pressing k.

This bug is present in both CONTROLS.md and docs.py.

How to reproduce

  1. Start rtv
  2. Press "?"

Proposed fix

 j     : Move the cursor down
 k     : Move the cursor up
michael-lazar commented 5 years ago

Yup I definitely got them mixed up, nice catch