Vencord icon indicating copy to clipboard operation
Vencord copied to clipboard

feat(plugin): FakeProfileThemesAndEffects

Open ryan-0324 opened this issue 1 year ago • 16 comments

This plugin is meant to replace FakeProfileThemes. It allows profile theming and the usage of profile effects by hiding the colors and effect ID in your About Me using invisible, zero-width characters.

  • It includes a UI (called the FPTE Builder) for generating the profile theme/effect (FPTE) strings in the User Profile and Server Profiles settings pages.
  • The FPTE strings can be made backwards compatible so that users of older plugins will still see your profile theme.
  • When the "Build backwards compatible FPTE" setting is off, the generated strings will use at most 22 characters (12 codepoints) of your About Me.
  • The FPTE Builder can be hidden through the plugin settings.

ryan-0324 avatar Oct 30 '23 03:10 ryan-0324

this is a crazy amount of code for such a small feature. why is all of this needed? it seems very overengineered

Vendicated avatar Nov 09 '23 01:11 Vendicated

LGTM 👍

ImpishMoxxie avatar Nov 10 '23 22:11 ImpishMoxxie

Looks good

tiagorangel1 avatar Nov 17 '23 13:11 tiagorangel1

How do I use it? @ryan-0324

beatmakerofficial avatar Jan 12 '24 15:01 beatmakerofficial

@rniii Do you know what could be causing the Invalid header errors? All of my headers were automatically added on save.

ryan-0324 avatar Jan 30 '24 02:01 ryan-0324

get that thing declined, because this is reupload from discord assets.

now if only we get custom profile effects, rather then just a reupload one, this would've be prefect.

omegion3 avatar Feb 20 '24 14:02 omegion3

get that thing declined, because this is reupload from discord assets.

now if only we get custom profile effects, rather then just a reupload one, this would've be prefect.

What are you talking about

MrDiamondDog avatar Feb 20 '24 14:02 MrDiamondDog

What are you talking about

nah i'm just kidding, even though it's a reupload assets, but i hope this plugin will merge

omegion3 avatar Feb 20 '24 15:02 omegion3

nah i'm just kidding, even though it's a reupload assets, but i hope this plugin will merge

It doesn't have reuploaded assets..?

MrDiamondDog avatar Feb 20 '24 15:02 MrDiamondDog

It doesn't have reuploaded assets..? well it does sadly, but hopefully someone might plan to make custom profile effect or this plugin will finally merge.

omegion3 avatar Mar 02 '24 11:03 omegion3

What's the status on this? Anything blocking a merge on this?

CompeyDev avatar Mar 20 '24 08:03 CompeyDev

What's the status on this? Anything blocking a merge on this?

vee gets around to PRs "eventually" this is also (relatively) a big plugin

Sqaaakoi avatar Mar 20 '24 08:03 Sqaaakoi

Makes sense, I'm running a manually compiled build from ryan's repo and it seems to work about fine. Haven't really looked much at the code though.

CompeyDev avatar Mar 20 '24 08:03 CompeyDev

Some issues:

  1. The stock profile effect editor is patched by this plugin resulting in it displaying all effects as unlocked and are unable to apply any & it falsely showing the remove effect button. Should behave like Decor where it doesn't modify the stock feature.

  2. The FPTE Builder should auto fill with the values from the bio.

  3. The FPTE Builder should have buttons to save to bio without manual copy & pasting being required And likewise have a clear button.

Covkie avatar Aug 31 '24 11:08 Covkie

Some issues:

  1. The stock profile effect editor is patched by this plugin resulting in it displaying all effects as unlocked and are unable to apply any & it falsely showing the remove effect button. Should behave like Decor where it doesn't modify the stock feature.
  2. The FPTE Builder should auto fill with the values from the bio.
  3. The FPTE Builder should have buttons to save to bio without manual copy & pasting being required And likewise have a clear button.
  1. This seems to be a recent breakage. Thanks for letting me know.
  2. I've though about adding something like this before, and it seems reasonable.
  3. This has been suggested before, and I don't think it's worth the added complexity:
    1. Should I just prepend the FPTE string to the user's bio, displaying an error if their bio doesn't have enough space or already starts with an FPTE character?
    2. Or should I try to automatically detect preexisting FPTE strings and strings from other plugins, and remove them from the user's bio, showing an error if there still isn't enough space?
    3. If not (ii), when the user's bio already starts with an FPTE character, should I add a non-FPTE character as a separator, or should I display an error?
    4. Should I send an API request to save the user's bio, or should I have the user do it manually?

ryan-0324 avatar Sep 01 '24 18:09 ryan-0324

The stock profile effect setting still falsely shows the "Remove Effect" button

Or should I try to automatically detect preexisting FPTE strings and strings from other plugins, and remove them from the user's bio, showing an error if there still isn't enough space?

Bio processing is already in place, can you not just replace the detected/processed fpte string

Should I send an API request to save the user's bio, or should I have the user do it manually?

The user should do it manually imo

Covkie avatar Sep 13 '24 02:09 Covkie