#๐พ-core-development
1 messages ยท Page 271 of 1
and now you broke description ๐ญ
huh how
I'm blind...
some things definitely look a bit more bold
in vencord settings
like these titles
unless my eye is lying to me
or these buttons
yeah true
its cause I changed this component
the HeadingSecondary
it used to be bigger but I made it like Discord's FormTitle
we should rename BaseText -> Text once we remove Discord Text
we need to recheck everything I already changed to the new Headings
cause it'll all be wrong size
it doesn't have to be 100% identical
it just has to look good still
Last TODO is double checking all Button usages to ensure they look okay
After merge:
- Make a second commit that migrates all code to use the new components directly and deprecate all Compat webpack commons exports
- After a while (and Vesktop update), remove the deprecated things
also we can still improve looks later
I removed the useless descriptions and made the ones that need full restart alert
do I push?
also this has too much empty space in the bottom
that card is pretty useless, it's just there as a placeholder for the restart card
cause otherwise the entire ui shifts
I mean I could also descriptions that add nothing new LOL
but having Requires a full restart as descriptions seemed weird
i guess this is fine
discord does it too
yeah it's fine
just remove the useless descriptions
but it should still tell you that you have to restart
via toast or popup
ye it does
okay
btw we should try to readd the thing that makes the entire part of the FormSwitch clickable
rn only the switch itself is clickable
I tried to do some cursed thing where the entire FormSwitch is wrapped in Clickable and uses a ref to the switch input element and calls ref.current.click()
but it doesnt work right
pull my changes
this
wtf did pnpm smoke
how is it too small
it looks normal
compare to main branch
nothing changed i think
we should also remake Card and Clickable soon
but not this PR
we need to get this merged sooon
Card and Clickable aren't that high priority
Settings are fine without all the Cards
still usable
HHAHA
with clickable cursor
and onClick, onRightClick and keyboard listeners
ah yeah
why is there an empty div lmao
i tried removing it but it breaks everything somehow
wtf
idk what causes the overflow
ill remove if its not needed anymore
we have exactly the same css as Discord
Now testing! 
need to check all Buttons first
Experiments (found no module):
ID: -
Match: ```
H1,title:"Experiments"
findByProps("bannerButton")
Experiments (found no module):
ID: -
Match: ```
H1,title:"Experiments"
**__ReviewDB (found no module):__**
ID: `-`
Match: ```
.SIDEBAR,shouldShowTooltip:
bruh
lmao
good that i checked
I just merged dev into sadan branch
i pushed that to main lol
๐ญ
what did you find
8eabb11 ClearURLs: use rules from ClearURLs browser ext... - thororen1234
479d01a fix FavGifSearch regression - Vendicated
9c0af5a Fix broken MessagePopoverAPI not working (#3661) - thororen1234
fbc2dbe FakeNitro: fix nitro themes not working for som... - sadan4
56d25b0 UserVoiceShow: Improve tooltip & add icons for ... - gabrielmar
e8ea561 StartupTimings: Move section to developer options - Nuckyz
link button is bad
not compatible with stable, but doesn't matter
It's only the sidebar profile anyways
vtest dev2
Now testing! 
ReviewDB (had no effect):
ID: 983559
Match: ```
/children:[(?=[^[]+?.SIDEBAR}),\i.bot)/
WebContextMenus (took 9.1ms):
ID: 29264
Match: ```
/!\i.isPlatformEmbedded/
Seems like everything is working fine (for now) 
vtest uikit
Now testing! 
Seems like everything is working fine (for now) 
ReviewDB (had no effect):
ID: 983559
Match: ```
/children:[(?=[^[]+?.SIDEBAR}),\i.bot)/
ShowHiddenChannels (took 7.1ms):
ID: 950969
Match: ```
/(?<=.id)))(?=&&(0,\i.\i)((\i)))/
findByProps("bannerButton")
isnt that wrong
this is how it's supposed ot look yeah
yeah
i already fixed it using wrong button
lol
your deco is so funny ngl
wait
and added to another ๐ญ
what did you push
well
I put it on CustomizationSection
not realising that title is inside there
now
it looked wrong before
it used wrong button
it's the same
don't look at the Change Decoration above
I showed u earlier it's Discord using the old component
enable the show mana component toggle
yeah ik
still needs fixing
the old button had padding
the new one doesnt
it shouldnt have padding tbh
the plugin should just use flex
discord is insane for having padding
Discord's own new text button also doesnt have padding
oh yeah that makes sense
you can turn this on
discord is insane
dont ask me why some glow green some pink
whatever tf this is
em usually
I only use px when I'm copying Discord's ui
discord uses px for everything
are u gonna fix Decor
ye
fix uhh
the other one
I dont have it enabled
session viewer or what its called
I'm fixing yeah
our flex component is so garbage
I wanna make it better
for Decor, the Cancel button should use Primary button
same as this
(or if you're using our new component, thats secondary)
for?
yeah lol
the text was just floating at the top of the button lol
love
should we change the title to Decor (Vencord Plugin)
so it's clearer
imo yes
i didn't test the Link buttons in action much
how to remove the background
with compat
in cancel
also god that's a lot of gap
it should be 8px / 0.5em
what background do you mean
that looks right, just too much gap
this decor is so silly
lolol
it shouldnt even uses classes
cl("change-decoration-modal-footer", "modal-footer");
true
hmm
or plugin icon
next to it
i think (vencord plugin) is too verbose and will look ugly, subtle plugin icon when you hover that says "this is a feature added by decor!" would be cooler and nicer ui/ux imo
@austere talon what have you done to my vencord
It's the review db thing lol
it puts ErrorBoundaries everywhere
do you think vencord notifications setting title is needed here
or just the buttons
rate
(right now I removed)
i prefer with title
i added it so it's more explicit that it's not Discord Notifications
and yet people still get confused lmao
should we
as lewi said, writing vencord is kinda verbose
I dont really like it
change description to Settings for Notifications sent by Vencord. This does NOT include Discord notifications (messages, etc)
why arent my buttons highlighted
i split the array types into different files now for so its more maintainable and more readable
ts is never getting merged
@austere talon what's left
pull
I did
I had removed the css
the tooltip is still useful
but the css is still needed
trolley
let me pull and test
LOL jumpscare
im listening to music rn and I have watcher running
when I pulled, it rebuilt and reloaded css and for a fraction of a second that meant it didnt have vencord css
so the spotifycontrols album cover covered the entire app
the pic here
actually not entire app buit like this
the default html image styles are so insane
it just becomes as big as it wants
AHAHAH
even the permviewer is mana highlighted now
vtest uikit
Now testing! 
Seems like everything is working fine (for now) 
ReviewDB (had no effect):
ID: 983559
Match: ```
/children:[(?=[^[]+?.SIDEBAR}),\i.bot)/
PinDMs (took 5.2ms):
ID: 831218
Match: ```
/(?<=channels:\i,)privateChannelIds:(\i)(?=,listRef:)/
findByProps("bannerButton")
need to fix that bannerButton shit
but they are gonna end up bumping stable at some point
still gotta fix this
maybe make these buttons too
what do you think
lemme show you
look
it's just variant="link" and variant="dangerSecondary"
dang nice
the link variant isn't a button Discord uses actually
it's component v2 button
vp Logger
lmao
lmaoo
yeah its good
this is last thing
then I need to check Vesktop
then we can merge
I thought u always used vesktop
even old version is kinda ugly
maybe I will change button kind
also idk why Discord thinks im in Mainz
im hours away
wait where
AHAHAHHAA
(it only shows in dev build btw)
and yeah it's not supposed to show for that element
is this good
should be fully good to go now
ye
let me test one more thing
i added new build var remember
IS_ANTI_CRASH_TEST
okay we good
is it normal that there are no settings on web lol
the margins are off yeah
the css priority is wrong
good that i tested web
lol
is it an issue at base text?
the css-core should be above everything else
no it wont change anything lmao
ah okay
good catch
the problem is that we have a race condition
yeah I think so
I can't for the life of me spell specificity
im gonna hit you with the approve
Looks okay to me (I wont say good because I don't want to)
nooo now it's above ๐ญ
Now testing! 
Seems like everything is working fine (for now) 
ReviewDB (had no effect):
ID: 983559
Match: ```
/children:[(?=[^[]+?.SIDEBAR}),\i.bot)/
findByProps("bannerButton")
before releasing web
wait a little
users on desktop will likely catch any issues fast
oh i noticed an issue with our Switch
it doesn't have a highlight for when it's tab focused
I will add
not rn
tmrw
talking about this ring
i see what u mean now
we could just release and if it's bad we tell xinto to cancel
doubt they will approve within 8h
even if they do it's not really worse than rn
im gonna push tag
alr
discord shopping on Friday evening ๐
how is canary stable and stable broken ๐ญ
average "lets pray with prod and leave it to oncall"
i got confused and thought discord broke something oops
they pushed the select changes hours early to the development client and had me going crazy on why it was only affecting me
WHY DO YOU USE THAT
black icon > piss icon
Custom icon:
effort:
discord overrides it on every update
I think. I did it before
gave it my nitro sakura icon
Just hop on vesktopp
no
Why
the crates/shared crate is for shared logic
it was made mainly because @Vendicated wanted CLI and GUI to be separated, though I'm not sure if it should stay this way or they should share a crate but separated by features, we also thought about having this logic just be inside of crates/core but at least to me it would feel a bit messy
Content
- go to a server with reaction roles style verify
- make sure you have all channels visible
- click the button to verify
- notice all channels have the lock icon utill they're clicked
probably an issue with not using state from stores
Request Agreement
- [x] I have read the requirements for opening an issue above
This plugin allows locking threads that you created, even if you don't have manage threads permission.
For some reason, this is a built in feature on mobile, but not on desktop.
is that really a mobile only feature
It might just be because I'm on an old version but I don't see it
vext
The Vencord Chrome Extension is out of date! (v1.13.2 vs v1.13.3)
i merged that horribly
we should add support for the party thing on vencord customrpc
should be easy enough
also I'm really stupid, I forgot you can just use label for this
eh
is that really so bad?
it uses SettingsSection which all other options also use
seems fine for that case
I'll just make it allow you to customise the tag
with label, the entire thing controls the Switch
I did that and added focus ring
vext
The Vencord Chrome Extension is out of date! (v1.13.2 vs v1.13.3)
tbh we need to come up with a better solution to the ErrorBoundaries
tired of this xD
Isn't that the whole point of noops
it'd be great if you could get like the context that the component is used in
like if it's too big, it collapses into a โ ๏ธ
I made them always show in IS_DEV builds
so we can easily spot them
Then what's the issue
If it's only in dev who cares
I care
add option
GUHHH I LOVE PAYING FOR NITRO AND STILL GETTING ADS
*quests
well thats a choice discord
look at the clips tab
I โค๏ธ web design
I cant even disable game overlays or anything lol
make noop error boundary become notification
i wonder though if you could like detect the amount of space available for an element
THATS ALSO ANNOYING
without rendering it
render an invisible element and measure it
then why make noop show up ๐ญ
if you render invisible element it will still shift layout
but i was thinking measuring it + falling back to popup if there's no space to make it readable
I think we should just not use an error boundary and crash discord instead
Can't be an annoying UI element if there's no UI
nop
@austere talon FormSwitch behaves differently than plugin settings swich
thats why I asked why it uses a different component :P
didnt we complain it wasnt centered
optimally it should be uniform
you did
so which one should be the correct
i dont really care tbh
both are fine to me
it's fine being two different components because of SettingsSection
but they should look the same haha
I like this idea
will put into practice
add descrip`tions to all of them so people dont notice
it was that way
Still happening. Additionally, this plugin needs to suppress replies to blocked users and reactions from blocked users to be truly useful.
I'd love to see this as well!
This looks fine and should probably get merged soon
soon (in 5 years)
Fix(stickerPaste): Resolve conflict with fakeNitro
This resolves a bug where having both stickerPaste and fakeNitro enabled would cause an infinite loop when sending a sticker.
The Problem:
The stickerPaste plugin works by intercepting the sticker send event, preventing the sticker from being sent, and instead pasting a link into the chatbox. However, it did not clear the sticker data from the message composer.
When a user would then press Enter to send the message, ...
I have fixed this bug;
You can find the fix here: https://github.com/Vendicated/Vencord/pull/3719
description smells like ai
@fossil inlet check if this is right etc
very cursed code
try { return } finally {}
does that work lmao
defer in js
no way
wtf
yeah try finallys just work
thats so cursed
pretty sure try-finallys work in every lang
pr seems like ai slop
arguments[2] is an id, not an object
remove cursed try...finally block
I wanted to give you the benefit of the doubt but this is clearly AI-slop, and doesn't even work
@austere talon is stickerPaste meant to let you send more than one sticker in a message
doesnt work for me
fix
guh
Thank you, I was just trying to fix the open bug
But I tested both times though and the plugins both are still working and the bug does actually seem to be fixed. Could you elaborate what the issue is?
I wanted to give you the benefit of the doubt but this is clearly AI-slop, and doesn't even work
Thank you, I was just trying to fix the open bug
But I tested both times though and the plugins both are still working and the bug does actually seem to be fixed. Could you elaborate what the issue is?
I wanted to give you the benefit of the doubt but this is clearly AI-slop, and doesn't even work
Thank you, I was just trying to fix the open bug
But I tested both times though and the plugins both are still working and the bug does actually seem to be fixed. Could you elaborate what the issue is? @Vendicated
???? Holy shit
Try this fix out, I tested it and it works. Idk why he says it doesnt work
https://github.com/Vendicated/Vencord/pull/3719
im sorry, botting github followers?
look at it
waow
yeah that's deffo botted, or my dude works for some kind of international company that has a very large amount of private repos (but based on what else i see i heavily doubt that)
i dont think an internal company would explain it either tbh
it's just so, SO international it dfoesn't make sense
You might have thought the issue was resolved because i did a poor job of describing it in my issue
i did a terrible job of describing the issue in the video
you couldn't watch that and reproduce that
the thing is that the sticker is no longer accessible because the server fell below the boosting tier
but fake nitro bypasses that
and that's what cuases the bug (afaik)
You might have thought the issue was resolved because i did a poor job of describing it in my issue. I'll update it in a bit to be more clear on how to reproduce the issue.
i was confused lol but i see what the issue is now, I managed to get it fixed with stickers that are gifs but not stickers that are png's. Im trying
I'm going to be honest, I don't know if the reason to cause the bug changed over time or if I just somehow didn't notice it back then
I remade CustomRPC settings so it's less horrible
dcefa49 CustomRPC: add party option & overhaul settings ui - Vendicated
don't do this. just make the plugin export a function like buildContextMenuEntries() and import that
undo this change
const DeveloperMode = getUserSettingLazy("appearance", "developerMode")!;
use .map() and just return null to render nothing
We won't add plugins like this, as they offer questionable privacy (Discord has access to your private key) and don't really fit into Discord: you will always have a bad experience if you're looking at the messages without the plugin. Users who want encrypted messaging should just use Signal, Matrix or XMPP
Thanks for the contribution nevertheless
I dont even have this issue to begin with?
maybe its just .gif?
kinda doubt it tho
do you have fake nitro on
did you see this #๐พ-core-development message
patch is overcrowded imo and could prob be moved into a function
vext
The Vencord Chrome Extension is up to date! (v1.13.3)
Kk
@austere talon are u sure form switch should have a margin-bottom
imo the ui should handle that with gap
I just remade discord's old switch
if you don't want margin you can just set hideBorder
then it also won't put the divider underneath
for the settings ui it should just handle gap and dividers itself
I'm gonna do that like I said
im doing it rn
yup
its a lost sticker 
โฆnon-plugin users
Problem
Currently, when Vencord users send fake nitro emojis, they appear as proper emojis for other Vencord users but show as plain text links like [emoji name](https://media.discordapp.net/emojis/12345.png) for users without the plugin. This makes the fake nitro feature less useful since only plugin users can see the emojis properly.
Solution
Modified the FakeNitro plugin to send plain emoji URLs instead of markdown hyperlinks. Discord natively embeds these...
less gap is better
๐ซ Pull Request Rejected โ No Actual Issue Fixed ๐ซ
Hey there! ๐ Appreciate the effort, but this PR isnโt needed ๐
The behavior described here โ fake nitro emojis showing correctly for plugin users and as text for others โ is already the intended and current functionality โ What this PR calls a โproblemโ isnโt actually a bug or issue at all ๐ญ
โ The feature already works as designed: plugin users see proper emojis, non-plugin users see text links.
โ Changing it to send raw URLs d...
we should consider changing it to openAvatar(event, url) and calling stopPropagation inside that function, and pass event from all places, to ensure it doesnt double click anywhere else either
This plugin disables the Ctrl+R / Cmd+R reload shortcut in Discord to prevent accidental reloads (e.g., when trying to press Ctrl+E).
Its still a useful feature some might like?
It doesn't even use definePlugin
I think it does. It shows he made no effort to look through the existing plugins to see how things worked
I've seen this happen with regular unmodded discord as well
this has already been fixed bruh
Korbo tip: if an issue or PR has "The Problem:" and "The Solution:" in it, it's almost certainly AI
if anything that's a sign it's not ai
not sure about that
the only way ai would be able to make a plugin is by looking at the existing files to see how plugins are structured
thus it would know to use definePlugin
the first thing i would do if i was making a vencord plugin for the first time is to look at other plugins
every ai slop pr thus far has had the basic template down right (except the ones that were blatantly hallucinating)
It should be checking if saved status is invisible as well because the user might set the status to go invis during a game
if (settings.store.excludeInvisible && (savedStatus ?? status) === "invisible") return;
I learned how to hit a submit button 
I honestly think (savedStatus ?? status) could be a const but didnt really think it was needed at the same time
guys tf is this ๐ญ
update challenge
is that not the dropdown box element for switching between viewing all plugins, disabled plugins and enabled plugins?
It indeed is
b881b60 FormSwitch: make entire Switch clickable & add ... - Vendicated
dfbffd1 VcNarrator: make nickname fallback to displayNa... - TomW1605
dcefa49 CustomRPC: add party option & overhaul settings ui - Vendicated
40e67ad CustomRPC: add README - Vendicated
25c9ffb Merge branch 'dev' into feat/translation-v3 - lewisakura
I am using a type only namespace import because mouse event is also a global and I specifically want the react version and not the DOM version
You can rename the import from the react
just import directly and shadow the global
I wanted to give you the benefit of the doubt but this is clearly AI-slop, and doesn't even work
Can you please delete this comment, or delete this thread. I was just trying to help and this comment really makes me look bad on my profile
Who tf cares about github profiles
why pin
it's nino he pins everything

whos gonna tell them ๐ญ
cc01bb7 Replace Forms with our own independent componen... - Vendicated
c1556f0 bump to v1.13.3 - Vendicated
b881b60 FormSwitch: make entire Switch clickable & add ... - Vendicated
dfbffd1 VcNarrator: make nickname fallback to displayNa... - TomW1605
dcefa49 CustomRPC: add party option & overhaul settings ui - Vendicated
@austere talon did u see they bumped stable and reverted
lol
and then bumped again
fire
vtest dev2
Now testing! 
I could make it compatible with stable but that would mean a bigger wildcard which I'm not sure is a good idea
ShowHiddenChannels (took 6.4ms):
ID: 823748
Match: ```
/"renderMobileToolbar",()=>{.+?case \i.\i.GUILD_DIRECTORY:(?<=let{channel:(\i).+?)/
extractAndLoadChunks(["initialSelectedDecoration:", "/initialSelectedDecoration:\i,.{0,150}Promise\.all/"], /(?:(?:Promise\.all\(\[)?(\i\.e\("?[^)]+?"?\)[^\]]*?)(?:\]\))?|Promise\.resolve\(\))\.then\(\i\.bind\(\i,"?([^)]+?)"?\)\)/)
NoMosaic (took 6ms):
ID: 938353
Match: ```
/url:(\i).url});return /
actually it probably is
vtest dev2
Now testing! 
Seems like everything is working fine (for now) 
Seems like everything is working fine (for now) 
???
you should merge
Lmao LGTM
Do they know
i like this pr because it actually weakens some of the guarantees we have set up by switching ts-expect-error to ts-ignore
discord pushed crazy shit to canary @limber skiff
oh
it's the i18n update
we just need to update i18n function
blame @errant nacelle for breaking Vencord ๐ (don't)
https://github.com/discord/discord-intl/commit/e8598c6dde1d0f3479a8f216fda90265495e77ae
vtest dev3
Now testing! 
it's not gonna be able to send stable
we're gonna have to push chrome update yet again
Seems like everything is working fine (for now) 
how
dont we specify some raw - apparently we don't
should we add temporary compat that will convert any string using intl: to regex with both old and new hash
vtest dev3 canary
Now testing! 
Seems like everything is working fine (for now) 
vtest dev3
Now testing! 
findComponent((m) => m.prototype?.render?.toString().includes(countDownFilter)...)
findComponent((m) => m.prototype?.render?.toString().includes(countDownFilter)...)
Now testing! 
it just does both
Seems like everything is working fine (for now) 
how what?
what do you think about this
^
it won't work for finds


only broken thing is this
cause it does .includes(regex)
you could do some trick where strings are returned with monkey patched toValue (i think that's what it uses)
and regex the source or whatever it uses
then check at runtime
if it's canary or stable
ehhhhhhh
just temporarily
vtest dev3
Now testing! 
Seems like everything is working fine (for now) 
Seems like everything is working fine (for now) 
so horror๐ญ
as long as it's not dogshit slow
not for finds
so many regex finds seems extremely bad
how about we just wait for them to push to stable
more members joining to ask how to fix 
We're gonna have to lock support again please no ๐ญ
Chrome users gonna be asking about the issue until Monday or longer 
well that sucks

this is irrrlevant
its just context menu patch
Oh
actually idk it still works for me
if settings still work we could just let people update after everything breaks
I made it fallback to this location
meh
i say we just push now
it's fast
try it yourself
it's fine bro
regex is 4x slower but still under 1ms
some of the speed might even be the canonicaliz
but like
it will test the regex to find the right module
so like thousands of times
it's fine
we will remove in a week anyway
none of these regexes are gonna be complex
they're all super linear regex with no groups, lookarounds, wildcards
alr
vtest dev3
Now testing! 
ShowHiddenChannels (took 7ms):
ID: 424785
Match: ```
/(\i).isRoleSubscriptionTemplatePreviewChannel()/
Seems like everything is working fine (for now) 
b881b60 FormSwitch: make entire Switch clickable & add ... - Vendicated
dfbffd1 VcNarrator: make nickname fallback to displayNa... - TomW1605
dcefa49 CustomRPC: add party option & overhaul settings ui - Vendicated
40e67ad CustomRPC: add README - Vendicated
9c5eebe Decor: Fix decoration modal not opening - Nuckyz
vtest dev
Now testing! 
Seems like everything is working fine (for now) 
Seems like everything is working fine (for now) 
Now testing! 
did they already push
Seems like everything is working fine (for now) 
NoBlockedMessages will be broken for now but ehh
The plugin isn't working for me :/

vtest
Now testing! 
Seems like everything is working fine (for now) 
Seems like everything is working fine (for now) 
yeah
plugin that allows users to resize the Discord member list. Creates lots of empty space to the right side when making the Member list larger. Tried doing something with that empty space, but failed. Making it smaller works without issues tho
this is either AI slop or you blatantly ignored the contributing rules
Merge my pr and it won't be 
Wait does my pr even work
I confused myself ๐ญ
Oh yeah it does
oh I guess the shadow they add for nameplates to make the buttons visible made the illusion of being closer together
hm when I shrink my channel bar the buttons move left, cutting off my name, not falling off the right
#๐จ-css-snippets message
if you have the channel width set small, try this i guess
im sorry twin i kinda entered like 2 years of demotivation towards decor
i do appreciate yall keeping it working though
i hope it isn't too much of a burden
i'm really hoping i can find the motivation to do server v2 so it's not backed by such shitty code
vext
The Vencord Chrome Extension is up to date! (v1.13.4)
damn
vtest dev3
Now testing! 
Now testing! 
Seems like everything is working fine (for now) 
Seems like everything is working fine (for now) 
lol they did
bruh
dont worry about it
, it's all good
I forgot about this ๐ญ
Look VERY promising! Hopefully will get revived and pushed real soon! ๐
not this guy ๐ญ
lol what
BetterDiscord plugin vibecoder
vtest dev3
Now testing! 
Ape alt
getIntlMessageFromHash can return a list of strings so this || fails there
too bad
:(
its only temporary anyway
well ill just be making my own wrapper for them then for questify
Bloat
vtest
Now testing! 
NoServerEmojis (had no effect):
ID: 339085
Match: ```
/.get((\i)).nameMatchesChain(\i).reduce(((\i),(\i))=>{/
Seems like everything is working fine (for now) 
vtest dev2
Now testing! 
Seems like everything is working fine (for now) 
Seems like everything is working fine (for now) 
lol I didnt call the function
Developer Education is simply just dead for normal users and including the logout button would still cause a duplicate "Other Settings" section to appear, simplest solution I see is to just remove both because the logout button isnt really necessary in the context menu. Additionally even if you would keep it in, it wouldn't be clickable anyway, as discord prioritizes the other entry with the Changelog tab.
Another, probably more logical option would be to completely remove the "Other Setting...
@odd heath would this work on non-english languages
interestingly one of them would




