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

[Bug]: Last page / Next chapter button on panel has wrong state if last page is small #195

Closed
1 task done
dajotim937 opened this issue Jan 26, 2024 · 8 comments
Closed
1 task done

Comments

@dajotim937
Copy link

Preflight Checklist

  • I have searched the issue tracker for a bug report that matches the one I want to file, without success.

OpenComic Version

1.1.0 (well, I'm using your custom test build for me)

Operating System

Win10, 22H2, build 19045

Steps to reproduce

  1. Webtoon with last page smaller than current vertical size of reading area.
  2. Webtoon reading view.
  3. Click either last page button or click on last page from left side scroll panel

Expected Behavior

Button should change to "next chapter" since there nothing left to scroll (full last page fit into vertical size of reading area)

Actual Behavior

App showing "last page" button like there is something left to scroll. Clicking on it scroll webtoon up a bit, clicking again scroll to the end and changing button to "next chapter".

Error message

No response

Additional Information

No response

Sample file

No response

@dajotim937 dajotim937 added the bug label Jan 26, 2024
@dajotim937
Copy link
Author

Fixed in test build.

@dajotim937
Copy link
Author

Related: boundary situation, if you scroll last chapter with arrow key and last page is small last page/next chapter will have wrong state. Not that it doesn't matter but still.

And suggestion: open manga page after reading last chapter and clicking next chapter button.

@dajotim937
Copy link
Author

Related:

2024-01-30_10-37-02.mp4

@ollm
Copy link
Owner

ollm commented Mar 29, 2024

Can you confirm if the error still occurs in latest version v1.2.0?

@dajotim937
Copy link
Author

Original bug from first comment is fixed, related bug from #195 (comment) is not fixed.

Additional info: bug from video is happened only when there is next chapter. If there isn't, it scroll to the end as intended.

@ollm
Copy link
Owner

ollm commented Mar 31, 2024

@dajotim937
Copy link
Author

Looks like I can't reproduce it anymore.

@ollm ollm added this to the v1.3.0 milestone Aug 11, 2024
@ollm
Copy link
Owner

ollm commented Oct 4, 2024

Closed as fixed and available in v1.3.0

@ollm ollm closed this as completed Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants