OpenComic icon indicating copy to clipboard operation
OpenComic copied to clipboard

[Bug]: Messed up page sequence while viewing pdfs.

Open devneel001 opened this issue 8 months ago • 3 comments

Preflight Checklist

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

OpenComic Version

1.4.1

Operating System

windowsn 11 home

Installed via

.exe

Steps to reproduce

  1. open pdf
  2. view pdf

Expected Behavior

Should display pdf as per sequence.

Actual Behavior

Doesnt display pdf as per sequence. Tried opening pdf in other pdf readers such as chrome, brave, cover and sumatra pdf, works fine in those. Tried re installing, portable versions as well as 3 older versions, issue persists.

Error message

No response

Additional Information

No response

Sample file

No response

devneel001 avatar Apr 07 '25 19:04 devneel001

Thanks for reporting!

I think the problem is that you have changed the order, probably to just Name, I recommend you change it to Name / Number, please note that you have to be inside a folder/file, since the order of the main pages are independent.

I'll make sure this doesn't affect PDFs in the next version, as it doesn't make much sense, and also I'm going to try that the order can be changed from reading, as it may currently seem confusing.

Image

ollm avatar Apr 08 '25 18:04 ollm

Hey thanks for that. It really does not make sense that changing from name/number to name would affect the sequence but regardless thanks for the quick resolution.

devneel001 avatar Apr 09 '25 04:04 devneel001

This is because the PDF pages in OpenComic are named as follows page-1, page-2, page-3, etc. causing them to be sorted incorrectly when are sorted by name, something like that.

page-1
page-10
page-100
...
page-2
page-20
etc.

I made this change b2041ab73ca592a574bc0d2abb7f19229681e7cf so now leading zeros are added, making it sort correctly when are sorted by name.

If you want to try it, I've left you a build of the master branch.

Build: https://mega.nz/file/COZnVRjR#rKXQ9-yMoxaGucheUTzPVq04s-0J0OoX314exMIrjSc

ollm avatar Apr 09 '25 11:04 ollm

Closing this, since the fix is now available in v1.6.0

ollm avatar Jul 27 '25 18:07 ollm