#πΎ-core-development
1 messages Β· Page 267 of 1
make venapp auto detect aamias snippetr and close the pr
give me perms i will close everything
ya rules are pointless the people who need them wont read
just makes it easier to justify why a pr is being closed
i feel like having a rule about going around an important discord mechanism like spoofing quests or "getting around admin perms" might be useful
this should probably be closed as well
hi thank you for your efforts!
we recently did this same migration which supersedes this pr. if there are some new types in this pr, it'd be great if you could add them in a new pr
as for all your other changes, if you think they are still needed, send them as separate prs (one change per pr) and we will consider them. adding all of them in one single mega pr like this one makes it impossible to merge
<img width="804" height="338" alt="image" src="https://github.com/user-attachments/...
if a relevant person leaves feedback it automatically moves to in review
for the stale part we could either use stale bot or just do it ourselves with a cron trigger
cron could check once a day for stale prs
maybe we could introduce a fourth category "Needs Re-review"
if all feedback is marked as resolved ci will move there
that way it's super easy to keep track
have a working version
Is this actually happening? After over a year? I don't even know if the code in this PR is still up-to-date lol. Let me take a look at your review and I will get back soon :)
I had this for over a year now. It still works, never produced any crashes. But I'll see if there's something else to match.
Made a new patch replaces the component now so this is no longer relevant :smile:
Oops, just noticed I broke that it controls the speed for voice messages, working on a fix
Done. I had to grab the ref from above, let me know if the patch is okay
tbh i don't really see this as any issue, afaik DAVE is open source
clients can implement it
I think it would be fun to watch aliucord
(if they will is another question)
@dapper tiger do https://github.com/discord/libdave
Become the Princess Knight
As the lethal hunter Hornet, adventure through a kingdom ruled by silk and song! Captured and taken to this unfamiliar world, prepare to battle mighty foes and solve ancient mysteries as you ascend on a deadly pilgrimage to the kingdomβs peak.
Hollow Knight: Silksong is the epic sequel to Hollow Knight, the award wiβ¦
Price
$19.99
I am currently unavailable guys
its out?
see you all in 5 days
tell me
I can either get a ref from the actual const or from a component using the same ref but way closer
which is better
the one that's easier
this was d3sox's patch but I figured that was alot so I was making it shorter but I don't know if I should use the front for the ref or ref:
I just grabbed it from one using it 
figured I pr it to his thing if he likes it he can use it
yes I 100% hate the way I did that
no I dont plan on redoing it
also never played hollow knight
was it good
the second is better but it still seems not that good
hes replacing all of this with this
im grabbing ref from here while he gets it from here
module id: 201542
i never played the first but i kinda got interested
steam is just fucked because of its launch
I just got to the purchase page and then it died again
it's steam fault for not being better
real
apparently nintendo and ps shops are also affected
everything is/was down

I played the first and im def playing the second
I'm not home yet
alr
but rn I'm prob gonna say u can start with the second
I doubt they are gonna do so many things related from the first
and iirc it's likely a prequel to the first game
but I have yet to confirm
okayy
/\(0,\i\.jsxs?\)\(.{0,50}\.playbackRateContainer.+?}\)}\)(?<=ref:(\i),className:\i\.audioElement.+?)/
jesus
capturing group 1 is now the ref
and no more capturing group for the extra you just have to insert again
the whole match is what you need to remove
untested but unless I made a mistake it should work
I had to change a few of mine
they were so slow until I removed the lookback
lol
yes it depends how you use it
yup I realized after that day lmfao
tweaks section good
I know lol
I forgot your username to ping π
I'm not that smart
English isn't my first language if coding was one it would be
Then english
Yw
doesn't seem aligned and it'd be way better to make it a hover thing like the volume slider
<img width="463" height="198" alt="image" src="https://github.com/user-attachments/assets/21d18994-4988-4dc1-b6c9-4954923d5cd0" />
Closing as I will likely revive this in a different way from scratch
I like that idea. Might work on this later or tomorrow.
My leftover stuff that would be nice to have:
- Showing the current speed on the button
- Making it work in fullscreen are also still
Is this being worked on?
@austere talon we need a project for improvements
remove anything above 2 and add a slider for fine grained
theres a menuslider
this is your pr btw @austere talon
no it isnt lmao
it's someone else's
I accidentally closed it so I reopened it myself
are u not handling it now?
no
ahh okay
i just reopened it because I deleted his commits
I was so close
yep those sound important too
also would be nice to make it remember the previous speed (in the current session? or all sessions? idk), toggleable in settings
Afair it's the opposite, Silksong is the sequel
You can start from Silksong but the first one is such a gem that it'd be a pity to miss out on it
I'm done with the off topic

do you want the action to move them to in review when they're reviewed?
yeahh
if someone (relevant) reviews, move to In Review
if all reviews are resolved, move from In Review to Needs ReReview (new column)
if the pr is in no project, maybe it could just leave a comment saying to add it to a project manually
it's pretty basic rn
does it even need a token
yes
the token needs to be classic
and i needed to add these permissions to get it to work on my test repo
you dont need that
i tried that lol
if you can get it to work lmk
fine grained tokens don't work for projects
tbh i'd just make an alt and use the token for that account
i guess because the prooject is not actually part of the repo
insane
pr i forgot about got a review
oh and the patch is broken
lovely
gonna fix in a bit and fix the blockedcontext pr after 
is it stable compatible
make reporter work for prs @fossil inlet
actually shouldnt be that hard
but hard part is only testing the things in the pr
seems to be stable compatible
the find anyway
second one isnt, experiments should be

add a comment
// TODO: remove .button once it's gone on stable
(we will remove in half a year)
bing π
would be nice if they actually told you what the treatments are
you will forward datamining repo comment instead
?
You can just look in experiments tab???
effort
now that vencord toolbox is more out of the way, should we make it enabled by default?
so it's more useful and we can actually put stuff there
github loves not sending notifications
there was the other pr that handled plugins / managed style too so should i even work on this
which
3153
canUseCustomNotificationSounds looks like it was removed and for onepingperdm soundpack:\i, was removed
vtest
Now testing! 
onepingperdm's sound match is fine alone because its the only one using \i ? \i : void 0 but I figured anchoring it with onclick still would be better
Bad Patches
FakeNitro (had no effect):
ID: 74538
Match: ```
/(?<=canUseCustomNotificationSounds:)\i/
**__OnePingPerDM (had no effect):__**
ID: `974180`
Match: ```
/sound:(\i\?\i:void 0,soundpack:\i,volume:\i,onClick)/
No issues found
Seems like everything is working fine (for now) 
this entire chunk is gone now for fakenitro yes I know it match smth else but I was checking this for usage
canary
where c is used on stable
module id: 974180
Are they removing custom notification sounds
Probably because the find mentioning soundpacks failed as well
OnePingPerDM doesn't have stable comp
FakeNitro does as its just removed
this is literally the only one using ? : π
we can use pull request template + project workflows to automatically organise prs
i hate discord 
i have to find an entirely new place to put the patch into cuz otherwise there's just gonna be fucking gaps in the chat
and this is the second pr i have to do this with 
Thank you for the contribution!
In the future, please send pull requests from feature branches instead of the main (or dev) branch. Also your editor & formatter are clearly misconfigured
b6e96a4 MemberCount: also show members in voice (#2937) - ultard
i love when i have to work with a pr on a branch called dev and fuck up my local clone cause now it's tracking the wrong fucking dev π
why did they add themselves at the very top in the dev object bruh
insane
i didnt even notice
why is the icon for total members a border-radius css trick and not an svg π
I just noticed sometimes it becomes slightly elongated π
i really need to start using sadans companion
what's the original onClick? overwriting it kinda sucks
very misleading function name
renderOpenPreviewText: () => <Text variant="text-sm/normal">Open Preview</Text>,
how
why does it matter
im assuming subpixel width from the number shifting
theres still a gap
fuck you discord
cause it can cause elongated icon as I said above
i never expeirneced this
change it to a svg if you really care
noty
amazing way to hide a reply icl
^
the green blob is the same tho lol
yeah but the green blob doesnt have text to its left shifting in width
the online icon has the green text shifting next to it
oh dear lord that's horrid
i don't really think we need these but thank you
i don't really get the point of this pr, i don't think it adds any objective improvements
wtf is this experiment #π₯-vencord-support-π₯ message

i dont think its possible to cleanly hide replies to blocked messages anymore
@austere talon do u wanna take a look at it or should i just close the pr
??
i may also just be heavily incompetent
why wouldnt it be possible
this
no matter how i try to take the message out it ends up with a gap where it was
and idk what to try next
it's hard to hide messages properly
ya im noticing
thats why im asking if u wanna take a look at it
cuz i cant figure it out
but also im retarded so 
why do we need a new patch
to hide replies that already exist in the chat history?
tjey dont go through the old patch
oh the other message patches the blocked thing
unless the old patch just was fucked all along idek anymore
so cursed
yeaa
but im confused now on why the messagestore patch doesnt wor-nvm yea it obv doesnt work cuz its a separate setting
we could just patch getMessages and remove them from the array maybe
idk if that would have a negative effect on anything else
what are you tryna do
hiding replies to blocked messages
oh is the equicord version upgraded. myb
that one looks good
still has a small gap where the message would be but works
should i just steal
i feel like you surely could interfere earlier in the call chain to stop it from appearing at all
^
i tried to filter as early as chat history being rendered and failed
im looking into that rn
?
the gap is identical to the gap with my attempts
wdym profile though
does equicord just accept every single plugin
I never even heard of this shit in my life https://github.com/Equicord/Equicord/blob/main/src/equicordplugins/absRPC/index.tsx#L87
index.tsx: Line 87
name: "AudioBookShelfRichPresence",
if it works it probably gets accepted. but thororen has been working on merging a bunch of smaller plugins recently
yea
gonna make the GiveMeYourCreditCardInformation plugin
Bro just mad they wonβt add his token logger
how is it mad to say a fact
equicord accepts everything
Cope lil bro
Idc
why show deaf but not muted?
because you can't talk to someone who is deaf, but you can talk if they are just muted
LOVE
keep annoying drama out of here no one cares
@odd heath think I got a working patch
@austere talon what do you think of this? works fine (i don't notice any delay introduced by this, i think it should be fine since the messages are batched in arrays of 50 anyway)
only downside is you might have to reload after unblocking a user for obvious reasons
{
find: "compact:!0,messageGroups",
replacement: [
{
match: /(messageDisplayCompact:\i,)channelStream:(\i),(uploads:\i)/,
replace: "$1blockedChannelStreamOverwrite:$2=$self.filterChannelStream(arguments[0].channelStream),$3"
}
],
},
],
filterChannelStream(channelStream: any[]) {
return channelStream.filter(items => {
if (items.type === "MESSAGE_GROUP_BLOCKED") {
return false;
} else if (items.type === "MESSAGE") {
return !this.isReplyToBlocked(items.content);
} else {
return true;
}
});
},
huh
channelstream is on a parent of the message list
im fairly certain i tried to patch the same place earlier and it didnt work
it contains groups
yea ive tried to patch that but maybe ive patched it too late
yeah idk. did you patch it in the destructure line? or after?
i patched before it maps the messages
I originally was trying to patch arguments[0].messages._array but that doesnt work on first load
only on second+ load
but channel stream seems to work every time
even if you dont also patch arguments[0].messages._array

hmm
let v decide what to use 
oh yea i think you patch one or two functions before i patched the channel stream
thats prolly why its different
wdym
for what it's worth if I unblock and reblock with my method it instantly returns and hides the messages
no reload necessary
yeaa then imma do that
cause discord already reloads the messages since they're sent in MESSAGE_GROUP_BLOCKED groups
so im just piggybacking off of that by filtering that array
you patched the module that calls the module i patched when trying earlier
i have up on trying to patch that cuz it seemed to me as thatd make no difference but apparently it does 
ty
lol
it looks like this might also make the other patch(es) not needed?
well not my current impl but you could add another if statement to that list to check if the message itself is blocked
no need for multiple patches in that case
oh im already doing that with returning false on MESSAGE_GROUP_BLOCKED I think
the other one is less intrusive though
i think less intrusive and two patches > one more intrusive patch
what does the other one do
one patches messagestore so that completely prevents the message from being dispatched to the client and the other one just looks for the collapsed message and hides it
im not sure if this method prevents it from being dispatched
if it does then yea the other patch is essentially useless
well no im assuming this is after it's been dispatched
youre patching here and i patched inside of B.Z π
lol
i just assumed at that point itd have the same result
dont see how that's bad though
its not
i just dont think we need to remove a simpler mechanism if we can replace it with a more complicated one
thats all
hm yeah tested it. other 2 patches redundant now but if keeping them is better then 
wait whats even the point of the ignoreMessages setting
how is it any more ignored with or without that patch π€
one patches messagestore so that completely prevents the message from being dispatched to the client
but I can still use MessageStore.getMessage on the message
so...?
even after reloading. do I need to do a full restart
yeah the client is definitely still caching the message
so that setting isn't really doing anything
nothign wrong with that though as long its just cached and not processed furhter ig
it definitely has some sort of side effect that it does
itd be weird if it wouldnt
looks like it blocks new message notifications
could be worth changing the name then to be more clear but also 
dunno thats out of scope for my pr 
tbh that should be default behavior
the whole point of the plugin is to hide messages
why would you want a notification for a message you cant see
@elder plinth i found why my solution always had gaps
i never included a check for the message_group_blocked
π
oh is that it lol
ya
cuz you can make the patch waaaaaay simpler
i wonder if itll work if i go to my original patch again cuz that was even shorter
lemme check
that only works if you update isreplytoblocked to handle the channelstream format though
thats just more dependencies for a single function to handle
if u werent aware channelstream children arent messages. the messages are .content or .content[index].content for blocked message groups
wait have you tested that patch above
no way it works
it'll say cannot access g before it is initialized
thats the whole point of the blockedChannelStreamOverwrite:$2=$self.filterChannelStream(arguments[0].channelStream) syntax
the patch works but my even simpler patch from inside B.Z works too
mine patches the args passed to B
and inside of B i just filter before it maps the channel stream to messages
since that entire module only uses .map once the patch is pretty short too
I feel like this filtering of the channelstream might be introducing lag though 
you actually experiencing it or just vibes
just vibes apparently
for whatever reason discord takes its sweet time to switch channels rn but its not related to the patch
technically possible that messages._array needs to be patched too but I dont see the consequence of not doing that atm
for better error handling and avoiding minor bugs, avoid putting code inside patches
pure Javascript is incredibly fast
don't worry about it
very fair
the array filter is almost never gonna be your bottleneck
i was only worried cuz there was a 3-5 second lag in between switching channels π
what can be slow is the rendering aspects and if you misuse react
commit farming
How what
real
fuck I don't even know whats added anymore
I get yelled at everytime I try to remove shit π
so I just fix patches
also we do not add every plugin I swore against that just seems like we do because everyone wants everything lol
:P
hey that's me
ima just make indi do the hard work π₯ (make a list of plugins no one uses and remove them)
that project started only because I wanted mlenhanced and is only alive because I use it for a few extra plugins lol
equicord would've never existed 
ong merge it and all Ill shut down 
this is a joke I use equicord to test my plugins as well lmfao
see if people would use em or not
<img width="2114" height="719" alt="image" src="https://github.com/user-attachments/assets/3b440d4c-f65c-4a92-ab5a-2336711a172c" />
I am blind
Content
Could you guys add the new custom theme colors to vencord?
Request Agreement
- [x] I have read the requirements for opening an issue above
People are sooo good at ignoring the big loud "you're not a dev" sign
nookies review soon π
test review for project automation
@austere talon what category would a pr like this fall into?
https://github.com/Vendicated/Vencord/pull/3627
I was really confused why it changed like 15 plugins
maybe skip the error boundary if it's just a string
didn't merge dev
check now

PrefixSuffix
A plugin to add any prefix and suffix to your messages.
This can be use for simple things like automatically adding "-# " to your messages so they show as subtext, or for complex and annoying things to type, like ansi codes.
Features:
- Chat bar button:
- Left click can enable/disable the plugin.
- Right click opens the configuration modal
- Configuration Modal:
- Allows for convenient editing of the prefix and suffix
Example and Modal:
...
useless spam + text replace can do this
The "this user is wanted by the fbi" troll plugin
should I commit a regex warcrime or should i just delete it silently and act like it never happened cuz it wouldnt ever be merged anyway
This one should be ready and adapted to the latest changes again if I'm not entirely stupid
send a bump so I get an email
but ill review soon regardless
Share your email address and we'll bump you manually
nookies-the20thtimefor@email.spam
trust 
You fool 
whoops wrong one
I meant nookies-foremailspam@email.spam
my fault
wonder if spamming someone without an email server on their domain kills the servers π
If they don't have the MX record set up it won't get routed to the domain will it?
Is there any plugin (official or user) that lets you change the volume of discord's notification blip? Because that thing is so loud
π
added a nopendingcount fix not sure if thats how you all would want it tho π
old was s = l +a its now u = l.fractionalState === Z.a$.NONE ? a + s : 0 might make it match the .NONE to be safe not sure I did
you can probably just remove the = from the original patch
using the capturing groups should already make it specific enough to just have the a + b
younger me doing everything for a discord badge, i almost forgot i did anything here
it's been like that for a while
i was just going through some of discord's comments in their css
and found that
For the new tweaks feature i was planning:
Basically each small plugin that adds an objective improvement, and each plugin that changes some specific discord behaviour would be converted to a tweak
these tweaks would further be separated into enabled by default and disabled by default
Some examples of current plugins that would become tweaks:
- CopyEmojiMarkdown - enabled by default
- BetterRoleContext - enabled by default
CtrlEnterSend - disabled by default- NoProfileThemes - disabled by default
actually not ctrlentersend because it has settings
will there be a new tab for it or will it be an addition to the plugins tab
it would be a new tab
the motivation is to reduce the amount of plugins and also be able to add many more minor tweaks in the future
there are a lot of pretty niche small improvements that are kinda too niche to become plugins but we could just add it as a tweak
fixing discord bugs too
how will they be globbed
will they be something like
definePlugin({ tweak: true})
or their own dir
probably their own dir with one tweak per file
vencord will add conflicts
whar
two conflicting tweaks canβt be enabled
ykyk
like AddExtraNitroAds and NoNitroAds
things like that would probably be better suited as a BetterNitroAds plugin
defineTweak({
name: "No Onboarding Delay",
description: "Skips the very slow onboarding animation",
enabledByDefault: true,
patches: [...]
})
then it would probably just be a list of switches
isnt that just a plugin reskinned. how does this really fix the too many plugins issue
each one has a smaller scope + seperate UI for them
^
and also anything that most people would appreciate will be enabled by default
with a master toggle to disable all tweaks
the average tweak will probably just be a patch or two
this plugin is a very good example of what would become a tweak
its a super minor change that really has no reason to be a plugin
that's still the scope of the plugins that you would be converting to tweaks. nothing changes except a new ui for them. which I guess helps a bit
I guess you could think of them as plugins without settings
objectively good unconfigurable improvements

It's probably good on the dev side since defineTweak can have less shit available to use
you will d1e
husk
defineBalls()
nin0 going insane
I mainly think that having everything disabled by default is mediocre user experience the more features you add because it's not realistic for users to go through 200 plugins and understand what they do
it would be more user friendly if users get objective improvements for free while simultaneously reducing the amount of plugins they have to go through
having everything as plugin is kinda limiting because I always have to wager if something is worth adding a new plugin for
I rejected this plugin because it's too niche but I wouldn't mind adding it as a single switch (tweak) https://github.com/Vendicated/Vencord/pull/3200
(disabled by default obviously)
I think tweaks should have a setting dial realisitically
some tweaks would be plugins just for settings but also wouldn't be considered enough of a branch to be a plugin but more of a tweak
that kinda defeats the point
I see what you're saying now I read the explanation wrong
time to move the always* plugins to tweaks 
quite a bit of plugins could become tweaks just from the looks of it
so I do believe tweaks would be a nice addition to not having such a doom and gloom scroll to the bottom of plugins
how do users even find the right plugins in equicord π
I have no fucking clue
having to scroll through like 400 plugins
I dont enable plugins anymore
I wanted to paginate it but they were all agaisnt it
those stats on the top are kinda cool but they kinda look ugly ngl
they made it doom scroll but thats terrible as well
they were better I think
but I never fixed the classes and indis pc broke
I like the enabled plugins | total plugins
I was pondering the idea of adding very basic plugin telemetry that shares with the server which plugins you have enabled, so we can have sort by popularity
will also help with finding plugins for users but I'm worried it will end up biased because users will only look at the most popular ones so new plugins won't get a chance
I do like the "telemetry" idea but I also think people would go bonkers
i personally just went through every single plugin, do you think it might be a good idea to make the plugin tab have a way of having a kind of "discover the plugins" kinda thing?
plugin guide or something
Only Moh and Creations was iirc
defineTweak({
name: "Hide nin0.dev's messages",
description: ":trolley:",
enabledByDefault: true,
patches: [
{
find: ".messageListItem",
replacement: {
match: /renderContentOnly:\i}=\i;/,
replace: '$&if(arguments[0].message.author.id === "886685857560539176") return null;'
}
}
]
});
one of the other things I like that we have is this
it would collect as little as possible 
I feel like the stats encourage the whole "add as many plugins as possible" mentality lol
can just turn it off no reason to complain
Ignore activities from showing up on your status ONLY. You can configure which ones are specifically ignored from the Registered Games and Activities tabs, or use the general settings below
Authors
Nuckyz, Cookie
guh
i'm fine with the idea as long as no data is associated to users
isn't there a pr for it
yes
I would've used our bot (venbot's code) but I didnt wanna deal with it lmfao
it's been on hiatus because I plan to remake the command with componentsv2 etc
but i think users would go insane
I doubt most people would care
BD has the same telemetry
with bd it's not telemetry cause you have to press download on their site
I think having a disable option is key
but for vencord since it's all in your client we don't have that data 
I mean that is legally required anyway lmao
i would say just don't set out an announcement about it and have an easy way to opt-out in order to calm people who love setting things ablaze
that's a very different thing lol
poll_question_text
remove krystal prog role?
victor_answer_votes
2
total_votes
3
victor_answer_id
2
victor_answer_text
n
it's the same thing just in a different way π
it's still telemetry in the exact same way
not even close imo
if you do telemetry and we add it ill forward it to your server if you want (you prob don't)

3 users are using GoodPerson
telemetry I would use would probably be just to see how many people use a certain plugin
and whats not used to delete it
@austere talon userplugin telemetry and venbot updates a sticky in the plugin thread
also it should ignore userplugins
ofc it's not directly comparable but it ends up in exactly the same data about you being collected
ofc it will
what would we even do with that data
telemetry dead on dev π₯
but if you want I can pass our telemetry of vencord's related stuff to yours if you want (if I can figure it out)
prob adds another 2-3k
does equicord have telemetry?
nop
I mean when this all happens if it does
ill probably set it up as well for plugin count to see whats not used
if the telemetry is open sourced 
I 100% could switch back to vencord without issue rn π₯ (ignoring asar)
naturally

I was so happy when I saw venbots source
I wanted that discord reporter log π
I just always think you all are mad at me
why

it's so easy to make
it's mostly just github workflow 
just seems like the default response to me
did you get it working in your bot
mhm
i had a version before venbot
everything I make is open source
everything vencord uses is too
I do when I clean up the code
if its not clean its private
all equicords stuff is pub tmk
i think the only things that i have private are the things that would dox me
the one thing I also get scared of is I dont wanna violate the license because im dumb
I've done so much research to make sure I don't π
is equicord the one where I opened an issue complaining about license violation
or was it something else
pretty sure that was something else
I just dont want that to be us
you did an issue comment on one of my commits with badges btw thank you for that
-# Me and Inbest made that new goated version
I remember thats what made us just revamp the whole plugin
it wasn't actually that bad
as insane as equicord plugin selection is itβs the sanest fork ykyk
wrong idea
Im talking about fetching the donor badges
still sorry 
vencord badge api literally serves terabytes of data each month
vro im giving lore

sometimes create stuff thats all
vast majority is cached by cloudflare
Horror
do you codereview prs and ask people to make changes etc or do you just merge if its alright
I review all the prs
I ask for changes or make them myself if its good enough as is ill merge and fuck with it then or later
I would never not review the prs at least π
im not insane
how much time do you spend
vencord gets way too many prs I can't keep up
but I'm also perfectionist
yeah
make other people review
depends on what I'm reviewing
we've been through this dozens of times
it can be a couple of minutes to days
I normally review the prs as soon as I see them
normal takes like an hour or so without fixes
review every vencord pr thsnks

both mine are good trust 
π
statuswhileplaying is weird
alwaysexpandprofiles is good tho
the patch ehhh
I realized I wasnt matching clydes type of popout
so it was ignored
who cares about clyde π
ah
but bot was changed an had a very bad find
so I matched all 3 with view profile
and just did that
rather than having a terrible find and 1 missing
https://github.com/Vendicated/Vencord/pull/3618/files
also vee please don't do me like this
why
bouta make me go replace all of this on mine π
it's will just be find and replace (or get ai to do it)
vee is a vibe coder
most should be fine I agree then theres the random selection of 25 plugins
most prs could be closed imo
either already added already has a pr or just niche
originally I only wanted to replace the components for the settings ui so it stops breaking
but then I realised it's way more powerful and nicer to use than the form components
I agree with that using our own would be way better
https://github.com/Vendicated/Vencord/pull/3630
this should definitely show mute as well
just to give a heads up on what the users (un) mute/deafen status is
yop
How does this handle server mute/deafen?
NSFW voice channels also need an icon added like private voice channels are
should turn red
π emoji trust
does discord not have a function/component for the correct voice channel icon
eg: stage/private/nsfw
Im 100% compiling a list of plugins that are niche or dont really seem like vencords plugin style
in vencord prs or equicords
I just know it shows when a Vc is locked/ private but not when Itβs a stage or nsfw vc
π
you blow my ears off somehow

@austere talon here is my list
in no specific order or anything
I also didnt look hard or at code really

just put mantis pr out of its misery π
repgrep is amazing for mass replacements, i had to do a couple of such mass renaming, and it handled it really nicely
scooter also works
from my experience it's the best performing translation tool out there atm, although most features are paid. It uses an LLM which lets it pick up on context or language quirks better than most translators can
https://help.kagi.com/kagi/company/
https://help.kagi.com/kagi/ai/translate.html
https://help.kagi.com/kagi/why-kagi/ai-philosophy.html
re:
startpage is a pretty nice search engine
the ones I counted as niche are the ones I dont see people use much at all whether its the same plugin or close some are excluded
#1806: definitely not niche, there has been resounding support to get this merged over the past 2 years both in the PR and in this discord
prob looked at it wrong
I thought it was just os 
cant even update branch
WHAT
I didnt touch anything
hold on
it doesnt even exist
tf
am I loosing it?
I merged dev
I have no clue why the button is missing
it should be somewhere in sidebar
the sidebar in question
is it cause it's from a branch in your equicord repo which has non-default security settings?

and tbh I would probably pr from somewhere else than equicord
you cant enable maintainer edits on PR if the originating branch belongs to an org
you're kidding
π
org accounts enforce their own permission policies
this is why vencord isnt in an org 


good job ren
ren feels wrong
I started helping out because I fixed my git again
i gave it a more accurate name
cause it doesn't fix FakeNitro at all
Now testing! 
i need to implement reporter for prs guhh
Bad Patches
NoPendingCount (had no effect):
ID: 222059
Match: ```
/(?<={unviewedTrialCount:(\i),unviewedDiscountCount:(\i)}.{0,300}\i=)\1+\2/
No issues found
Seems like everything is working fine (for now) 
would be better
alr
although if they have existing feedback / convo make sure to mention the old pr
This plugin replaces user or bot's profile popouts with the full modal
Moved from https://github.com/Vendicated/Vencord/pull/3601 so it has proper rules
This plugin automatically updates your status to the status you have selected in the settings of the plugin when playing a game. When the game closes the status will revert back to its previous state.
Moved from https://github.com/Vendicated/Vencord/pull/2988 so it has proper rules
is there actually a use case for anything but DND?
Not really
Figured if I'm doing it make it customizable
cause "Auto DND when launching a game" could just be a tweak
My guess was if someone wanted DND or invis
i just have this on vesktop 
if discord is unfocused for more than 1 minute it puts me to idle
similar to the mobile app
That is fair I was planning on making them both tweaks which would kill settings
(well tweaks arent set in stone yet)
One day
b7f19bb NoReplyMention: add role whitelist / blacklist ... - Unionizing
77b016d ReverseImageSearch: add Bing (#2793) - Unionizing
1d00ba4 fix patches for Experiments and Vencord Toolbox... - sadan4
b6e96a4 MemberCount: also show members in voice (#2937) - ultard
e857f68 ShowMeYourName: respect streamer mode - Vendicated
Anyways you think I should keep the setting or just drop it?
english or french no more apparently
@elder plinth are you still gonna remake MemberCount with svgs?
do
it was not my plan to no
i thought u were gonna 
I dont look at that corner of the screen often enough to be arsed
i might do it then
it's currently css. which causes issues occasionally do to subpixel rendering
so replace them with svgs
π’
got one done for you
no gray circle emoji but surely you can just change the color of that
function L(e) {
return (0,
r.jsx)("mask", {
id: e,
maskContentUnits: "objectBoundingBox",
viewBox: "0 0 1 1",
children: (0,
r.jsx)("circle", {
fill: "white",
cx: .5,
cy: .5,
r: .5
})
})
}
discord:
there you go
also lowkey I feel like the new voice users icon and number should be blue not gray but thats just me
arent those literally just circles
no clue
nothing else to then
didnt test em yet
don't hardcode colours
what's the point of the mask on the online?
turning it into a circle from a square?
both masks seem useless
I didnt have it like that originally
online was just a circle
but offline idk how to do with out a mask
you want fill=none and then set stroke
didnt you say that code is what discord uses
it literally is a direct rip basically
that isnt right?
its using --status--offline
oh I might have the im not colorblind plugin enabled
yes you do
mm yeah
Why does it use offline for total member count anyway, it doesn't subtract the online members from the figure
true

oh god
discord does it too
stroke width exists
yeah keep it colorblind friendly
what width we thinkin?
whataver makes it look like the real one
tbh you should just use discord's impl idk why ven is having u make it custom
just realised it isn't even aligned π
the circles are top aligned
a lot of stuff in discord isnt aligned
I made the icons slightly bigger
there 18px now
I take that back the only 18px is green
for whatever reason 18px and its a good size for everything the rest are 15px
Use a circle svg instead of css for the dots
I think it looks better if it's slightly smaller
try to keep it the same as before
Alr
center align plz
old first new second
bruh that comma ruins everything π
maybe hit the icons with a 1 or 2px margin-bottom or smthn idk
you mean top?
the comma annoys me anyway cause...
top margin would just push them down further
you want to push the icons up
oh you want the mic margin?
fuck u
to account for the comma
im lost
<3
where are you from morphy
ren plz
you seem like you're from eastern euripe
maybe romania
you dont seem like an american
??
hes american
ya know what
I could never forget it π
in today's climate thats a compliment
that's true
u dont wanna be from romania
the comma causes the "center" to be lower than it should so the icons are lower than they should be. so to counter act that you add bottom margin
just set line height bruh
well idk if being called romanian is a compliment or not but being called not an american is
line-height: 1.2em;


