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

better custom packs support + import chibisafe albums as custom packs #58

Merged
merged 33 commits into from
May 6, 2022

Conversation

BobbyWibowo
Copy link
Contributor

@BobbyWibowo BobbyWibowo commented Apr 30, 2022

image

image

As always, I try to do micro commits, so despite the high commit counts, each commit tries to do the bare minimum


3 new settings

image

image
image


My custom packs for sample valid JSONs of non-chibisafe album packs https://magane.fiery.me/configs/

Bobby added 30 commits April 30, 2022 14:08
added a new section for chibisafe album urls
images.weserv.nl still doesn't support APNG -> GIF, but if one day...

also magane proxy doesn't have hasAnimation prop for emoji packs yet
so that it will always be able to fallback to file emoji even if the
entire album is made of non-image/video files
styling + error handling
to facilitate updates without having to delete the packs first
prevent it from closing when clicking things on bdapi modals
partial properties overwrites is now only enabled with an extra toggle
removed overwrite flag from window.appendCustomPack()

overwriting in itself requires extra steps such as double-checking
favorites and subscribed pack data, but that will be relegated to
the update pack feature

window.magane.editPack() is then added for those that still want to
modify some properties,
but due to its nature as a console function, it's meant for the more
advanced users that can handle the consequences
unified the feature into supporting all remote packs altogehter
where non-chibisafe album packs will be imported with JSON config file

also wrote it in a way that allows easy & clean expansion if required

additionally, adding custom packs will now be presented with a simple
consent confirmation
we never know if fetching third-party url and trying to parse its body
as json can somehow be abused by bad actors
@BobbyWibowo BobbyWibowo marked this pull request as ready for review May 4, 2022 11:56
@BobbyWibowo BobbyWibowo changed the title import chibisafe albums as custom packs better custom packs support + import chibisafe albums as custom packs May 4, 2022
@BobbyWibowo
Copy link
Contributor Author

@Pitu ready for review 👋

Quite a few commits, but hopefully the commit names, + descriptions if applicable, are clear enough

I sorta re-did the main feature (chibisafe albums as custom packs, specifically) by commit eaa2e8b, but most of the commits before it are still important too

Take your time

@Pitu Pitu merged commit f031dac into Pitu:master May 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants