manga-loader icon indicating copy to clipboard operation
manga-loader copied to clipboard

MLEP: Implementation Repository System

Open TsXor opened this issue 3 years ago • 6 comments

See https://github.com/TsXor/manga-loader/blob/enhance/MLEP-IRS.md for details. I don't think @fuzetsu can see this, but at least I should declare its existance.

This PR contains my previous enhances as well.

TsXor avatar Dec 06 '22 10:12 TsXor

@sprdk4 @SilentSparrow02

TsXor avatar Dec 06 '22 10:12 TsXor

I found that I got my fork repo in a huge mess so I extracted patches, reapplied them and made commits cleaner. Here is original git log if anyone is interested...

$ git log commit a1a7f808d2e6ccbb850e30785110e2c99b07e76f (HEAD -> enhance, origin/master, origin/HEAD) Author: 23Xor Date: Tue Dec 6 16:22:53 2022 +0800
Add Implementation Repository System proposal.

commit 515a04862a822a5af58cbfab2dec26e6ea870757 Author: 23Xor [email protected] Date: Tue Dec 6 11:44:51 2022 +0800

add impl editor

commit c66910795a7c814f27e933a31439c352fd432a91 Author: 23Xor [email protected] Date: Mon Dec 5 09:07:31 2022 +0800

adjust matching order

commit 703021a8ab81d5918f9a5e7b1e215f19056196d9 Author: TsXor [email protected] Date: Mon Dec 5 08:24:44 2022 +0800

fix ero-senmanga

i am lsp ;)

commit dce85e99c08a53ac554324880c94da390c23b08a Author: 23Xor [email protected] Date: Mon Dec 5 08:13:24 2022 +0800

bugfix

commit 0673889327fdfdf36f907509f244849931862a18 Author: TsXor [email protected] Date: Sat Dec 3 23:01:41 2022 +0800

Update and rename version.json to version

commit 68f14336efceeb9460d59d5090845f4fc7d79de7 Author: 23Xor [email protected] Date: Sun Dec 4 16:20:01 2022 +0800

now it is usable

commit 39fc9c6e8d30519539730fd65c4c3221ab043f61 Author: 23Xor [email protected] Date: Sat Dec 3 22:40:20 2022 +0800

*merge features from other branches*
Let Manga Loader load impls dynamically.
NOT READY TO USE

commit 64164293608661b171e3958f06e9aed5a59d268f (origin/change-1div) Author: TsXor [email protected] Date: Sun Dec 26 18:07:00 2021 +0800

Update manga-loader.user.js

commit c84897028cd346322f306fb95dcf0e2f512eadb0 Author: TsXor [email protected] Date: Sat Dec 18 21:43:47 2021 +0800

Update manga-loader.user.js

commit 8b5fc51bed67ea2cc701b751db39bb4a3445716a (origin/change-refresh, master) Author: Daniel Loomer [email protected] Date: Wed Aug 5 20:22:57 2020 -0400

get rid of bad domain in namespace field

fixes #68

TsXor avatar Dec 06 '22 10:12 TsXor

Glad that someone's trying to develop this. Does it work for you though? Your script isn't working for me on any sites

SilentSparrow02 avatar Dec 07 '22 12:12 SilentSparrow02

Glad that someone's trying to develop this. Does it work for you though? Your script isn't working for me on any sites

You need to set a valid repo URL in the settings panel. For example: https://raw.githubusercontent.com/TsXor/manga-loader/enhance/impls/ I set the default value to https://raw.githubusercontent.com/fuzetsu/manga-loader/master/impls/, which will not work until fuzetsu take any action ;) Oh I forgot we cannot open setting panel when there is no impl... a dead loop :( You can search for a line in the script which starts with var default_repo_url = and change the link after it to https://raw.githubusercontent.com/TsXor/manga-loader/enhance/impls/

TsXor avatar Dec 08 '22 04:12 TsXor

what kind of functionality/fixes does this bring that a non-dev user would care about? Though I added several user experience features to my own branch, javascript ain't my forte so I struggled enough trying to figure out what I was doing yet alone trying to be provide peer review. I'll note that I generally preferred having everything in one big .js file instead of being all spread out. Also the only non-nsfw site I found that worked while I was playing around with my changes was https://fanfox.net/ so I would be interested in knowing whether or not you managed to fix compatability with the other sites.

sprdk4 avatar Dec 09 '22 00:12 sprdk4

what kind of functionality/fixes does this bring that a non-dev user would care about? Though I added several user experience features to my own branch, javascript ain't my forte so I struggled enough trying to figure out what I was doing yet alone trying to be provide peer review. I'll note that I generally preferred having everything in one big .js file instead of being all spread out.

It will make fixing them easier, one big js will make finding a certain implementation inside the file kind of unbearable pain, Also, when you put all impls in one big js, browser have to load all impls every time, I not only splitted them , but also make them loaded only when needed. Also, the User Override and impl editor make it easier to test new impls and avoids confusions like https://github.com/fuzetsu/manga-loader/issues/81

About splitting the file, as you said

javascript ain't my forte

You should know that in other langs, the ability of modularization or just splitting the file is just a basic. When a project grow larger, they just have to do this so that they will not become an annoying disorder.

Also the only non-nsfw site I found that worked while I was playing around with my changes was https://fanfox.net/ so I would be interested in knowing whether or not you managed to fix compatability with the other sites.

For me, testing all those sites is also kind of pain because sometimes it is hard to know whether the site is really down or I am just not using the right proxy(<- illegal thing!). Just like you may not be able to test dmzj in your internet. You may use proxies only for being anonymous and secure your privacy, but it is what I need to access even github.

TsXor avatar Dec 09 '22 02:12 TsXor