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]: Some bug #194

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

[Bug]: Some bug #194

dajotim937 opened this issue Jan 17, 2024 · 17 comments
Assignees
Labels

Comments

@dajotim937
Copy link

dajotim937 commented Jan 17, 2024

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

Operating System

Win10, 22H2, build 19045

Steps to reproduce

Didn't anything special, just read manhwa in full screen mode. Got that message multiple times on different files.
I think most (or all, I don't remember first ones) messages shows when you open next file.

Expected Behavior

Actual Behavior

2024-01-18_00-35-44

Error from devtools
opencomic.js:872 Uncaught TypeError: Cannot read properties of undefined (reading 'replace')
    at escapeBackSlash (opencomic.js:872:16)
    at opencomic.js:1075:9
    at Function.<anonymous> (opencomic.js:985:13)
    at Function.each (C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\node_modules\jquery\dist\jquery.js:383:19)
    at opencomic.js:984:6
    at Function.each (C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\node_modules\jquery\dist\jquery.js:383:19)
    at Object.<anonymous> (opencomic.js:974:4)
    at Object.wrapper (wrapHelper.js:10:19)
    at 7 (C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\scripts\builded\templates.js:2443:100)
    at Object.prog [as fn] (runtime.js:333:12)
    at Object.<anonymous> (if.js:19:22)
    at Object.wrapper (wrapHelper.js:10:19)
    at 2 (C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\scripts\builded\templates.js:2428:47)
    at prog (runtime.js:333:12)
    at execIteration (each.js:50:9)
    at Object.<anonymous> (each.js:63:13)
escapeBackSlash		 @ opencomic.js:872
(anonymous)		 @ opencomic.js:1075
(anonymous)		 @ opencomic.js:985
each			 @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\node_modules\jquery\dist\jquery.js:383
(anonymous)		 @ opencomic.js:984
each			 @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\node_modules\jquery\dist\jquery.js:383
(anonymous)		 @ opencomic.js:974
wrapper			 @ wrapHelper.js:10
7			 @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\scripts\builded\templates.js:2443
prog			 @ runtime.js:333
(anonymous)		 @ if.js:19
wrapper			 @ wrapHelper.js:10
2			 @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\scripts\builded\templates.js:2428
prog			 @ runtime.js:333
execIteration		 @ each.js:50
(anonymous)		 @ each.js:63
wrapper			 @ wrapHelper.js:10
1			 @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\scripts\builded\templates.js:2405
prog			 @ runtime.js:333
execIteration		 @ each.js:50
(anonymous)	 	 @ each.js:63
wrapper			 @ wrapHelper.js:10
main			 @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\scripts\builded\templates.js:2526
main			 @ runtime.js:230
ret		 	 @ runtime.js:250
loadTemplate		 @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\scripts\template.js:8
addHtmlImages		 @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\scripts\reading.js:253
images.<computed>.onload @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\scripts\reading.js:4935
load (async)
(anonymous)		 @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\scripts\reading.js:4923
each			 @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\node_modules\jquery\dist\jquery.js:383
each			 @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\node_modules\jquery\dist\jquery.js:205
read			 @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\scripts\reading.js:4917
openComic		 @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\scripts\dom.js:2102
await in openComic (async)
(anonymous)		 @ VM1165:1
setTimeout (async)
showNextComic		 @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\scripts\reading.js:1585
stayInLine		 @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\scripts\reading.js:736
resized			 @ C:\Users\***\AppData\Local\Programs\OpenComic\resources\app.asar\scripts\reading.js:2503

Also, with this error message, view area becomes white and image align left with some margin.
Also, I think the issue rise when OpenComic and other app trying to get access to HDD at same time (new file that is opening and other app also write something on same HDD).
UPD: or maybe alt+tab triggers something while loading next file.

Error message

No response

Additional Information

No response

Sample file

No response

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

dajotim937 commented Jan 18, 2024

Same situation, but different error.
2024-01-18_02-17-18

Error from devtools
C:\Users\***\AppD…ipts\reading.js:848 Uncaught TypeError: Cannot read properties of null (reading 'classList')
    at goToImageCL (C:\Users\***\AppD…s\reading.js:848:12)
    at C:\Users\***\AppD…s\reading.js:1186:3
    at eachImagesDistribution (C:\Users\***\AppD…s\reading.js:3625:5)
    at goToIndex (C:\Users\***\AppD…s\reading.js:1184:2)
    at images.<computed>.onload (C:\Users\***\AppD…s\reading.js:4946:6)
goToImageCL			@ C:\Users\***\AppD…ipts\reading.js:848
(anonymous)			@ C:\Users\***\AppD…pts\reading.js:1186
eachImagesDistribution		@ C:\Users\***\AppD…pts\reading.js:3625
goToIndex			@ C:\Users\***\AppD…pts\reading.js:1184
images.<computed>.onload	@ C:\Users\***\AppD…pts\reading.js:4946
load (async)		
(anonymous)			@ C:\Users\***\AppD…pts\reading.js:4923
each				@ C:\Users\***\AppD…\dist\jquery.js:383
each				@ C:\Users\***\AppD…\dist\jquery.js:205
read				@ C:\Users\***\AppD…pts\reading.js:4917
openComic			@ C:\Users\***\AppD…scripts\dom.js:2102
await in openComic (async)		
(anonymous)			@ VM849:1
setTimeout (async)		
showNextComic			@ C:\Users\***\AppD…pts\reading.js:1585
stayInLine			@ C:\Users\***\AppD…ipts\reading.js:736
resized				@ C:\Users\***\AppD…pts\reading.js:2503

@ollm ollm self-assigned this Jan 18, 2024
@dajotim937
Copy link
Author

dajotim937 commented Jan 20, 2024

UPD: or maybe alt+tab triggers something while loading next file.

Related: Alt+tab in fullscreen mode cause jump to some fixed position.

Screen record
2024-01-20_06-38-50.mp4

Also I added error message from devtools under screenshots.

@ollm
Copy link
Owner

ollm commented Jan 26, 2024

Thanks for the reports.

I have fixed most of the bugs (And those of the other issues), you can try the following build and test if there are still any that happen.

  • Improve stay in line in vertical reading 2a6254a
  • Some errors in scroll reading a4887c3
  • Some errors on reading in scroll 1485b25
  • Next chapter button not work inscroll mode if last page is smaller than the window and "Adjust to width" is enabled 2bbd49e
  • Error when pressing a key at startup 8f57f65

Build with fixes: https://mega.nz/file/Hf4WRTIL#_gfCjq91K3T8aTWCbjmxm2t0rcJwxnpyI3v4Elxl9Lk

This build also has some of the features requested in other issues.

  • Show the current reading title in app window 9520faa
  • Option to open file location of current reading from the file menu c9215dc
  • Setting to enable/disable go next/previous chapter with mouse scroll (Vertical reading) 37612bf

@dajotim937
Copy link
Author

Okay, thank you. I'll comment in relevant issues if I get those bugs again.

@dajotim937
Copy link
Author

Related: Alt+tab in fullscreen mode cause jump to some fixed position.

It's still there, but lesser than before. Mostly I can see only small movements but in general chapter stays on same position. But sometimes it still jumps.

Screen record
2024-01-27_18-15-49.mp4

@dajotim937
Copy link
Author

2024-01-27_18-23-00

Error from devtools

2024-01-27_18-26-02

Same situation. I click button on keyboard to open next chapter, alt+tab, got 3 same error window. In devtools there is 2 error message.

@dajotim937
Copy link
Author

2024-01-27_18-31-41

This one was not with alt+tab, but with F11 key (exit from fullscreen) when file was loading, I guess.

@ollm
Copy link
Owner

ollm commented Jan 27, 2024

It seems that the two errors occur when resizing the window before it loads (One when activating full screen and the other with alt+tab).

Try this build with this fix 770afba https://mega.nz/file/vCAj3RRD#qt89OlMa7jXHXtquckMMmTGeRmsjhYaTIP8iYHLWGXA

The full screen jump bug seems to be because a resize event is also triggered when using alt+tab in fullscreen, OpenComic tries to keep the image centered, but sometimes it doesn't seem to work correctly.

@dajotim937
Copy link
Author

dajotim937 commented Jan 29, 2024

Tested a bit. Looks like previous errors are gone. Now I don't get any error messages, but there is still error message in console.

2024-01-30_01-04-27

UPD: Well, I have read 50+ chapters with usual my read-flow and couldn't manage to get any other errors except error above.

@dajotim937
Copy link
Author

Said me, and immediately got an error when exiting from fullscreen while chapter was loading.

2024-01-30_05-27-30

@dajotim937
Copy link
Author

dajotim937 commented Jan 30, 2024

Not sure how I got this error. I think it happens while next chapter was loading and I click back button on the panel (left one button).

2024-01-30_08-09-12
2024-01-30_08-09-36

@dajotim937
Copy link
Author

2024-01-31_17-21-22

This one also didn't show any error messages, only in console.

@dajotim937
Copy link
Author

dajotim937 commented Mar 1, 2024

The full screen jump bug seems to be because a resize event is also triggered when using alt+tab in fullscreen

I guess this one also related to your message. Can't scroll past some position in fullscreen (F11).
Not webtoon mode (but still vertical), and used custom zoom in without "adjust to width" option. With this option everything's fine even in fullscreen.

2024-03-01_22-20-56.mp4

@dajotim937
Copy link
Author

One more error. Happens when folder has corrupted cbz file. In my case it was created file from torrent but it hasn't been download when I opened OpenComic (because I downloaded to folder that I added as master folder).

2024-03-07_14-51-04

@dajotim937
Copy link
Author

2024-03-18_03-22-02

Error from devtools
node:internal/validators:163 Uncaught TypeError: The "path" argument must be of type string. Received undefined
    at __node_internal_captureLargerStackTrace (node:internal/errors:490:5)
    at new NodeError (node:internal/errors:399:5)
    at validateString (node:internal/validators:163:11)
    at Object.normalize (node:path:305:5)
    at Readable.<anonymous> (C:\Users\***\AppD…-manager.js:1828:34)
    at Readable.emit (node:events:513:28)
    at addChunk (node:internal/streams/readable:324:12)
    at readableAddChunk (node:internal/streams/readable:297:9)
    at Readable.push (node:internal/streams/readable:234:10)
    at Object.bodyData (C:\Users\***\AppD…\src\maybe.js:77:12)
    at C:\Users\***\AppD…src\events.js:83:11
    at Socket.<anonymous> (C:\Users\***\AppD…\lifecycle.js:64:51)
    at Socket.emit (node:events:513:28)
    at addChunk (node:internal/streams/readable:324:12)
    at readableAddChunk (node:internal/streams/readable:297:9)
    at Readable.push (node:internal/streams/readable:234:10)
    at Pipe.onStreamRead (node:internal/stream_base_commons:190:23)
__node_internal_captureLargerStackTrace	@	node:internal/errors:490
NodeError	 @ node:internal/errors:399
validateString	 @ node:internal/validators:163
normalize	 @ node:path:305
(anonymous)	 @ C:\Users\***\AppD…ile-manager.js:1828
emit	         @ node:events:513
addChunk	 @ node:internal/streams/readable:324
readableAddChunk @ node:internal/streams/readable:297
Readable.push	 @ node:internal/streams/readable:234
bodyData	 @ C:\Users\***\AppD…-7z\src\maybe.js:77
(anonymous)	 @ C:\Users\***\AppD…7z\src\events.js:83
(anonymous)	 @ C:\Users\***\AppD…src\lifecycle.js:64
emit             @ node:events:513
addChunk	 @ node:internal/streams/readable:324
readableAddChunk @ node:internal/streams/readable:297
Readable.push	 @ node:internal/streams/readable:234
onStreamRead	 @ node:internal/stream_base_commons:190

@dajotim937
Copy link
Author

2024-03-18_03-44-49

Error from devtools
C:\Users\***\AppD…scripts\dom.js:1194 Error: incomplete frame
    at Object.readJson (C:\Users\***\AppD…pts\cache.js:229:33)
    at file.readCompressed (C:\Users\***\AppD…e-manager.js:207:20)
    at file.read (C:\Users\***\AppD…le-manager.js:65:24)
    at file.images (C:\Users\***\AppD…e-manager.js:445:36)
    at getFolderThumbnails (C:\Users\***\AppD…ipts\dom.js:1166:28)
    at C:\Users\***\AppD…ripts\dom.js:245:25
    at async Object.loadIndexPage (C:\Users\***\AppD…ripts\dom.js:778:19)
getFolderThumbnails       @ C:\Users\***\AppD…scripts\dom.js:1194
await in getFolderThumbnails (async)        
(anonymous)               @ C:\Users\***\AppD…\scripts\dom.js:245
Promise.then (async)        
loadFilesIndexPage        @ C:\Users\***\AppD…\scripts\dom.js:167
loadIndexPage             @ C:\Users\***\AppD…\scripts\dom.js:778
onclick                   @ index.html:1

@ollm
Copy link
Owner

ollm commented Mar 29, 2024

I'm going to close this for now, since I don't know which ones are solved or not, if there are some that still happen in the latest version v1.2.0, please open a new issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants