anka-213 / webcomic_reader

Webcomic Reader userscript at
https://openuserjs.org/scripts/anka-213/Webcomic_Reader
MIT License
120 stars 27 forks source link

Please add MangaSee #25

Closed kobuki closed 7 years ago

kobuki commented 8 years ago

Please consider adding http://mangasee.co/

Thanks!

anka-213 commented 8 years ago

@kobuki It should work now. If it doesn't, please reopen this issue. (You might have to tell greasemonkey/tampermonkey to force-update the script.)

kobuki commented 8 years ago

@anka-213 I force-refreshed it, and can see the new related code but it doesn't work at all. All other supported sites work as usual.

anka-213 commented 8 years ago

@kobuki Hmm. Strange. It works for me. Are you using Firefox or Chrome? Is there any output in the console? Can you paste the url of a page that does not work?

kobuki commented 8 years ago

@anka-213 Oh, my bad, sorry. I forgot to delete my own custom settings for the site which overrided the new site support code. Works perfectly, thanks!

kobuki commented 8 years ago

This might warrant a separate issue, but first I'm going to post here. Please close and I add a new ticket if needed.

The site uses a bar on top in the viewer that covers a sizable chunk of the images at the top, sometimes hiding speech bubbles and thus making easy reading hard without scrolling up and down sometimes. Can an option be added to change the viewport size to set a frame for scaling and positioning per site? The current settings can't solve this in a useful way. I can add large borders but then the images become a lot smaller... Not really ideal.

anka-213 commented 8 years ago

Doesn't minimal layout solve the issue?

kobuki commented 8 years ago

Only half of it. It turns off the site's navbar which forces other quirks on the user for navigating on the site - switching back and forth when reading up/switching between the latest followed chapters, requiring the site navigation tools, for instance. In a certain regard it's better because it actually allows to see the top of the pages, but worse in other aspects.

anka-213 commented 8 years ago

I can:

a) Add the navbar to the extras, so you can find it below the comic. (so minimal mode works) b) Add some css for the site so that the navbar is no longer hovering (i.e. scrolls with the page instead of staying with at the top) and thus no longer covers the comic. or c) Try to fix so it scrolls to the correct place while keeping the navbar hovering.

Which would you prefer?

anka-213 commented 8 years ago

("a" is probably easiest)

kobuki commented 8 years ago

Hm, yeah, I figured a) would be probably the simplest, though I'd personally prefer b) -- this is what the site should have done originally :\ Most sites work like that.

anka-213 commented 8 years ago

C is hardest, so I'm glad you didn't pick that. ;) I'll give b a shot, it shouldn't be too hard.

kobuki commented 8 years ago

Wonderful support we have here, thanks ;)

anka-213 commented 8 years ago

Haha, thanks. (^○^)

anka-213 commented 8 years ago

I threw in "a" as well, as a bonus, since it was almost no effort at all ;)

kobuki commented 8 years ago

Nice, working great! Ghostery interferes with the new change (images get shifted down by ~12px) but that's not the script's fault at all.

kobuki commented 8 years ago

The script stopped working... The first image is correctly resized and positioned but it's impossible to go next/back with the green arrows. See for example: http://mangasee.co/manga/?series=PlatinumEnd&chapter=8&index=1&page=1

Tmp341 commented 8 years ago

it's working for me, even though i have ghostery.

kobuki commented 8 years ago

Hmm, weird. Now it does work for me too. It was broken for a few days, though. Not necessarily the script - the site went through several small changes recently. Maybe that. Ah, well.

kobuki commented 8 years ago

OK, it works only for the few pages of any manga. Then it indefinitely shows the hourglass instead of the roght arrow. If I do several page refreshes, it continues to work for some chapter images and then the same issue again. All other sites I frequent do work flawlessly, this is the only one exhibiting the problem. Tried in FF 47.0.1 and Chrome 51.0.2704.106. Maybe something about image prefetch?

Tmp341 commented 8 years ago

I got something weird on FF 48.0b6. I've tried a random manga, it worked 4 or 5 pages. Then script stopped showing arrow and then my mouse cursor changed to it's loading animation. Until i change the page from the menu, nothing happens. Even tried it with/without Ghostery.

Edit: It seems there was an update to FF 48.0b7, but nothing has changed.

kobuki commented 8 years ago

Thanks for reporting this, it seems it's not only on my setup then.

Tmp341 commented 8 years ago

It seems my problem isn't occuring anymore. Last night, i've done a deep cleaning with CCleaner. Today, i've finished a chapter without a problem.

kobuki commented 8 years ago

No change here. Problem persists in both Firefox and Chrome as well.

anka-213 commented 8 years ago

Do you get any error messages in the console?

kobuki commented 8 years ago

None I can make sense of. Some CORS errors. Attached is a console copypasta.

console.txt

anka-213 commented 8 years ago

From looking at the log, it seems like the problem is that it tries to go from the domain mangasee.co to a1.mangasee.co. The script doesn't work accross domains, because the browser blocks cross domain requests (CORS). I'll take a look at what causes it.

anka-213 commented 7 years ago

Oops, wrong bug number

kobuki commented 7 years ago

LOL, well, it seems that the problem solved itself in the meantime. Now it works properly as before. They probably changed something in the page's code 1-2 weeks ago. Please don't close this yet - let's allow others to comment for a while. Thanks!

anka-213 commented 7 years ago

Nice. :D

You can still comment on closed issues, but sure, I'll leave it open for a while if you want.

anka-213 commented 7 years ago

I'll close this now. If the site stops working again, you can always open a new issue.