OpenComic icon indicating copy to clipboard operation
OpenComic copied to clipboard

[Bug]: Some bug

Open dajotim937 opened this issue 1 year ago • 12 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.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 avatar Jan 17 '24 22:01 dajotim937

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

dajotim937 avatar Jan 18 '24 00:01 dajotim937

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

https://github.com/ollm/OpenComic/assets/96221151/bee285e2-25b6-4b1a-94da-4acc001d96b8

Also I added error message from devtools under screenshots.

dajotim937 avatar Jan 20 '24 04:01 dajotim937

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 2a6254ae3875e08fd329b5410bd9156962cac6bc
  • Some errors in scroll reading a4887c3
  • Some errors on reading in scroll 1485b25c39eb1ebfcd352d7920e7a08c4d2af6f5
  • 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 8f57f65c3b13d37eb49ac4e8cfd251c5d9a06509

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) 37612bfdce13ce73348bda997bf3aeb32b8915af

ollm avatar Jan 26 '24 19:01 ollm

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

dajotim937 avatar Jan 27 '24 10:01 dajotim937

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

https://github.com/ollm/OpenComic/assets/96221151/d62c244e-82f9-41dc-b17b-5d0abb674491

dajotim937 avatar Jan 27 '24 16:01 dajotim937

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 avatar Jan 27 '24 16:01 dajotim937

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.

dajotim937 avatar Jan 27 '24 16:01 dajotim937

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 770afba6bd58e54b5ed33533bbc668cff49bca9b 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.

ollm avatar Jan 27 '24 17:01 ollm

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 avatar Jan 29 '24 23:01 dajotim937

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

2024-01-30_05-27-30

dajotim937 avatar Jan 30 '24 03:01 dajotim937

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 avatar Jan 30 '24 06:01 dajotim937

2024-01-31_17-21-22

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

dajotim937 avatar Jan 31 '24 15:01 dajotim937

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.

https://github.com/ollm/OpenComic/assets/96221151/4a9e0a9b-05cf-4f04-9983-3a497c8381f9

dajotim937 avatar Mar 01 '24 20:03 dajotim937

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 avatar Mar 07 '24 22:03 dajotim937

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 avatar Mar 19 '24 13:03 dajotim937

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

dajotim937 avatar Mar 19 '24 13:03 dajotim937

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.

ollm avatar Mar 29 '24 16:03 ollm