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

Open manga folder via context menu #347

Closed
yagamiyakumo opened this issue Jan 24, 2024 · 6 comments
Closed

Open manga folder via context menu #347

yagamiyakumo opened this issue Jan 24, 2024 · 6 comments
Labels
bug Something isn't working

Comments

@yagamiyakumo
Copy link

yagamiyakumo commented Jan 24, 2024

Type of installation

Portable v2.19.1 | 32-bit (.zip)

Type of reader (if reader related)

None

Steps to reproduce

  1. Right click manga folder
  2. Select "Open in Yomikiru"

✔️ Expected Behavior

Load the entire manga folder with all the zip files within

❌ Actual Behavior

Error popup window:

No images found
Folder doesn't contain any supported image format.

@yagamiyakumo yagamiyakumo added the bug Something isn't working label Jan 24, 2024
@github-project-automation github-project-automation bot moved this to For Review in yomikiru todo Jan 24, 2024
Copy link

Thank you for your feedback and suggestions. Please note that the author might only respond on weekends, so we kindly ask for your patience and understanding.

@yagamiyakumo
Copy link
Author

Going into the folder and open any of the zip files via context menu work correctly as expected behavior (load all chapters zip files)

@mienaiyami
Copy link
Owner

Load the entire manga folder with all the zip files within

Unzipping is resource heavy process, and if you are unzipping this many files then more so. You could say that it should open the first zip file then.
Because of how context menu option works for whole app, its not a great idea to load the entire manga. Context menu option will only load anything as a "Chapter".
Note that if you load a chapter, other chapters in same folder will became available in side-list automatically.

@mienaiyami mienaiyami closed this as not planned Won't fix, can't repro, duplicate, stale Jan 27, 2024
@mienaiyami mienaiyami moved this from For Review to Closed in yomikiru todo Jan 27, 2024
@yagamiyakumo
Copy link
Author

yagamiyakumo commented Jan 28, 2024

Load the entire manga folder with all the zip files within

Unzipping is resource heavy process, and if you are unzipping this many files then more so. You could say that it should open the first zip file then. Because of how context menu option works for whole app, its not a great idea to load the entire manga. Context menu option will only load anything as a "Chapter". Note that if you load a chapter, other chapters in same folder will became available in side-list automatically.

Wouldn't opening the folder and loading/extracting only the first/last chapter in the list and the rest of the chapter showing up in the side-list be idea then? I'm almost certain that's how it used to work in an earlier version until the recent update or two.

I'm not requesting for all chapters to be extracted in case there's any misunderstanding. I just want to be able to right click a manga folder and launch it in Yomikiru with either first/last chapter extracted. Kind of like how it would behave if I go inside the manga folder and open any of the chapters (zip file) via context menu, with the rest of the chapters in the manga showing up on the side-list, only extracting that particular chapter if I select it

@mienaiyami
Copy link
Owner

Wouldn't opening the folder and loading/extracting only the first/last chapter in the list and the rest of the chapter showing up in the side-list be idea then?

It's not hard to make it work, but the "Open in Yomikiru" option is meant to open a chapter. Imagine the case of manga folder with chapter "folders", here I need to check if any folder have "images" in it. And if there was no image found, it will have to go over all folders. It will get out of hand very quicky, in code as well.

If you are telling to make this feature just for archive files, then again it creates an anti-pattern.

I hope you will understand, adding new features is not hard, but just adding anything creates lots of problem. When I had just started working on Yomikiru I did this mistake too often, and now I'm facing many issues trying to maintain and adding important features.

So for now, the "Open in Yomikiru" option is only opening chapters. Might add different option for manga folder later when major issues are fixed.

@yagamiyakumo
Copy link
Author

I see. Thank you for your explanation. Here's hoping for the day open for manga folder gets added as well!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Closed
Development

No branches or pull requests

2 participants