Skip to content
This repository has been archived by the owner on Jan 16, 2023. It is now read-only.

Start from a file #371

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

Naheel-Azawy
Copy link

Like the "-n" option, "-F" start from a file instead of its index.

@Naheel-Azawy
Copy link
Author

Sorry for the window.c thing, just ignore it, looks like you removed the entire full screen thing anyway which is even better!

@wjtk4444
Copy link

I would like to bump this pull request as it's a feature (#179) I find both useful for me and absolutely sane as a default for any image viewer. If You (@muennich) think otherwise, how about making it optional with for example -l (short for load directory).

sxiv is designed for command line users. Please use a wrapper script for that.

Here's why I would like you to reconsider:

  • it does not harm command line users in any way
  • it does expand the amount of use cases, including command line ones
  • I believe that it mosly reuses pieces of code already present and used in sxiv

sxiv-rifle existing and still being mentioned 6 years later is a proof that this is a requested feature, and I see no reason not to have it implemented in sxiv itself. I would compare

Please use a wrapper script for that

to removing sxiv directory -r being a thing, because you could use find directory -type f | sxiv - as a wapper script for that. It would make sxiv more simple, but I don't think it'd make it a better image viewer.

I'm not trying to argue that more features equals better software, rather, I find this one particularly useful for an image viever, same as the ability to open a directory or load it recursively.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants