Vencord icon indicating copy to clipboard operation
Vencord copied to clipboard

feat(UserVoiceShow): update for simplified profiles

Open vishnyanetchereshnya opened this issue 1 year ago • 7 comments

patches update for simplified profiles Discord_0MPZMH7xoG Discord_lBw7b2Ee5e

vishnyanetchereshnya avatar Jul 04 '24 01:07 vishnyanetchereshnya

you shouldn't override old patches, in case

  • by some modern day miracle some people manage to turn off simplified profiles
  • bots still use old profiles

oh yea, u right

vishnyanetchereshnya avatar Jul 04 '24 02:07 vishnyanetchereshnya

~btw I'm not sure to where place voice status in user popup. originally this component located between send message section and roles section. in simplified profile already exists some "voice status"(activity), I just override it by section from this plugin and now this one located ABOVE roles section. would it be better to move it as in original plugin (below roles)?~

UPD: just remembered that activity shows not only voice but also other information 🥴. removed the override of this section and moved the voice channel section under the roles section

vishnyanetchereshnya avatar Jul 04 '24 02:07 vishnyanetchereshnya

could you show a screenshot please?

Vendicated avatar Jul 15 '24 02:07 Vendicated

could you show a screenshot please?

yup, added into main pr message

vishnyanetchereshnya avatar Jul 15 '24 03:07 vishnyanetchereshnya

it takes up too much space. the header should probably be removed in the small profile and replaced with a vc icon

as for the large profile, the header style is inconsistent with discord

Vendicated avatar Jul 15 '24 04:07 Vendicated

it takes up too much space. the header should probably be removed in the small profile and replaced with a vc icon

as for the large profile, the header style is inconsistent with discord

popout: will replace header by vc icon on left side of button modal: will change text style of header

btw as nino-dev said bots still uses old profiles. does plugin have to have patches for them?

vishnyanetchereshnya avatar Jul 15 '24 04:07 vishnyanetchereshnya

as for the large profile, the header style is inconsistent with discord

I know I've been talking about this a lot (and even opened a technically redundant PR related to it #2690 ), but you can use the Section component I found (see the previously linked PR and #2635 for examples)

btw as nino-dev said bots still uses old profiles. does plugin have to have patches for them?

enable the experiment for simplified bot profiles. i would ideally make sure it works there

also, not entirely related to this patch, a user can be in multiple voice channels at the same time, as long as they are in different servers. this might be more common with bots so maybe check if that is considered

Sqaaakoi avatar Jul 15 '24 11:07 Sqaaakoi

Superseded by https://github.com/Vendicated/Vencord/commit/bae9f8cbe696b3f4ddfd41f80e0bfdb5ccc3a755, thanks regardless!

Nuckyz avatar Sep 16 '24 18:09 Nuckyz