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

Linemode option documentation fix #2979

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DanilaBerezin
Copy link

ISSUE TYPE

  • Improvement/feature implementation

RUNTIME ENVIRONMENT

  • Operating system and version: Ubuntu 22.04 LTS
  • Terminal emulator and version: Gnome Shell 42.9
  • Python version: 3.10.12
  • Ranger version/commit: 38bb890
  • Locale: en_US

CHECKLIST

  • The CONTRIBUTING document has been read [REQUIRED]
  • All changes follow the code style [REQUIRED]
  • All new and existing tests pass [REQUIRED]
  • Changes require config files to be updated
    • Config files have been updated
  • Changes require documentation to be updated
    • Documentation has been updated
  • Changes require tests to be updated
    • Tests have been updated

DESCRIPTION

Fixed inconsistency in ranger documentation where it was stated that commanding 'linemode humanreadablesizemtime' changed the linemode to display human readable modification time and file size, but the correct command for this is 'linemode sizehumanreadablemtime'

MOTIVATION AND CONTEXT

Found the issue when I tried to follow documentation to set the default line mode. Realized the issue and figured I should make this change since it makes the documentation accurate to what the software actually does and will likely help future documentation readers in the future save some time.

TESTING

Testing broke for the same reason disclosed here

IMAGES / VIDEOS

…commanding 'linemode humanreadablesizemtime' changed the linemode to display human readable modification time and file size, but the correct command for this is 'linemode sizehumanreadablemtime'
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant