i think youve forgotten the existence of UserBG 🙏
#👾-core-development
1 messages · Page 198 of 1
Bad Patches
None
Bad Webpack Finds
None
Bad Starts
None
Discord Errors
None
Bad Patches
None
Bad Webpack Finds
None
Bad Starts
None
Discord Errors
None
Discord Account
No response
What happens when the bug or crash occurs?
when clicking a persons profile whom has lots of roles, the profile just gets stuck, you're not able to scroll up or down, leaving to have to use the right click menu if you want to try and add them as a friend and such.
What is the expected behaviour?
I expect ShowAllRoles to handle this kind of probl...
aw hell naw
Link isn't real
it's <a>
Isn't that not closed anyways?
Shouldn't it be <Lionk href="" />
look 2 lines after

I have the reading capability of a plugin request opener/j
It should be on react
For dynamic sites I think Link is being used, tho it might not make sense in this case
Tbh not a terrible idea
how many percent is this done?
then host it
the pinned post in question
she said to open a modmail to get access
This means that your repository needs to have its index.ts file in the root. One plugin per repo! why do I feel targeted 
I assume I wasted my time here as FakeNitro had a lot of changes recently which would require me to go over this agin
Any updates? This is such a simple plugin and I resolved every review remarks a few minutes after they were sent..
how many percent is this done?
not necessarily a %, I work on this on personal time and add features if I feel they're necessary. just wait for pr to be marked ready for review
how was it supposed to have worked before
it only checks userplugins/plugin/index.ts
not userplugins/nico/plugins/index.ts
that's fucking awful
vencord could support nested folders so you can clone entire repos
but also people should not have to install all your plugins just to use one
vencord might update userplugins for you
the entire reason the new rule is there so that all userplugins are git repos you can pull
instead of downloading files manually
in the future i might make vencord ignore userplugins that are not git repositories
to enforce proper installation that allows updating them
with some way to still load ur own plugins without a git repo
HORROR
oh i should probably read first
simple, add another plugin loading folder thirdpartyplugins/hj
cd userplugins/coolplugin
git init
Done
fair
but then vscode gets all pissed
i think
nah
i wanted to use submodules for userplugins
but submodules don't work in gitignored folders
@austere talon i just thought
how do we do localisation for userplugins
or is that a "we're not doing it"
cuz atm everything is in one folder at the root
plugins could probably have a locale folder now that i think about it
i think i can make i18n ally support that
we don't
so only first parties get localisation?
why in the world would anyone localise their tp plugin
not happening lmao
powercord allowed plugins to localise and there were only like 2 devs using it and they just did one locale (their native language)
df32e8d fix missing space in MoreKaomoji (#2625) - Jacob1Tm
d61a930 Developer Documentation has moved to https://do... - Vendicated
32c2128 ReplaceGoogleSearch: improve search engine sele... - OtterCodes101
3688c7e ShowMeYourName: compare username & nick case in... - Bloofield
cd205b1 ReviewDB: add to context menu in new profile ui... - programminglaboratorys
that was messy :<
firstMessage
This plugin allow see a first message timestamp was sended or recived to user by chat id
✨ DOM manipulation ✨
(maybe not i just quick looked from mobile app)
why
git branches should be fine IMO but I have asked about it
i will add you to ticket
horror
is there a fucking permission required to add to ticket
per thread i think
me when snowflakeutils
me thinks the idea of plugin vendors are still a good idea now
sended is crazy
dom manipulation is

why point them to the rules then close it
they can remake pr when they don't use deranged practices imo
you can ping people in modmail to add them

You will hop on MessageLogger
and see
I tried
and discord was like nuh uh
actually only mods can yeah
because the plugin requires a full rewrite anyway so it's better to resubmit from scratch where you can get fresh reviews
- it's clear they didn't read the rules in the first place
they opened 3 prs
3 rule breaking
nvm
yop
so sad
the answer to "Should I be using DOM Manipulation in my Vencord plugin?" is
Short answer: No
Long answer: 99% of cases, no, however Vencord's core does it for things like CSS. If you MUST, do it within React with a ref for very very niche cases
i fixed it
This pr has been reopened because vendicated closed the other pr for almost no reason that makes sense (#2648)
Just so you know vendicated, we are using github.io and we cannot do anything with that!
Also, why was the PR closed? You’ve mentioned the plugin rules but that didn’t explain much why it got closed.
Anyways,
This plugin allows you to add custom profile effects on your Discord profile
You just simply create an effect, submit it and wait for approval
Everything a...
we are using github.io and cannot do anything with that!
bro really just reuploaded some else’s pr for them
love how they pretend to not know why it was closed and also know why it was closed
No untrusted third party APIs. Popular services like Google or GitHub are fine, but absolutely no self hosted ones
do not open deliberate duplicates or you will be blocked
how much we wanna bet they are going to reopen that pr?
Popular services like [...] GitHub are fine
??
genuinely confused
aka: come back when you have a reputation
just look at that persons prs https://github.com/Vendicated/Vencord/pulls?q=is%3Apr+author%3Ahappyendermangit+is%3Aclosed
GitHub
The cutest Discord client mod. Contribute to Vendicated/Vencord development by creating an account on GitHub.
this is different because they load urls from json and fetch them in the plugin
they could put arbitrary urls in that json
oh what the fuck
do you mean
arbitrary code or just "what if secretly ip logger"
probably latter
the latter
it’s exactly the same, he just reuploaded a pr for someone
unless they find a vulnerability in chromes image parser

isn't decor cooking profile effects lmao
i just realized what you mean,
i hope not
the one thing I hate about decor is that there's no "unlisted" privacy option for you to steal other people's decorations
blackbox is usrbg
no offense meant
but this user just seems to make low effort / rule breaking contributions in an attempt to get a contributor badge
or do they already have
they already have
also “serstars” is a co-dev on almost all them makes me suspicious on the fact that they prolly didn’t actually write any of them themselves or that’s their alt-account or somethin
idk likely both collaborate on plugins
lol
constants.ts: Lines 445-448
HappyEnderman: {
name: "Happy enderman",
id: 1083437693347827764n
},
also
@austere talon is putting links to your donation site allowed in plugin settings, if this plugin requires a custom server that you operate yourself (not planning to do that just wondering)
Decor does it afaik
reviuwerdb also does
huh it doesnt
plugin settings
all their PRs seem to be just to get the contrib badge
yet they already have 
sure
but unless you provide some perks i wouldn't expect anyone to donate
but makes me think that one or the other is doing most of the actual work (prolly not the ender dude) and their just slapping their name on it
true
guh idk
tbh we shouldn't speculate on something we don't know and probably won't know
they got it after one of their prs got merged 
actually it was someone elses pr
it was some else because that one has better formatting then their pr, just look at the their version
why were they even added to authors
they state it was an edit, however I dont see anything in common
for that badge i’m guessing
prolly a friend of theirs that just them slap their name on it, i would guess then
it says it's an edit of their pr, but I dont see anything in common really
honestly i’m just going to leave it on what Cynosphere said
petition to block people who repeatedly submit prs that are obviously just to get the contributor badge (also pretty sure I saw this when that pr was opened)
one sec let me try to find that
vencord has the ability to keep someone as a dev but hide the badge 

So just validation for what the great the Cyno said
yes
💀
at least they admit it
nor are they granted automatically
yes, they are-
they need to add themself to the devs constant
I take automatically as automatically giving it to them once their pr gets merged, even if they didnt add themself to the devs constant in the pr
photoshop pro over here
me when no updates
linux has vendor
that’s why dev edition because you get the update a few month before other people and you have more control over it what you can with it
VendorOS when vee?
isnt android just linux
android is based on linux yes
MiXplorer reference
i want to add vendor to vencord
wth is /vendor/libdrm.so
token logger stuff don’t worry about it /j
is any of this malware
pixel screenshot editor my beloved
What if they do a check inside of plugin smth like: "image source should always be from github.com"
it really was just an edit of his pr
lol
btw at that time I didn't know there's even badge for it, I thought u'll just have ur avatar in plugin settings 
vencord experience
Before I explain what this does, here's what it doesn't do (for people who stumble upon this PR)
This does not and cannot bypass permissions for that menu (I have tried before)
This adds a few more options to ShowHiddenThings, related to the Members page, visible as a channel linked in the sidebar, and in the settings. Unfortunately, Discord adds arbitrary restrictions based upon if your server is a "community" or not to decide where the page is visible.
Options
Show Members ...
yes I know titlegore much
Oh i've wanted this for a while
@rugged spire i think it's possible to do a similar thing with the roles page, you can view them and their permissions regardless of permission
I might investigate it
Pretty sure someone dropped some webpack override code in #🗳-plugin-requests to do this
Oh word
fun fact
I was gonna originally make the redirect to channel page button always shown
because I (naively) assumed the redirect icon was its own button to do that
index.tsx: Lines 59-68
{
find: 'location:"UserMention',
replacement: [
{
match: /user:(\i),channel:(\i).{0,400}?"@"\.concat\(.+?\)/,
replace: "$&,color:$self.getUserColor($1?.id,{channelId:$2?.id})"
}
],
predicate: () => settings.store.chatMentions,
},
I can't stop thinking about correcting those. WHY THEY ARE ARRAYS IN THE FIRST PLACE
Multiple replacements in the same module
index.ts: Lines 63-65
if (!Array.isArray(patch.replacement)) {
patch.replacement = [patch.replacement];
}
you can always just uh 
inb4 added themselves to devs
my life is- yeah I knew about this already or I guessed
ok not the case
insane™️
I am unsure if I am feeling satisfied or not
I am going to watch netflix 
the comedy tho
block them and hide the contri badge on their profile
hes more annoying than funny tbh
this is a funny take considering you are terminally obsessed with #🗳-plugin-requests to the point of being annoying to maintainers
wild
i mean, they got mad at me for pointing out the checklist that should be auto-closing the issues anyways
how come?/genq
because they'll somehow understand it was that and come back 2 minutes later with the same bullshit
if you don't lock them they can complain there
fair
Don't they get banned from posting requests
They can be but not automatically
read from here down
#🗳-plugin-requests message
2 line patch that effectively does nothing
Yeah, I read that. I just thought they were talking about issues that had the funny check box checked
discord update detector
VencordTab.tsx: Line 96
note: "You need a theme that supports transparency or this will do nothing. WILL STOP THE WINDOW FROM BEING RESIZABLE!! Requires a full restart"
It should only show the on Windows
It's at least been broken since the Jun 18 Doomsday, and nothing is logged to the console during or after the decryption attempt. I don't know enough to confidently debug the script to find where the problem is. Using StegCloak in Node REPL decrypts the message as intended.
127 switzrr@switz-nyarch ~ % node
Welcome to Node.js v20.15.0.
Type ".help" for more information.
> const StegCloak = require('stegcloak');
undefined
> const stegcloak = new StegCloak(true, false);
undefined
>...
Closes https://github.com/Vendicated/Vencord/issues/2645
Discord seems to have changed embed fields. I compared them with embeds from bots and took fields from there, and now it works fine
Before fix:
After fix:
given it took so long for someone to bother fixing this i wonder if we should just remove the plugin tbh
I seriously doubt anyone even uses it
At least 2 people
basic form of hiding activity
how to do something people wont see
- make a server
- dont invite anyone
- profit
physical access joined chat
how to deal with that
- remotely turn on the computer
- profit
how the ever living fuck do you remotely turn on and unlock a computer you dont own
exactly they never expect it
💀
anyways, it’s pretty easy to remotely turn on a PC. I do it quite often when I’m a over the road with my computer
my computer
emphasis on "my"
and if someone else has access to your computer, you obviously dont care that much about hiding stuff.
It’s still the fact that I can turn on and shutdown computer, remotely then after that all you need to do is crack the password and with the person that’s pretty easy
my entire point though is that its your computer
you cant argue against my point by saying "but just ignore that tho"
WHAT DID YOU COOK
what
no
insane
real
what is vee cooking 🔥
i love DevilBro
he takes 2.5k lines to do something u can do in 100 lines
what the fuck
well more lines once i add ui and stuff
but still nowhere near 2.5k
true
port of DevilBro’s editusers ig
edit users? I tho its useless
like 100+ are just for dependency ensuring
no its highly requested
i dont think it is
It can have some uses
my motivation for making it is @opaque pewter
why
why the fuck do you have 75 mutual friends with a single person
I DIDN'T EVEN NOTICE THAT

its gory? I tho its beautiful (just me? alright I will shut up)
I don't even have 75 friends total 
this my best I can find within my friends I think
i have 91 total friends, my most mutual is like 15 😭
poggers
I found a slight bug, in a role category with only 1 user, whom is blocked. It shows the category empty.
That is a description indeed.
Whom
get that'd loser

Bad Patches
None
Bad Webpack Finds
None
Bad Starts
None
Discord Errors
None
Bad Patches
None
Bad Webpack Finds
None
Bad Starts
None
Discord Errors
None
Venware my beloved
what did i just see
rm -rf message by github
(Uses a lot of code from platform indicators)
Adds a display to the badges, messages, and member list to indicate your relationship with the user.
So far, theres blocked, and friends.
If anyone thinks of anything else that should be added, please let me know.
Just a few nitpicks, overall, pretty well written.
Add alt text to these 3, rather than just "image"
Why define a variable just to use it once?
pronoundb still needs to be fixed for new profiles
yeh I kinda prefered the old too
I never had issues with the old
combine the two, I am thinking of gray button with text, and the icon aside the text
I don't see much use for that with the new profiles already having a "friend added" icon
i want to see what they do without hovering
normal buttons are fine as long as there's text
but this is really bad
also it clashes with the buttons on the bottom
they also had an indicator before.
also, this is in the chat & the member list as well.
maybe the badge isnt needed, but ya know.
also, theres no indication in the member list if a user is blocked or not.
niche
this can also be said about platform indicators lol
yea, but still, thats four other buttons that just look out of place and i have no idea what they do
yea, i saw that
remove the view log button, and move the notification settings into the settings area
just give the buttons labels
read your dms damn
what
its 1 dm
its 7 and 2 requests
if you don't like the new quick actions, then suggest an alternative cause the old ones also were not it
the old ones were better
maybe like this
yeah exactly
now edit messages 
i should cherry pick those webpack improvements onto dev
happy?
ty bestie
That is so good
🧌
But now it looks weird because it needs more buttons to level them out
oh good catch
impossible to fix
why do you think i used icons
just add another button
The famous do nothing button
Actually yes that would be funny
just a button that does nothing at all
Align-items: center
Justify-content: center
Flex-wrap: wrap
which you mean
that would work
makes it look better
then u don't have evenly sized pills
looks awesome
True
that’s a small price to pay
still would make it look better tho
I think it would still look better that way
smaller pills don’t matter just needs to look even
feel free to send a pull request 😉
You will ignore for 3 years
no
OK then I will
Can edit client theme be added to quick actions?
Ah
how about a donation button or something
nvm
it’s above
in love
stupid question but would it be possible to make that accessible to plugins similar to toolboxActions?
would be fun
am I dumb, there is no error in logs
LOOKS SO MUCH BETTER
.vc-settings-quickActions-card {
display: flex;
flex-wrap: wrap;
}
i like that but the edit clienttheme button need to be in the middle
YES
PERFECT
i dont think it does
well my ocd does
I will make the buttons evenly sized for you then
one second
okay im done now 
not that
i think it looks bad if they're inconsistently aligned
then add a another button that does something to even them out
Can you force push an empty commit? just to annoy some people
i already told you thats not possible
thats too bad
look at changes in webpack folder https://github.com/Vendicated/Vencord/commit/b333deb7312107a629920737a0e9cd56936a35a5
yeah I saw
cleaned up webpack methods code and made some more functions able to search by regex and use canonicalizeMatch
i needed it for mapMangledModule
I'm gonna add the regex functionality later
to my refactor
didnt think we would ever need it lmao
was an array of code not enough?
well
318661: function(e, t, n) {
"use strict";
n.d(t, {
Of: function() {
return c
},
ZP: function() {
return l
}
}),
n(47120),
n(470079);
var r = n(442837)
, i = n(594174)
, a = n(251625)
, o = n(255564)
, s = n(621853);
function l(e, t) {
return (0,
r.e7)([i.default, s.Z], ()=>c(e, t, [i.default, s.Z]))
}
n(484459);
let u = (0,
a.oH)((e,t)=>new o.Z(e,t));
function c(e, t) {
let[n,r] = arguments.length > 2 && void 0 !== arguments[2] ? arguments[2] : [i.default, s.Z]
, a = n.getUser(e)
, o = r.getUserProfile(e)
, l = r.getGuildMemberProfile(e, t);
return null == a || null == o ? null : u(o, l)
}
},
not very robust
also technically regex should be faster
hmm no idea of it's not faster
string searches are super fast
I'm not against adding the support for regex but I would still use array of strings when possible
oh i see that's 3 matches
I thought it was one for a second
makes sense
how is it so fast
uwu
oops
average github pr
hi! please don't open spam prs with titles like "sajidfjiasdjiodjioasjiod" or you will be blocked!!
you tell em girl!
schizophrenia
The pipeline

quick find some funny example to reply with that you SHOULD do
hi!!!! please DO go insane like I did or you will explode
GitHub
CSS IS A FUCK
5 SECOND LAYOUT REFLOW
2348923 WASTED WATT HOURS OF LAPTOP BATTERY LIFE
Content
I saw there is a pluging for Apple Music RPC on GitHub but cant find in Vencord
Request Agreement
- [X] I have read the requirements for opening an issue above
vp applem
Discord rich presence for your Apple Music!
Authors
RyanCaoDev
Traits
desktop only
you are most likely on discord web. this plugin is only available on discord desktop or vesktop
@nimble plaza fyi the plugin is hidden if you're not on macOS
you should include that in your reply 
oh
oh
native.ts: Lines 94-113
const playerState = await applescript(['tell application "Music"', "get player state", "end tell"])
.then(out => out.trim());
if (playerState !== "playing") return null;
const playerPosition = await applescript(['tell application "Music"', "get player position", "end tell"])
.then(text => Number.parseFloat(text.trim()));
const stdout = await applescript([
'set output to ""',
'tell application "Music"',
"set t_id to database id of current track",
"set t_name to name of current track",
"set t_album to album of current track",
"set t_artist to artist of current track",
"set t_duration to duration of current track",
'set output to "" & t_id & "\\n" & t_name & "\\n" & t_album & "\\n" & t_artist & "\\n" & t_duration',
"end tell",
"return output"
]);
pretty sure the windows client doesn't let you do any of this
applescript horror
Im using Win11 Vencord
Oh okay, is it possible to see it on Windows?
lint is not passing btw
Bad Patches
None
Bad Webpack Finds
None
Bad Starts
None
Discord Errors
None
Bad Patches
None
Bad Webpack Finds
None
Bad Starts
None
Discord Errors
None
W
bda0e18 browser: increase minimum browser versions - Vendicated
169edcb improve support helper - Vendicated
086c31c new plugin ShowAllRoles: show all roles in simp... - Luna-devv
2d570a5 friendsSince: add support for new profiles (#2623) - Chicken
8a7c0d7 WebContextMenus: implement context menu for tex... - Masterjoona
99b41db improve settings ui (again) - Vendicated
b5fdb43 Merge branch 'dev' into immediate-finds - Nuckyz
f6147c6 Merge branch 'dev' into modules-proxy-patches - Nuckyz
8abb68c Merge branch 'immediate-finds' into immediate-f... - Nuckyz
0d0c63f Merge branch 'modules-proxy-patches' into immed... - Nuckyz
fixed lint
[Vendicated/Vencord] branch deleted: reviewdb-simplified-profile
[Vendicated/Vencord] branch deleted: permviewer-simplified-profiles
Bad Patches
None
Bad Webpack Finds
None
Bad Starts
None
Discord Errors
None
Bad Patches
None
Bad Webpack Finds
None
Bad Starts
None
Discord Errors
None
i was just gonna fix it in a later commit tbh xd
but that's fine too
maybe .dev plugin?
thinking better wouldnt it be better to have this enabled by default
less errors for users to send us screenshots
with this disabled
Incomplete string escaping or encoding
This does not escape backslash characters in the input.
WHY
we already tell them to select error only
yeah and the plugin removes useless errors that always happen
like the font one
can u add something that stops the spotify store from logging ur spotify name and access token
good idea
an option or default behavior?
i love when people share debug log and i have to tell them to deauthorise discord from spotify cause the screenshot included their spotify access token 😭
default
maybe option
but default enabled
Incomplete string escaping or encoding
This does not escape backslash characters in the input.
I KNEW IT
okay so then I should make the plugin enabled by default and not dev only
yeah it's my id
found it
this has too many logs with the account id, I will just disable the logger
it logs username and token
ye somewhere
Devices updated for {name}
oh it's account id
why is my account id my username
I disabled it completely regardless
god
actually we cant fix that i think
the fucking error
might be able to put an error event handler
okay so its here
hmm maybe not
it does have a onerror
that's so stupid
shitty hacky fix idea:
in electron is it possible to specify default settings for devtools?
ie: console filter
anyways if we dont have any other ideas I'm merging the plugin soon
i looked into this previously, i dont think so
your best bet is a devtools extension
Chrome for Developers
How to create a Chrome Extension that adds features to Chrome DevTools.
what if we made patch helper a devtools tab

scary
I once thought of making a Recovery tab
independent from vencord
would have things such as disable all plugins
wait we could make a devtools tab for updating vencord
clear themes
okay
will be officially added if good
wait idea
what if we patch the spotify ws url to
?aaaaaaaaaa<repeat>&access_token=blabla
so console shortens the url

since we can't hide the spotify token anyway
why now that I merged 😭
i don't think it works
aw
i didnt know u were gonna merge instantly 
💀
they gon force you to learn a new language 🙏
you mean that I should have added that?
i mean sure if u want
i think its because i have that one dev option on actually
i dont remember if rpcserver spams like that without it
just kidding it does
guess i was hallucinating again 
this stuff
its either super noisy or not at all
depends
i just have it filtered all the time though
Send filter
esbuild 0.15.18 (released 2022-12-05) doesn't support RegExp literals using the v flag. may i update our esbuild dependency to a newer version?
it looks like https://github.com/Vendicated/Vencord/pull/2446 moves the bounds from ^0.15.18 to ^0.21.5, but that topic branch includes a non-trivial amount of other work
i'll go for submitting a standalone "esbuild": "^0.23.0" then
@Vendicated Requesting review for a pretty minor change set. The .m{j,t}s{,x} linting added here is useful when modifying build scripts, which is why #2446 also adds that.
actually, @limber skiff, since you're around, could you review #1661 please?
I'm gonna let vee see it, but if they dont I will do
it really is just (all in scripts) pnpm run for future-proofing, factoring out the lint scripts in the style of "lint": "pnpm run lint-user --ignore-pattern src/userplugins", adding test-user and testWeb-user which call -user variants when appropriate, and adding .m{t,j}s{,x} to the eslint extensions
if you're strongly opposed to linting src/userplugins then you should be able to cherry pick just the last commit just fine
@Vendicated Sorry for bugging you again, but can you please take a look at this?
Very funny - 1. why would you have that many roles - 2. I haven’t really tested it with that many roles to be fair and I haven’t large screens - 3. question is how to solve this, scroll(?), limit the amount of roles (?) like 50(?), add a setting to set the amount(?)
Adding this to your quick css makes the roles scrollable if the roles list is longer than the height of discord
[class*="userPopoutInner_"] >
[class*="body_"] {
overflow-y: scroll;
}
[class*="userPopoutInner_"] >
[class*="body_"]::-webkit-scrollbar {
display: none;
}
https://github.com/Vendicated/Vencord/assets/71079641/98d0d937-aa46-4911-b63f-0a1a04958901
uwu
jumpscare
turns out that #2446 doesn't handle concurrent esbuild logging robustly. kind of a pain to set up but got that working on 0.17.18 at least, along with proper summary emulation.
buildWeb --watch seems weird but i think it was that way before?
oh, @dire trench isn't here
okay rest of the versions done yay
[!NOTE]
This PR is a patch stack. Do not squash these commits. This patch stack depends on:
+
I wanted to use RegExp literals using the v flag, but esbuild was too old. Whee.
The build scripts are first moved over to TypeScript (.mts) so upgrading with esbuild is sane. VS Code wouldn't show errors for the previous .mjs files for me, at least. tsc will catch future type errors now. (I can split this out to a separate PR if you really want.)
esbuild is then mov...
Very funny - 1. why would you have that many roles - 2. I haven’t really tested it with that many roles to be fair and I have large screens - 3. question is how to solve this, scroll(?), limit the amount of roles (?) like 50(?), add a setting to set the amount(?)
i was thinking scroll but like the same way as if you were hovering over a long status bubble within a profile and it just shows the rest of the content. here's an example of how i copied the same behavior but with my about me.
...
Is this gonna be pushed as a plugin for Vencord? Im waiting for a year already <3
when it doesn't freeze all of devtools
what if we made it so that it splits all modules in dev builds
so we don't have to deal with that massive chunk anymore
why?
devtools doesn't like so many source files
okay

as someone who has used that for long enough that I ended up heavily forking it
there is (probably) a reason why it hasnt been merged
not that it is a bad plugin, it's just
- very complicated
- doesnt actually hook around Discord's history wrapper
last time i used it, the main issue i had is that ctrl+click doesnt open in new tab, and there is no option to opne all channels in new tabs by default
mmmm
might fork it myself and add those features
also the fact it feels laggy just because of expecting tabs to be instant
i felt no lag using it
isnt there an insane 500 line util file dedicated just to tab management
also this favorites section superseded it for me
problem with favorites is uhhhhhhhhh
you cant search in them
i need to Shove CSS into it
respectfully why did you apply a windows icon skin on linux
W > L
cursed
!remindme 10h look through this
Alright @verbal pumice, in 10 hours: look through this
💡 Did you know you can set your timezone with "!timezone set"?
all I ask is
don't steal my code that tried to do it it's Also Broken
yeah
wasnt going to
i just gave up on trying to do anything more w the plugin after horribly failing to add keybinds
[!NOTE]
This PR is a patch stack. Do not squash these commits. This patch stack depends on:
+
For ease of authoring patches:
- PatchHelper's find now canonicalizes
RegExps (e.g./\i\.foo/) - PatchHelper's find supports single & double quoted string syntax (while still allowing unquoted strings)
- PatchHelper's match supports
RegExpsyntax - PatchHelper's match supports single & double quoted string syntax (while still allowing unquoted strings)
- PatchHelper supports ...
tf is a patch stack
it's probably a "patch series" in gitworkflows(1) terminology
what do you mean "probably" you made this PR 
alternatively, "atomic commits"
i don't do enough git or kernel workflow stuff to feel super confident about those terms?
real
git scary sometimes
what in the world is the point of this
esbuild is then moved forward one breaking release at a time, to make both authoring and reviewing easier.
the changelog doesn't group breaking changes together either
because esbuild is something i really didn't want to break, this is meant to make reviewing with the changelog open easy
you can see what i thought needed changing for each set of breaking changes on esbuild's side
also if something did break that i didn't catch you can have a better experience bisecting
git bisect cant fix merging in fucked up code that was force pushed over on the remote
it is 
i wanted to reduce the pain of debugging a broken build if that happened. the commit list is longer but you can see that most of them only change 2 files and skip them with a better amount of confidence in manual review once you've confirmed that the breaking changes on esbuild's side are irrelevant
also i think pnpm run build from here is now faster with the new logging handling. it at least feels faster than pnpm run build --summary from here, but that may have extra delays to preserve terminal output because esbuild kept on outputting over itself in my local tests (see attached). i haven't benched any of this.
esbuild technically does lock some when logging but it's obviously not enough under concurrent builds across multiple contexts.
also, current pnpm run build output with an error. a bit more compact but i think it's good enough
vee loves sending me merge conflicts

I would keep using Vencord.Plugins here
it's all executed later so it causes no issues
and importing plugins might cause circular
Light mode fixes of client theme replaces all --white-500 backgrounds with --primary-100, but since discord started lazy loading css chunks, the old approach of generating all these fixes at DOMContentLoaded is no longer functional.
This PR adds a mutationObserver to the head of the DOM to generate new fixes for lightMode as the app is used.
nah
it cant
only if u use on top level
theres alr circular dep somewhere
how do you know?
plugin settings 
Bad Patches
None
Bad Webpack Finds
None
Bad Starts
None
Discord Errors
None
Bad Patches
None
Bad Webpack Finds
None
Bad Starts
None
Discord Errors
None
well it works
gonna keep that as a draft
[Depends on utility functions exposed in #2662]
About Plugin
Discord often hardcodes colors despite having css variables for all it's colors.
For example, --primary-160 is #ebedef, but in the code, they have hardcoded the color hex instead of using the variable:
.defaultLightModeCustomGradient_e77fa3 {
background: linear-gradient(rgba(0,0,0,0) 20%, #ebedef 100%);
}
This causes issues for theme devs who want to make stuff by directly modifying color variab...
i love you
Procrastinated for like 5 days then realised it was a 2 line fix, Also added tooltips to the badges
This is very prevalent when using ClientTheme and looking at "channels and roles"
huh
that's fucking hilarious did discord fix it at the same time

inb4 they unhardcoded every colour
was it ever broken?
this is on canary btw
are you on canary or smth?
Discord is just tactically planning updates to piss off vencord contributors
wth ur right 
yea ive seen the screenshot in the pr 
@austere talon or @long stream
run https://github.com/Vendicated/Vencord/pull/2663 on canary and tell me if <style id="hardcodedColorFixes"> contains anything
why don't you?
Vee is a sacrifice to the canary gods
don't have a canary install right now 
do u not have vesktop
Shrimply vesktop
missing out
so true
I should use vesktop
how do you point vesktop to a custom vencord install?
in the settings
Smh
SHUT
Shrimple
omgg its this issue again 😭
whyyy
is my dev install that broken
I'm no expert but i'd say it cannot read properties of undefined (reading 'Settings')

disable BetterNotes
im trying 
it errors regardless of if its enabled or not
There are still plenty of hardcoded colors
oh those border colors probably won't work
need to fix how I insert var(...)
Discord Account
terminaate
What happens when the bug or crash occurs?
It doesn't show current user's voice channel -
I guess it's happening because of discord ui update
What is the expected behaviour?
Display the current user's voice channel
How do you recreate this bug or crash?
Open any user profile that currently in any voice chat
Errors
Replace th...
we need to disable issues on vendicated/vencord and make some private issue tracker exclusively for contributors
soo many people ignore requirement
Bad Patches
None
Bad Webpack Finds
None
Bad Starts
None
Discord Errors
None
Bad Patches
ShowAllRoles (had no effect):
ID: 579285
Match: ```
/return null!=\i(?=?\i.slice)/
Bad Webpack Finds
None
Bad Starts
None
Discord Errors
None
Ruh roh raggy
patches update for simplified profiles

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
some people manage to turn off simplified profiles
sounds like their issue in my opinion
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
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 (under roles)?
reminder to self (because i don't feel like doing this rn) remember to update description to: "Allows you to open more connections in browser!"
ZoidCord
@verbal pumice, <t:1720032162:R>: look through this
!remindme 8h wake the hell up before i slap the living shit out of you
Alright @weary bolt, in 8 hours: wake the hell up before i slap the living shit out of you
The ViewIcons plugin was blocking users from opening the bigger profile by clicking the pfp in the right side panel in dms
Image of where I'm talking about
Before fix: Clicking on this pfp would open the pfp as an image preview
After fix: Clicking on this pfp opens the big profile view as expected
maybe this should be a toggle instead of flat out removing it
but it should be at least a toggle
click the username?
too much work
@weary bolt, <t:1720070107:R>: wake the hell up before i slap the living shit out of you
omg epic sigma skibidi rizzler code
that’s cringe, not funny
damn, i cant even make a brainrot joke without some weird gay mf getting mad lol
Finally got around to updating this.
@austere talon I think you are better equipped to answer this than me, I don't know what the point is I just re-implemented it for the latest https://github.com/msfragala/eslint-plugin-path-alias/issues/1#issuecomment-2209332213
fuck i now gotta slap myself
Thx ven
lol
someone's obsessed
what?

Bad Patches
None
Bad Webpack Finds
None
Bad Starts
None
Discord Errors
None
Bad Patches
ShowAllRoles (had no effect):
ID: 579285
Match: ```
/return null!=\i(?=?\i.slice)/
Bad Webpack Finds
None
Bad Starts
None
Discord Errors
None
pain
plugin is obsolete anyway
i’m aware
should I not fix
I guess it's fine to keep it working for now
omg epic sigma skibidi rizzler code
that’s cringe, not funny
damn, i cant even make a brainrot joke without some weird gay mf getting mad lol
you’re literally a full stack developer making dumb jokes like this, and you run your own website, get a life instead of being so brain dead
damn
omg epic sigma skibidi rizzler code
that’s cringe, not funny
damn, i cant even make a brainrot joke without some weird gay mf getting mad lol
you’re literally a full stack developer making dumb jokes like this, and you run your own website, get a life gen ass
the commit is part of a pr
for some reason those commits count
Least obvious pr bump
😭
Bad Patches
None
Bad Webpack Finds
None
Bad Starts
None
Discord Errors
None
Bad Patches
None
Bad Webpack Finds
None
Bad Starts
None
Discord Errors
None
hop online first thing i see is 1 minute ago deleted message of someone being a clown 
@austere talon can you make a repo for the code you already have for the new reporter?
i mean there's not much more to do, i might just finish it tomorrow
also it's in go not js
I've done stuff with go in the past already
Decently certain youre meant to use the simple header.
Also, this doesnt apply to any of the specific parts, but im pretty sure you need to add a README.md
This defaults to false, no reason to define it.
This defaults to false, no reason to define it.
why is this included in this PR ;-;
why is this included in this PR ;-;
Just leaving this here despite this being a draft;
Any chance you could include an option to use a colour picker for the banner?
really @fallen vale
yes ;-;
thats why i asked "why"/gen
okay sure if it was another contributor or first time contributor
but it's vee, he wouldn't do stuff for nothing
it's not the first time you are making useless comments/reviews
im aware. which is why i was curious, and asked about the reasoning.
i didnt suggest removing it, just asked as to its reasoning
yea ik, i pointed out the checkbox, and got kinda yelled at for it lol
and then i stopped doin that, and then the checkbox was removed shortly after
whats the point of opening a pr, if a review asking a genuine question is going to be met with "really, my god, stop making useless reviews"?/genq
my god
im not tryna be rude, just understand the situation a bit better
it's literally the owner of the project
i get that
and thats enough of a reason to not ask why that was placed in that PR
it's needed changes which are gonna go to dev regardless
vee just took the opportunity to do inside that pull requests
my pull requests do that all the time
okay, thank you/gen
this, i personally disagree with.
but your project, your rules. ill drop it.
not even my project
no ik
i was sayin that if thats what the owner of the project thinks, and thats whats agreed on here, its non beneficial for anyone to argue against it
alr, i hid the comments
Looking at it, this doesnt seem like a fix, more of just a.. feature removal..
i'd add a setting for it, and add a predicate to those 2 patches.
I know i mentioned this on the other PR, and i did mean to response, but it ended up getting closed;
I feel like there should be a separate setting to define a delimiter, it would allow for the use of things that contain ,, and you could always just default it to a comma.
i did mean to response
i need to go to sleep

