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

Pages Per Row odd offset sometimes doesn't work #366

Open
Phallss opened this issue Mar 4, 2024 · 0 comments
Open

Pages Per Row odd offset sometimes doesn't work #366

Phallss opened this issue Mar 4, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@Phallss
Copy link

Phallss commented Mar 4, 2024

Type of installation

Portable (.zip)

Type of reader (if reader related)

image

Steps to reproduce

Below is a small webm of the bug:
clicking.webm

The same thing happens when using the shortcut keys:
shortcut.webm

Note: the feature isn't entirely broken and works in some cases. Since you might not find a case where it is broken, I have provided 10 pages of a case where it doesn't work.

✔️ Expected Behavior

The double page view switches between the odd and even pairings of available pages.

❌ Actual Behavior

The double page view '2' pages and '2 odd' pages options are the same.

@Phallss Phallss added the bug Something isn't working label Mar 4, 2024
@github-project-automation github-project-automation bot moved this to For Review in yomikiru todo Mar 4, 2024
@Phallss
Copy link
Author

Phallss commented Mar 15, 2024

It seems the issue lies with the detection of what is and isn't a double page. Whenever I delete all other sized pages and only keep the single pages, everything works as it should (i.e. that's the workaround). If you could point me to the part where this detection happens I might take a look at fixing it myself.

I'd detect double pages simply by checking whether the width is greater than the height. If true then it can be a double page, otherwise it'll be detected as a single page. That way the offset should hopefully be fixed as well as there's no confusion anymore in case there's more than 2 different page sizes in the chapter.

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
None yet
Development

No branches or pull requests

1 participant