#๐พ-core-development
1 messages ยท Page 144 of 1
not api
we could take this opportunity to stop using csv to make it easier to manage them :P
keep the csv file as a fallback but then new badges get done in a different way
then kill the csv when barely anyone is requesting it

migrate to a db 
?
it doesnt get easier than csv
csv's annoying to edit if you need to remove something automatically
i hate dealing with csv so much
well youre not the one dealing with it
well
yes
i'm expressing my opinion on csv
lol
reading and writing by tools has so many pitfalls
u had a json file laying around u never told no one ๐ญ
?
u wont be using the github one right?
review the pr that adds shit to ClientTheme
it's done
Pretty confident this is up to Vencord standard
Pretty confident this is up to Vencord standard
mood
please wait, I was reviewing decor yesterday
devtools crash is fixed btw
Gm vencord dev 
HOW do i write a plugin 
lovely 
but you can prob learn more looking at other plugins
yeah, the docs are kinda meh starting point it's easier to reference other plugins
๐ I did my first plugin looking at the docs
ok i give up
or should I say, my first patch
Discord Account
megumin
What happens when the bug or crash occurs?
When clicking next/previous when using a device via Spotify Connect, the request is getting wrongly rejected.
While caused by Discord's code, it would be good for us to fix this functionality.
SpotifyUtils.SpotifyAPI wrongly rejects the promise if the status is 202 (Request Accepted), which causes Discord to re-send the request after 5000ms. This means that the track is skipped twice. I'm unsure as to why ...
^ any opinions on which solution is best
I did my first patch in a vc with ven months ago LMAO
i wanted to make something but need persistent storage
did you want to make a persistent message logger
no
i wanted to make a thing where if you copy a link it prompts you for a title and it saves it permanently somewhere
there are solutions to that
oh ok well there's persistent storage
so you can copy memes all day and have them available
That seems easy enough
@dapper crest I think I'm gonna reimplement the color picker to use a round button like this
nookies abt to write this plugin in full for this person
I found out the color picker component we use runs the CustomColorPicker on click
oh noo please dont do that 
so I just need to make the parent wrapper which is the round button and on click handler
nono I got more to do
i will just do the first solution if people don't really care lmao
oh I didnt see
@umbral hedge I think second is better
spotify utils is part of what? spotify controls?
ye but we have a wrapper around it too
ig just make a patch in spotify controls to fix that
it should be easy prob
whar?
is this a new thing?
ive never had that happen to me
@umbral hedge
using speakers and stuff
i would just make it not reject on 202
ye
that's the 2nd solution i wrote in the issue
it's still really weird that they reject on 202
discord moment
is it a intentional reject?
ye
weird
the request works fine and the track skips
but discord has a specific check for 202
yeah thats weird af
yeah lmao
cause that's how you normally use lodash?
shrug, it's not really necessary but not really a issue either
just was curious why conform to that in a plugin
yeah
lodash style guide
checkbox variable seems to not return 0 or 1, or there is an issue with the stylus preprocessor:
regardless of the status of the checkbox variable, the stylus preprocessor always interprets the output as true, when using a text variable, the stylus preprocessor interprets 0 as false and 1 as true, even though in the checkbox, it should also return 0 or 1
my code: if you go into settings you should see the left side with a red or blue background, it is red when the checkbox if condition re...
sounds good
I like the current color picker because it's identical to the one the original experiment used
@coolesding fixed, thanks for flag
re delay: one of the features is instant hotreloading - parse errors should generally be silent anyway so this shouldn't be a user facing issue
be588a2 fix(SpotifyControls): Fix request double-sendin... - MeguminSama
Fixes https://github.com/Vendicated/Vencord/issues/2022
Disables the check for status 202 which causes the promise to reject.
ez
i thought more of like a two second delay, but yeah the notification is pretty small and doesnt matter much
lewi how do i update to the newest commit lol, do i have to do it via the cli?
this means?
did you use gh cli
ye
recheckout
oh okay
just realised that im not actually handling strings properly
but
i cant handle them properly in stylus
if this is a problem why don't we lint it out
i mean in practice
if someone types a string like "this is lewi's testing string" then that's literally unparsable
by stylus
why did you add it though
because i have to escape one or the other to make it a string
it wasnt there, doesnt need to add
eslint
but you cant escape
added it
so essentially in stylus you can't have a string with both types of quotes in it
its just impossible
we really should have support for it though
i already support it for less by replacing all quotes with \"
maybe you could prioritise " over ' so if " " is around ' ' then you handle ' as a string
concat the strings
wait can you open a string with " and close it with '?
replace " with "+'"'+"
what
in variable definitions it might
ahhh it does
hmmmmm
good to know
alright i can work with this
your fix works perfectly fine btw
yip just fixing text
thank you :3
oh also, is it possible to have options that depend on the state of a checkbox, for example if you have Hide-Nitro enabled you can then show more checkboxes to hide something like the gift icon or the nitro button in dms
custom value 
this sucks
me and my dad collectively discovered that stylus doesnt have escape sequences
(yes my dad got interested in me writing vencord code)
lol
pog
@umbral hedge
oh i replied but ig i forgot to send it or smth
they probably noticed sometimes requests return a 202, but it's not documented in the API docs for some reason, so they probably rejected it since it doesn't meet the api spec
when spotify.exe is running, it returns 204, but when using spotify connect instead, it returns 202
there's nothing actually wrong with a 202 response, it basically just says "We've forwarded the request to your device"
Because that's how you normally use lodash. For reference, see the code on the lodash.com homepage
tbh I always thought lodash as _ is a little odd, following that even in a plugin seems silly to me but ultimately it's just preference and doesn't really matter
probably has its roots in being designed as helper methods for common problems, so the code you write using the library should be small, so having _. instead of lodash. is prefferable
why not just destructure in imports? /shrug
Destructuring and Imports were added to JS in 2015
Lodash started in 2009
So it would've been a pain in the ass to extract every method you used from lodash it started, so they went for _ to be short?
yeah
I mean I understand why lodash is how it is but we're 14 years down the line I don't think it has to be used like that anymore
because its convenient and there is no real downside
it's whoever who wrote those who are wrong, not me /s
where do i download vencord DEV build?
git switch dev
and if you don't understand that, you probably don't want the dev build
what happened
the canary one
I don't understand #๐ง-discord-changes
(webpackChunkdiscord_app.push([[''],{},e=>{m=[];for(let c in e.c)m.push(e.c[c])}]),m)
try it
it returns []
findByProps exploded
uh
i just updated and my client seems oook
can you screenshot webpack runtime
hmm
doesnt seem so
okay I ran
its just chunkid or something
yeah it's chunk id
according to the silly @mortal fractal
how nice of them (die?)
discord try not to break client mods for a year challenge (fail)
it's running on stable first
NoTrack (found no module):
ID: -
Match: window.DiscordSentry=
None
None
None
so true
well at least reporter seems to work
NoTrack (found no module):
ID: -
Match: window.DiscordSentry=
None
None
None
my vencord is fine too so idk
we vibing
what's this though
how do you actually know they changed stuff @shy veldt
some error from patchWebpack lolz
fails to read the value of require.c[id]
odd
wait
script when it just r.u is not a function
it's not gone all finds worked
๐ข
yeah i misinterpreted
devtools took me to the wrong line
and not just findByProps
hollow
do I have like some shitty old ass vencord
what is the latest version
(extension version)
lmao you are making me so confused
sorry
it works but it doesn't (real?)
1.6.4
wreq.u is still a function, reporter uses it
okay so yes i do have the latest
NoTrack (found no module):
ID: -
Match: window.DiscordSentry=
None
None
None
**loadLazyLoadedChunks.ts: **
// @ts-nocheck
export default async function loadLazyLoadedChunks(): Promise<void> {
await (async (r) => {
for (let i = 0; i < 100000; i++) {
if (r.u(i)) {
try {
await r.e(id);
} catch (error) {
console.error(error);
}
}
}
})(webpackChunkdiscord_app.push([
[Symbol()], {},
e => e
]))
}
NoTrack (found no module):
ID: -
Match: window.DiscordSentry=
None
None
None
yeah no i just tried main branch on latest canary im all good
maybe its web specific issue????
silly
hollow
they dont return the value of the callback anymore
r is undefined
it's been like that since ages
ages
it stopped working 1h ago
:wires:
so when you do e => e, . push doesnt return e
i just tried vesktop its fine...wtf
vencord broke cuz of that like a month ago
when rspack I think i don't remember
rsfold when
but if you look how we grab require we not longer depend on.push returning it
hopped on pc
im just confused why dziurwas vencord webpack is fucked
fix it @shy veldt
dziurwagpt
how code?
no
fierygpt so rude tf
**loadLazyLoadedChunks.ts: **
// @ts-nocheck
let r;
webpackChunkdiscord_app.push([
[Symbol()], {},
e => r = e
])
export default async function loadLazyLoadedChunks(): Promise<void> {
for (let i = 0; i < 100000; i++) {
if (r.u(i)) {
try {
await r.e(id);
} catch (error) {
console.error(error);
}
}
}
}
real...
hopping on github mobilum rn.,,
indentation is kinda fucked up but whatever
NoTrack (found no module):
ID: -
Match: window.DiscordSentry=
None
None
None
lmao the normal reporter ran
what's the error
NoTrack (found no module):
ID: -
Match: window.DiscordSentry=
None
None
None
yeah ๐
**loadLazyLoadedChunks.ts: **
// @ts-nocheck
export default async function loadLazyLoadedChunks(): Promise<void> {
let r;
webpackChunkdiscord_app.push([
[Symbol()], {},
e => r = e
])
for (let i = 0; i < 100000; i++) {
if (r.u(i)) {
try {
await r.e(id);
} catch (error) {
console.error(error);
}
}
}
}
horrible indentation but fixed
yes
it did
you just have to fix indentation lol
clienttheme is in dev right
I'm already on my way to diff it (fuck this im not doing it)
why?
i was thinking clienttheme was in dev and that the light mode pr was wanted before dev -> main merge
getScoreWithoutFetchingLatest -> getScore
getFrequentlyWithoutFetchingLatest -> getFrequently
ah
I was messing around with client theme today
but I haven't got a thing to work yet
rip
I didnt try for long though since I had to go
I'm surprised it was still working
whar
real
i did anyway
nothing interesting
wrong
they still do
it's just that in the early stage of webpackGlobal it gives some yeeyee ass fake webpack require
without cache
it's only added later
well dzi stuff just broke cuz of that
I see
**webpack.ts: **Lines 77-79
instance.push([[Symbol("Vencord")], {}, r => wreq = r]);
instance.pop();
if (!wreq) return false;
when I was experimenting in how vencord grabbed the require I noticed that
you just need to either load later when it's already initialised, or keep pushing until it works
it's cuz the first pushes happen when.push isnt patched yet
cause for the first like 80 chunks or so, webpack.push isn't the proper webpack push yet
it isnt
no lol
it wouldn't be a big deal anyway
i misinterpreted devtools error
so true
it took me to wrong line i was misinformed
i already have a hack for working around unexported webpack cache
i will hack you..,,
huh?
webpackRequire.c
no
huh??
its there
no its not
guh???
freshly loaded instance
one moment ig
hi, do u maybe know that when ur using the vencord browser extension u can still add plugins from other mods?
or do u need to have vencord installed?
vns
then ven probably added the private cache extractor idk
it's not added
canary
it's right there
???????
canary, just updated
did they revert it
magicrequiresplosion
tf is wrong with your clients wtf
[default] [BUILD INFO] Release Channel: canary, Build Number: 250415, Version Hash: 1d24c75bf1c1c67cde5fbee4b506d79ef3ccc6f4
Canary 250411 (0d3fa7c) Vencord 3e7d946Electron 22.3.26Chromium 108.0.5359.215Host 0.0.357 arm64 OS X 10.15.7 (23.1.0)
um...
[BUILD INFO] Release Channel: canary, Build Number: 250415, Version Hash: 1d24c75bf1c1c67cde5fbee4b506d79ef3ccc6f4
web.1478845241b4821128fa.js yes?
c is still there
is the magic require not that magic.. 
huh
WHY IS SENTRY OVERWRITING WEBPACKREQUIRE
sentry.402450ae91f0dd03660c.js is overwriting webpackRequire
at least for sideloaded webpack modules
w/e im just not going to say shit here anymore
why :(
my own mental sanity
what's the issue with you saying here
I can't post in the third party plugins channel, which is very cool, so I have a quick question - I notice that hardly any plugins interact with the member list. Is this due to lack of interest, or is it impractical to do so, for example, to override member colors?
my own mental sanity and the fact i feel like i have stockholm syndrome or w/e
ohhh did u accidently grab sentrys wreq
that happened to my webpack grabber userscript, i accidently got react devtools webpack and was so confused

shits so fucked its like 3 instances of webpack running at once that im surprised the normal app isnt beyond fucked
discord moment
im just
beyond fucking frustrated and burnt out
i dont want to have to fuck up the entire internals of hh just to get the fucking client to work
i should be in fucking bed by now but i want this to just not fucking be postponed
discord at its finest
there is no sound
Add "Capture application audio"
Set it to your app via Proprieties
Then set "Monitor audio and send it to the output" on Monitor audio
I recently got a new laptop and for some reason I cannot preview the voice messages I record anymore, any idea of what could solve this?
This is what it looks like after I have recorded a vm. Usually it would let me listen to it before sending it
Why did they do it in the installer repo
probably because its the closest thing to vencord in the vencord org
using the exposed Vencord object in the browser console, how would I mute the user (as if they clicked on mute in the bottom-left)
er wait, I'd probably also have to check if the user is in a vc in the first place
although I could probably just listen to the join event for that, right?
yeah ig this still applies then
Useful plugin for Linux users who use middle-click autoscroll to prevent
Discord's default behavior of redirecting all paste events anywhere to
the chat box.
@D3SOX @Nullmatic can you test the plugin in https://github.com/Vendicated/Vencord/pull/2024 to test if it gets rid of the unwanted pastes?
Small quality-of-life thing. Allows the user to peek at Discord via the tray without moving the mouse from the tray to the window decoration or Alt-F4ing.
Can move to a setting if this is undesirable for some users.
huh
new channel just dropped
being able to pin plugins would be great
also those 2 buttons do the same thing in clienttheme plugin settings window, along with clicking outside of it
which is save and close
cancel button doesn't cancel lol
Not sure if it's possible for me to fix this.
A core component of the thing is being able to see changes right as you make them
And I don't know how to integrate reversal of changes when cancel is pressed.
Delete user's account
ClientTheme will have it's own option in the vencord theme menu and the discord appearance menu eventually.
There is a lot being worked on so that will take a bit
what about saving the current theme color to then apply it with the cancel button
I don't know how to do anything with the buttons on the settings modal
about that I started working on it yesterday but couldnt do much cuz I didnt have much time, and today I'm kinda busy too
Can we just merge the light mode support and theme menu button in the mean time?
oh ok
I wanna pr the appearence button and then review later
to be fair it's no biggie for light mode support to wait a little lol
The button in the theme tab is kind of a biggie imo.
not really, if someone enabled client theme they already know where to find the color picker
and they prob wont change the colors so much
it doesnt have to be added in a hurry
True
startospeech
@limber skiff just to check did u see that i asked for ur review
I didnt lol
oh lol
yeah if u can pls review my insanity
yeye as soon as I can, since I will be busy with client theme until I can get the thing I wanna do to work and review
dwdw
what a simp
is this what schizophrenia feels like
yea
hi
horrific
@limber skiff you think maybe main merge today? wondering if i should hold off on releasing winter decors
it's not in my control
oh ok wasn't sure if you did those sometimes
main branch is protected and you can only merge if it passes tests
but for that it requires you to pr
only ven can push without pr
unless I can?
because now the dev branch is tested
@austere talon can I try to git push origin dev:main?
and then the reason I dont do it is cuz I dont want all the commits to turn into one
yeh
wouldnt that push main to dev
right
huh

wrong order ๐
isn't that the dev origin and the main branch
yeah i dont think thats correct at all lmaoo
I fixed the order
is that actually a thing
huh
nookies.. merge
doesnt push work
stop pressuring people so much fiery
i doubt
i didnt mean like literally merge
i meant git merge not git push
to be fair I havent tried in so long I dont know lmao
no merge may
im 99% sure this command doesn't work
when you specify two args to push the first one is the remote (origin) and the second the branch
how do you push dev commmts to main?
yea exactly i was trying to point out it's git merge not git push 
okay I fixed the command now
no merge 2024
oh maybe that works
git checkout main
git reset --hard dev (or git merge dev but it makes merge commit)
git push
oh lmao
anyway ill merge later
I wasn't trying to pressure, i was just wondering. I'm sorry if it came off that way?
it does work
it's just cuz I want to see if I can push to main
because before I couldnt since it required tests to pass
I knew i wasnt tripping @junior olive ๐
yeah that command makes more sense
git push dev main on the otherhand
yeah I had wrote it wrong


can u find a webpack module's code by its props out of curiosity
no
thought so rip
you can
im going to assume theres been 0 issues with ReactCurrentDispatcher disappearing halfway through a render on vencord
After installing manjaro-pipewire, the button with audio selector has appeared, but still no sound in screenshare
so im relatively new to vencord as one would be probably able to guess, im trying to patch the updateVideoQuality function in discord, any tips? its very different from betterdiscord as we all know
the
and itโs deleted
NoTrack (found no module):
ID: -
Match: window.DiscordSentry=
SuperReactionTweaks (found no module):
ID: -
Match: .hasAvailableBurstCurrency)
None
None
None
NoTrack (found no module):
ID: -
Match: window.DiscordSentry=
SuperReactionTweaks (found no module):
ID: -
Match: .hasAvailableBurstCurrency)
None
None
None
@D3SOX @Nullmatic can you test the plugin in Vendicated/Vencord#2024 to test if it gets rid of the unwanted pastes?
Sorry for the late reply. The plugin did limit the number of pastes when enabled on my side at least. It doesn't stop them but makes it harder for it to happen every click so that is a plus.
The funny thing I also found when testing is that when enabling native Wayland capabilities using the flag: ```--ozone-platform-hin...
Sorry for the late reply. The plugin did limit the number of pastes when enabled on my side at least. It doesn't stop them but makes it harder for it to happen every click so that is a plus.
Can you try increasing the reenable delay slider in the plug-in settings? Also check the other setting for when to allow pastes- the text input is selected until you click on another element and by default I still allow pastes when text boxes are selected.
The funny thing I also found when testin...
Can you try increasing the reenable delay slider in the plug-in settings? Also check the other setting for when to allow pastes- the text input is selected until you click on another element and by default I still allow pastes when text boxes are selected.
For me, when I set the settings to "Whenever a textbox is active" and set reenable delay to 1000ms usually after the first middle-click, the second one will paste into the chat box. The same happens when setting to "Only when clicking ...
Ah! I forgot to clear the reenable timeout function after getting another click. I'll fix that
bro look at this match @junior olive
let a = /(\.trackEmojiSearchEmpty,200(?=.+?isBurstReaction:(\i).+?(\i===\i\.EmojiIntention.REACTION)).+?\[\2,\i\]=\i\.useState\()!1\)/;
i cant tell if good or bad
ah r u fixing that
NoTrack (found no module):
ID: -
Match: window.DiscordSentry=
None
None
None
NoTrack (found no module):
ID: -
Match: window.DiscordSentry=
None
None
None
made a bit more generic
that one wasnt mine iirc
I mean I made the new patch more generic
I modified this one with force push
Yeah
@Nullmatic moving conversation from the other thread here. I fixed the timeout so rapid double clicks won't trigger pastes, can you test it?
Some pastes that don't get a target will still get passed through ComponentDispatch. This is pretty uncommon though (I mostly got this when middle clicking on a popout menu, which you can't scroll on anyway...).
Another approach that might work better is blocking input to the text input itself instead of trying to block paste events.
+1 on this, I noticed you offer an RPM but it seems to be Fedora compatible only as it doesn't run on OpenSUSE.
At this point I would favor a flatpak so you don't have to rewrite a new version for every distro.
Yeah. Add Flatpak. Best and don't have to maintain many packages for different distros.
As of Discord Version Stable 250759 (5d4b938) this issue has been fixed and no longer persists.
OKAY YOU DONT HAVE TO YELL
@topaz thistle i might need your help in a bit
When this requires a Vencord plugin anyway I don't see the need for this anymore as https://github.com/Vendicated/Vencord/pull/2003 exists
i'm trying to package vesktop for flatpak so i need to know if there's anything special i need for venmic to work
Sure
i've already added the pipewire run dir as a permission for flatpak but am unsure if any other perms need to be granted or any flags need to be passed to electron
we also need pulseaudio perms
already added
im using the armcord flatpak as a base if you want to have a look
With that as as base and pipewire perms added it should work fine ^^
alrighty
that pr isn't mergeable @vagrant topaz
cause it reinvents the wheel
you can also just implement it in vencord with the flag
yes
because im the maintainer
of the flatpak
it's how they do things
like how we have shiggybot fork winget-pkgs to itself because its submitting it
(it forking to vencord org was actually a mistake in the end so)
Content
Hi,
For the past 2 days everything was as smooth as butter. Today when I was updating the css of my plugin, I found out that pnpm inject no longer works as there's a problem with the installer file. Here's the full error log:
D:\Modding\Git\Vencord>pnpm inject
vencord@1.6.4 inject D:\Modding\Git\Vencord
node scripts/runInstaller.mjsDownloading VencordInstaller.exe
node:internal/errors:490
ErrorCaptureStackTrace(err);
^TypeErr...
ojfajposikfjpikoawipjofwqajio
you love
honestly i have no clue why it pastes as grafik ahahahha
my github is in english
i have my firefox in germna
maybe thats why
too lazy to recompile vencord
crap
didn't mean to ping
my bad
i'm not sure this is good UX tbh. no other app does this, and i personally might spam click the tray to show vesktop
c39678d vesktop.desktop: remove faulty WMClass field - Vendicated
just close it? i don't understand why it being always on top is an issue
this is also not a discord behaviour, so it would be inconsistent with the main app and probably confusing
just close it? i don't understand why it being always on top is an issue
Because it wastes screen real estate? I don't think a update window should be blocking other applications by always being on top.
blud leaving the update window open for hours
someone try https://github.com/Vencord/Vesktop/pull/248
this doesn't work, the splash doesn't show but the main app does not start minimised
yea but it shouldnt
cause
in fact the main app starts maximised!
ahahaha
worked it out
if the app was maximised when you quit it, it will restart maximised regardless
let me edit my comment
ah, it seems like this only doesnt work if the app was maximised when it was quit. when windowing the app before quitting, it works as intended
this will likely fix it
if (!startMinimized && Settings.store.maximized && !isDeckGameMode) {
oh ic
if you dont do it then when you restart the app it wont be maximised
weird side effect of electron i suppose, but it makes sense
you cant maximise a nonexistent window
it doesnt support pnpm
error This project's package.json defines "packageManager": "yarn@pnpm@8.11.0". However the current global version of Yarn is 1.22.21.
LOL?? no it doesnt lmao
how do we fix it 
rerun!
i dont think that will work ๐ญ
no it's an issue with the new yarn release
lol
the action uses the latest version of yarn
we could just stop using the action
its so little code
like this is really just
pnpm install
pnpm build
pnpm electron-builder --linux --publish always
if u read it
yeah literally
alright
i'll just move away from it
let me rewrite it
Cool I can delete my repo then
oh i can do it rq
nop
me
- name: Run Electron Builder
run: |
case "${{ matrix.os }}" in
macos-latest) platform=mac ;;
ubuntu-latest) platform=linux ;;
windows-latest) platform=windows ;;
esac
pnpm install
pnpm build
pnpm electron-builder --$platform --publish always
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
should work i think?
GH_TOKEN
wdym
env should be GH_TOKEN
oh right
GITHUB_TOKEN was a thing that only the action did
no pnpm
I FORGOT
yeah
idk what ur smoking
strategy:
matrix:
os: [macos-latest, ubuntu-latest, windows-latest]
include:
- os: macos-latest
platform: mac
- os: ubuntu-latest
platform: linux
- os: windows-latest
platform: windows
yeah that might work
though out of curiosity
do we actually need to run it like this
electron builder supports cross compilation does it not
Donโt expect that you can build app for all platforms on one platform.
oh nevermind
oh god
i still have ptsd from trying to use electron builder in gitlab CI
kitkat
DoubleClickRename plugin?
have you made it?
no haha it's a concept
i think it could be useful if you only want to rename the channel, especially when you have multiple ones to rename
kinda useless just right click > edit
going through the channel settings page then leaving it everytime is pain a bit
it's just a suggestion though, i think it could help going faster
and since double clicking a channel doesn't do anything currently, i find it being a good opportunity
this doesnt belong here anyway
ah my bad, the suggestion channel is closed so i sent it here to talk to devs and get their opinion
the suggestions are closed for a reason and not an invitation to abuse dev channels
girl what... @nimble plaza
it was the same issue as a while ago when native plugins were introduced
it got mad if nothing was exported
so we can't share ideas at all?
correct
simple fix, export default () => {}
ohhh
might be what you want, but it's not how it has been received last time so
it exports something when theres no export calls
so u need to do it urself with a function
i dmed u the broken plugin a while back
this has no exports
but it does have imports
does your file that's broken have impoorts?
send the file contents
terrible file
AHAHHAHAH i downloaded and opened and it tried to open as a video
Hey, just tested AppImage of 0.4.4 on Steam deck and it reports that c++ lib is too old when it's not.
(deck@steamdeck lib)$ ldd --version
ldd (GNU libc) 2.37
Compiled by GNU CC version 13.1.1 20230429.
Error in console shows:
Failed to initialise venmic. Make sure you're using pipewire Error: /usr/lib/libstdc++.so.6: version `GLIBCXX_3.4.32' not found (required by /tmp/.org.chromium.Chromium.NgEKiM)
at process.func [as dlopen] (node:electron/js2c/asar_bundle:2:21...
the version you checked is glibc, not glibc++
try the flatpak version, it will likely work. it's currently still in development so you'll need to build from source:
cd /tmp
git clone https://github.com/lewisakura/flathub
cd flathub
git checkout vesktop
sudo flatpak install flathub org.freedesktop.Platform//23.08 org.freedesktop.Sdk//23.08 org.electronjs.Electron2.BaseApp//23.08
sudo flatpak-builder --install build-dir dev.vencord.Vesktop.yml --force-clean
/usr/bin/fla...
the version you checked is glibc, not glibc++
try the flatpak version, it will likely work. it's currently still in development so you'll need to build from source:
cd /tmp git clone https://github.com/lewisakura/flathub cd flathub git checkout vesktop sudo flatpak install flathub org.freedesktop.Platform//23.08 org.freedesktop.Sdk//23.08 org.electronjs.Electron2.BaseApp//23.08 sudo flatpak-builder --install build-dir dev.vencord.Vesktop.yml ...
ayy flatpak works on steamos @turbid hatch
๐
https://github.com/Vencord/Vesktop/pull/248#discussion_r1412835063
Could you explain the though behind the
mainWin!.maximize();
mainWin!.hide();
part instead of simply checking if it should or shouldn't start minimized before maximizing
because if you just skip it, it won't be maximised once you do launch it
just tested this on i3 window manager, seems to work well!
When this requires a Vencord plugin anyway, I don't see the need for this anymore as https://github.com/Vendicated/Vencord/pull/2003 exists which does it just as a plugin. Or better yet add the flag via plugin so it only requires the plugin. Alternatively I think what you're doing in the plugin right now could be moved to here under
src/renderer/patches
Personally, I think the approach of both the desktop setting and plugin is better than the manual feature because
- the desktop setting ...
I can definitely see your point here now. I haven't ever tried Vencord in a standalone Chromium browser with that flag.
The funny thing I also found when testing is that when enabling native Wayland capabilities using the flag:
--ozone-platform-hint=autoand having middle click scroll enabled, after the first scroll it crashes the client.
I was unable to replicate this on Plasma 5, middle click scroll works as normal when launched with native Wayland.
@austere talon my friend has had a deranged idea where if we do the konami code on the website it shows men kissing
should i implement
horrorrro
"that's a yes" - dy
I understand that it would be annoying for this to be the default if you like to make redundant clicks on the bar. Would this be acceptable as an settings option? With #261 patched in I basically only use the system tray to launch the Vesktop
I don't think it'd bad UX- for almost every system tray icon clicking it once opens something or does an action and clicking it again closes it or reverses the action. It's somewhat against the standard convention to have an icon do nothing once you c...
NoTrack (found no module):
ID: -
Match: window.DiscordSentry=
None
None
None
Another thought: if the window is shown but not focused / behind a bunch of other windows, clicking the tray icon should focus it (helpful for responding to dms/pings from the notification badge).
NoTrack (found no module):
ID: -
Match: window.DiscordSentry=
None
None
None
I understand that it would be annoying for this to be the default if you like to make redundant clicks on the bar. Would this be acceptable as an settings option? With https://github.com/Vencord/Vesktop/pull/261 patched in I basically only use the system tray to launch the Vesktop
Maybe? But it's still a really weird option to have.
I don't think it'd bad UX- for almost every system tray icon clicking it once opens something or does an action and clicking it again closes it or rever...
All of my tray icons do one action when you click them, and doesn't undo the action. The only case I've ever heard of this happening is when the tray icon completely changes to respect the new action (e.g., recording software). It seems like it's against convention to give a tray icon multiple actions depending on context.
What kind of tray icons do you have? All of my icons or widgets do something when you click them again...

๐ ๐ ๐ ๐
that's actually reviewable
you can make all of this a single findBulk
look at how permissions viewer does it
the first one is just webpack commons moment
you are right
the code is kinda insane but nothing that can't be fixed
yeah, but thankfully it's not 200 files
Are we finally getting semi stable class names for themes
no
more CPU intensive and there are some duplicate names out there
the theme I want to use makes the UI unbearably laggy cause of all the advanced selectors (and my computer isnโt that weak)
tysm for the tips, i tidied things up n did what u guys said, had some issues with findBulk with some modules (guessing it's cause they load rlly late compared to the other ones? dunno) but it should be a bit better now,,.,
should I report this as a bug or is it a wontfix?
wontfix
arrpc needs to impl it
and likely won't
get arrpc to add support then it's no longer a wontfix
I see
good to know
though I don't think that:
a) I'll be the first one reporting this and
b) My report will have much of an impact
But I'll give it a try later
Do not introduce unnecessary whitespace in your replacements.
I was unable to review the plugin in depth, as the stream quality settings seem to only be available on Discord desktop (I checked Vesktop and browser). If this is true, you should add .discordDesktop to the name of the plugin's folder.
Do not hardcode minified variable names (e.g., S) in your patches; instead, use \i.
You could use {entries.map()} in the return statement instead of adding to a temp array with entries.forEach().
Use > 0 instead of !== 0, as OptionType.NUMBER accepts negative numbers.
whitespace in a replacement doesn't matter and if anything it's good because it makes the replacement more readable
while most of these are valid theyre just nits that have no reason to be brought up
right ryan isnt here
well whatever
They left?
they got banned
LMAO why
๐
If this is no longer best practice, please remove this from CONTRIBUTING.md.
having no idea what theyre talking about and giving bad advice, and just generally being a dick
which now that i think about it, also applies to these reviews
kinda mini arjix
just kinda too direct and thus rude
and insisting on certain points
is this a linux thing? i think this is a linux thing





)