search

multiSnow / mcomix3

End of Fork
Other
96 stars 39 forks source link

Missing all delete options #115

Closed schneckenschnabel closed 3 years ago

schneckenschnabel commented 4 years ago

Neither in the normal view or in the library view is there any option for deleting an archive. Where those simply removed (when yes, oh god why?) or is there simply a problem on my end?

Tried the fedora 32 build, and the git build from here, all the same.

multiSnow commented 4 years ago

Since https://github.com/multiSnow/mcomix3/commit/bcd3f3dbc41ca5cd6ea5b4894851179719e1e481, mcomix itself does not provide a way to delete something from disk. Please do it through a file manager or some command like rm at your own risk.

wyatt8740 commented 4 years ago

Boooo.

I used this feature a lot, and I don't want to give up an 'open with' option for the hotkey.

There was even a nice warning to make sure you wanted to delete something. I really don't know why this was removed completely. The way I see it, this is preventing feature-parity with the 2.x versions.

While it admittedly may be confusing for many cases, there are also use cases (like mine; I don't use archives or dual pages) where it makes perfect sense. I do understand the rationale, I just do not like it.

brunoais commented 3 years ago

https://github.com/multiSnow/mcomix3/pull/137 Push for more commands, then. For now, I aimed for 20. I don't know how to safely add more without clutter