#👾-core-development

1 messages · Page 218 of 1

rugged spire
#

yeah vencord doesn't patch this

jolly egret
#

its always after merge

charred monolithBOT
charred monolithBOT
fossil inlet
#

😭

#

Whar

rugged spire
fossil inlet
rugged spire
#

Oh no

#

what the fuck 😭

#

i understand now

fossil inlet
#

Once I found the patch it was pretty easy to fix

#

(It was also the first patch I tried blobcatcozy

fossil inlet
#

Wonder if those are all needed

limber skiff
#

I mean theres an explanation in every single one

fossil inlet
#

I knew because I read every single one while fixing it

#

Most do tho

limber skiff
#

most are just hiding or replacing components

#

for the channel screen

#

making mentions work and etc

fossil inlet
#

I remember quite a few for stage channels

limber skiff
#

voice

#

stages

#

lol

uneven needle
limber skiff
#

trick to capture variables

uneven needle
#

it doesnt even capture anything tho?

limber skiff
#

it does

#

channel:(\i)

uneven needle
limber skiff
#

oh

#

uhh

#

optimization trick

#

shouldnt matter much

#

but it matches callContainer first (which has no wildcard)

#

and makes sure it's the right thing by matching \i.callContainer after (rematching the previous callContainer)

uneven needle
#

neat i guess

limber skiff
#

the context menu api patches (which apply so many modules) use the same trick

#

match: /Menu,{(?<=\.jsxs?\)\(\i\.Menu,{)/g

fossil inlet
brazen bone
#

Four

limber skiff
#

it's definitely not needed in that case

hot hatch
# rugged spire yeah vencord doesn't patch this

Is there a reason why or is it simply no one just wanted to?
Also I wonder how expensive performance-wise it’d be to uh modify that list of the strings every time the class gets constructed to do something like that (not really familiar with js but it seems like that’s the way you can add one array to another): this.terms = […original_list, …list_from_vencord];

limber skiff
#

no one looked at it

hot hatch
limber skiff
austere talon
#

😨

#

dont look at that code

limber skiff
#

prettier in client when

austere talon
#

it's so bad

limber skiff
#

nah honestly

#

pretty clean code

#

ngl

austere talon
limber skiff
#

sure could be bad practices

austere talon
limber skiff
#

but it's pretty clean

rugged spire
#

fyi there is another array of objects containing an array of settings pages
These must be populated too

limber skiff
rugged spire
# austere talon wpex with prettier

Not only is this useless, it's an anti-feature
browser devtools does this for me, and i can toggle it off
It is actually super useful to be able to toggle it off there

limber skiff
#

electron.webFrame.executeJavaScript((() => (${patchPush.toString()})())(););

#

hehehe

rugged spire
#

Not directly above

hot hatch
#

oh I see what you mean now, thank you, I’ll see what I can do

rugged spire
brazen bone
#

It's for when you want to write lisp, but you're stuck in jsland

austere talon
#

it used an old method to bypass context isolation whcih was very bad

#

xd

limber skiff
#

yeah i saw

#

lol

rugged spire
#

Thanks for confirming that I'm not a dumbass and that this is cursed

austere talon
#

running in preload instead of renderer for no reason

#

i copied that from powercord cause i didn't know what i was doing back then

#

but it's just bad lol

#

bad for security cause you break the isolation of nodejs and the browser
bad for integrity cause if any error happens (like react error), entire app will crash
does not have any benefit

fossil inlet
#

Its pretty good

charred monolithBOT
charred monolithBOT
hot hatch
#

wha

opaque silo
#

are you gonna manually write out each entry or are you gonna automate it somehow

hot hatch
#

uhhhh

#

I mean the vencord settings is just an array of settings like that, I guess I could uh make a func that would gen an array of searchable strings based on that array

#

idrk much tbh

opaque silo
#

2 billion entries for plugin settings

hot hatch
#

sorry wdym?

#

I'm missing the point isob

austere talon
#

ignore her she had too many cookies today

#

her brain is sugar poisoned

opaque silo
#

STOO

#

SOTP

#

STOP

austere talon
#

see

#

she cant even type

opaque silo
austere talon
#

eh it's fine

hot hatch
#

like you want to make recursive search that'd include each plugin name?

opaque silo
austere talon
#

its fine

hot hatch
#

I still don't get what 2 trillion entries there will be
the settings searchbar just searches for the tab name (and maybe some other keywords)

opaque silo
#

you can search the individual settings no?

hot hatch
#

uhh no?

#

huh actually

opaque silo
#

yeah you can

hot hatch
#

yeah you can but ig there's no point in making it search for EACH plugin's settings

opaque silo
#

no point implementing it if if its not the best it can be

austere talon
#

yeah

#

but with how vencord plugin settings works idk how to do it

#

and have to restructure a lot to make it work like discord

#

it's not as simple as just adding vencord's section names

#

a lot of the code has to be rewritten

opaque silo
#

write insane parser

#

maybe just highlight the plugin with the setting for now

#

showing the popup and filtering would be insane

hot hatch
#

oh god I just reliazed that it not only finds the tab that has this string but also removes everything but the string you've searched for if it's in the tab's body death

opaque silo
#

yeah its good design but def horror to patch

hot hatch
#

yeah and the fact that it's just a big list rather than like automatically generated one is ehhhhhh

opaque silo
#

i bet the dev who wrote it made a script to generate it from their source

hot hatch
#

alright ig it's time to set up the dev env to actually start making something

#

how can I test a patch without vencord vscode plugin?

#

if I got it right it can be done with just vencord client?

#

assuming from the plugin src code

sterile oak
hot hatch
hot hatch
#

also back to the settings thing, I just noticed this tab in settings lol

frail skyBOT
#
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

fossil inlet
#

?remind 5 days fix companion erroring when extracting a patch with all:true

shut vineBOT
#

Alright @fossil inlet, in 5 days: fix companion erroring when extracting a patch with all:true

charred monolithBOT
glass jasper
#

Discord exploded clienttheme

limber skiff
#

canary?

glass jasper
oak hull
#

blobcatcozy (me)

limber skiff
#

works for me

glass jasper
#

hum…

grizzled halo
#

works for me too

glass jasper
long stream
#

i think the destkop refresh uses completely different css vars now fubuki_pain

charred monolithBOT
rugged spire
rugged spire
#

I'm probably the first person to ever discover this bug (softlock?)
https://github.com/Vendicated/Vencord/blob/dev/src/main/ipcMain.ts#L111-L113 can error with
UnhandledPromiseRejectionWarning: Error: EMFILE: too many open files, watch '/home/sqaaakoi/.config/Vencord/themes'
Shouldn't this be wrapped in a try catch block (or something similar?)
Completely blocks the desktop client from loading

grave mangoBOT
rugged spire
#

With OpenAsar it would lock up their loading page and without it wouldn't open anything at all

#

If anyone wants to know why this happened, try breaking r2modman in some weird way where it sits in the background after being closed eating many file handles

oak hull
# long stream i think the destkop refresh uses completely different css vars now <:fubuki_pain...

i found a small fix, but its better on dark mode than light mode
no idea why

const newVariableRegex = /(--neutral-([1-9]|[1-9][0-9])-hsl):.*?(\S*)%;/g;
const newLightVariableRegex = /^--neutral-([1-9]|[1-5][0-9])-hsl/g;
const newDarkVariableRegex = /^--neutral-[5-9][0-9]-hsl/g;```

```ts
createStyleSheet("clientThemeNewOffsets", [
        `.theme-light {\n ${genThemeSpecificOffsets(variableLightness, newLightVariableRegex, "--neutral-24-hsl")} \n}`,
        `.theme-dark {\n ${genThemeSpecificOffsets(variableLightness, newDarkVariableRegex, "--neutral-55-hsl")} \n}`,
    ].join("\n\n"));```
opaque silo
#

99% of the client is var based

charred monolithBOT
#

Discord Account

shivam01

What happens when the bug or crash occurs?

I have 3 categories in DMs. First 2 have 5-10 pinned. 3rd is the Group DM category and it has like 15 pinned. Like scrolling or navigating through them, the bottom part of the DM becomes invisible (more than half of the screen can't view the DM list)

Video Proof: https://drive.google.com/file/d/1kSGCMcRXsBplnPSPpPSPScbq0Ejhkp9V/view?usp=sharing

What is the expected behaviour?

I expect my DMs to act normall...

#

Discord Account

shivam01

What happens when the bug or crash occurs?

I have 3 categories in DMs. First 2 have 5-10 pinned. 3rd is the Group DM category and it has like 15 pinned. Like scrolling or navigating through them, the bottom part of the DM becomes invisible (more than half of the screen can't view the DM list)

Video Proof: https://drive.google.com/file/d/1kSGCMcRXsBplnPSPpPSPScbq0Ejhkp9V/view?usp=sharing

What is the expected behaviour?

I expe...

jolly egret
#

oh my god did you need to quote reply the entire message

jolly egret
#

also they edited their message to untick the stable requirement 😁

charred monolithBOT
#

Changelog

Redis Allows Passwords

Allowed a redis password to be inputted into the bot, preventing an error lacking authentication.

File Changes

  1. globals/globals.go
    • Added REDIS_PASS as variable
  2. .env.example
    • Added REDIS_PASS as example variable
  3. main.go
    • Added error for missing password to client console
  4. routes/discord.go
    • Added error for missing password to client console
austere talon
#

into the bot

#

did he use ai for the description

brazen bone
#

The "file changes" is also silly, there's a whole tab for files changed

hot hatch
#

what does \i in find regexp mean? is it vencord custom thingy as I couldn't find such token

fossil inlet
#

It references any JavaScript identifier.

#

If you want to learn more look at the canonicalize functions

hot hatch
#

thank you!

#

just noticed this tho

charred monolithBOT
dusk blaze
#

true

#

useless if u use docker

jolly egret
#

i don't get why you would need password support

#

like unless you're running this on an untrusted machine (you really shouldn't run something like this on an untrusted machine) password shouldn't be needed

frail skyBOT
#
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

long stream
#

if it's on a vulnerable machine then you can just read the .env file triagle

charred monolithBOT
#

While not exactly necessary, I always try to choose redundant security when
possible, so I personally put a password on it. I know its a very niche
case but it does allow for a more clear error if some other random person
does try to do it. Simply put, I don't see a reason not to do it.

On Fri, Oct 11, 2024 at 2:26 PM lewisakura @.***> wrote:

This isn't necessary in the supported Vencloud deployment since it runs
its own isolated Redis instance. Why do you need this?

...

austere talon
charred monolithBOT
rugged spire
brazen bone
#

Probably

rugged spire
#

i am confident that this will be closed as soon as a maintainer looks at it

scenic brook
#

I can only imagine they don't understand what a pull request is

cunning canyon
#

monkey see a big green button appears after changed a file, monkey press

fossil inlet
charred monolithBOT
limber skiff
#

what's unsafe about it?

rugged spire
limber skiff
#

makes sense

charred monolithBOT
austere talon
#

arbitary file write

#

you can write malware to Discord.exe

limber skiff
#

yeah I didnt notice renderer was handling the file writing path

glass jasper
desert cosmos
charred monolithBOT
weary cave
#

is there a way to tell vencord to disable certain plugins?

fossil inlet
#

dont think so

weary cave
#

ex, for legcord, I want to disable the ability to enable the arRPC plugin

fossil inlet
#

wait, why is it called legcord now?

fossil inlet
#

bit full of themselves

weary cave
#

I KNOW RIGHT

dusk blaze
#

idk

#

also is armcord unironically legcord now

weary cave
#

we just fetch the dist

fossil inlet
#

@weary cave guhhh

weary cave
#

bruh

#

is that already in there

fossil inlet
#

yes

weary cave
#

husk

fossil inlet
#

did you change your global to legcord?

weary cave
#

probably

#

one sec

fossil inlet
#

pr a fix or smth if it works

weary cave
#

yeah

charred monolithBOT
opaque silo
#

where were u when armcor was kill

jolly egret
#

would've been more funny to just completely rebrand from ArmCord to armcord and just remove all references to the company

#

i don't think they would be very happy with that one tho

austere talon
#

should have rebranded to 💪cord

#

🦾cord

#

with the emoji

frail skyBOT
#
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

long stream
#

i always thought armcord was an arm-native discord client or something

fossil inlet
#

i always knew armcord was a custom client, but thought it was arm only

charred monolithBOT
#

Discord Account

No response

What happens when the bug or crash occurs?

Cant stream in nitro quality anymore.

What is the expected behaviour?

Success change stream quality.

How do you recreate this bug or crash?

Try to change stream quality.

Errors

No response

Request Agreement

  • [X] I am using Discord Stable or tried on Stable and this bug happens there as well
  • [X] I have read the requirements for opening an issue above
sterile oak
still wasp
brazen bone
#

Why are you colorizing a whitename

jagged cloak
#

LOOOL

haughty valley
limber skiff
#

is this how pronouns in compact mode are supposed to look

charred monolithBOT
limber skiff
#

any suggestions?

#

(besides a readme ill add that later)

charred monolithBOT
charred monolithBOT
woeful sable
#

Legcord

sterile oak
#

what's armcord or legcord that everyone keeps yapping about

glass jasper
fair forum
brazen bone
#

Are they gonna sue the human body

frail skyBOT
#
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

jagged cloak
bleak blaze
#

@uneven needle doesn't open when in vc and clicking button

#

WebpackModule493683:2 Uncaught (in promise) TypeError: Cannot read properties of undefined (reading '_getRecipients')

#

you have to open a DM, first in order for it to open the sidebar chat otherwise it'll error out like that

uneven needle
#

I thought i handled that tf

torpid vessel
#

if in compact mode

limber skiff
#

I'll see

#

thanks for the suggestions

torpid vessel
#

eh

limber skiff
#

before the clan tag is probably the best option

rugged spire
rugged spire
#

i refuse to believe anyone uses this

prime dew
torpid vessel
#

just strip everything because compact mode is supposed to emulate irc

glass jasper
#

showBadgesInChat for compact mode

rugged spire
prime dew
glass jasper
prime dew
#

Vencord's new fork
Bloatcord

torpid vessel
#

omw

rugged spire
prime dew
bleak blaze
#

so you can add an entire server and go through channels and select one and if the server, is selected add a back arrow so once selecting a channel you can go back

#

you’d obviously have to make adjustments to the width and height but it’d work out probably

#

(suggestion) ^

uneven needle
charred monolithBOT
bleak blaze
# uneven needle more trouble than it's worth

yeah you’re right. have you considered adding voice channel members to the right? so when you’re navigating you can see who’s in a call to mute / deafen / disable their camera

#

option to add a voice call i don’t know if that was implemented

uneven needle
#

use this?

limber skiff
#

they are cooking

austere talon
#

👀

limber skiff
#

they already have it in the client lol

long stream
#

ye it's been in the client for a while

#

this uses new intl

limber skiff
#

dope

bleak blaze
uneven needle
#

Why does it need to be added as sidebar then, its sidebarchat anyways

fossil inlet
#

has anyone else noticed whoReacted not updating untill you hover/click

glass jasper
fossil inlet
#

?remind two weeks

shut vineBOT
#

Alright @fossil inlet, in 2 weeks: …

still wasp
bleak blaze
#

easily access the voice chat channel?

#

when there’s multiple people in the call using what you mentioned it’ll be like (+8) rather than rendering all the users

austere talon
#

I didnt invent this xd

#

many js build tools and such do similar things

#

because typescript doesn't allow you to type default exports and using a define function is more powerful because it allows you to use generics

#

for example vencord's plugin definition allows you to have custom members like functions. if you did const plugin: Plugin = {}, then

  1. typescript would complain if you add new members
  2. it wouldn't have the information that this member exists so you can reference it in another member function
charred monolithBOT
charred monolithBOT
opaque silo
#

can the Vencord category get a submenu for these buttons

fossil inlet
opaque silo
#

THERES A CAT ON MY SCREEN

glass jasper
frail skyBOT
#
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

charred monolithBOT
sterile oak
#

ahm, don't mind me

#

someone delete this..

jagged cloak
#

whar

#

oh lmaoo

fossil inlet
jagged cloak
#

happens to everyone at least once unless you educate on git lol

fossil inlet
jagged cloak
#

trol

#

i do not know git i just search how to do whatever with git when i need it

fossil inlet
#

It’s so good

limber skiff
#

just an interactive rebase is already so powerful

#

u can do so much with it to fix issues

brazen bone
#

And you can do so much with it to cause bigger issues

gritty iris
charred monolithBOT
opaque silo
wary relic
glass jasper
jolly egret
#

anyway... vee/lewi/any of the maintainers wanna open an issue?

#

on both the installer and the mod lol

#

because they finally patched the mod too

glass jasper
#

prolly vee will when they wake up or whatever

hot hatch
#

oh wait

#

wrong timestamp

#

lmfao

hot hatch
#

mhm

jolly egret
#

seems a bit unusual for everything to be botted

glass jasper
jolly egret
#

they have actual plugins in the tree

glass jasper
#

prolly skidded from somewhere/someone else is the thing

jolly egret
#

It gets worse while i'm looking over this

#

they changed all the developers in the client mod to themselves

glass jasper
#

They just took their plug-ins, slapped to their name on it and claimed it as their own

hot hatch
somber thorn
#

man some people suck

glass jasper
#

^they are the code skid that stole them

jolly egret
#

no way this person is human

#

@obtuse vessel

#

its a 2016 discord account

grave mangoBOT
glass jasper
jolly egret
#

guessing so

#

Anyway hope this shit stain of a project gets taken down this is pure evil

#

if it was just the license headers and they actually fixed it i wouldn't care but you can't just remove everyone else from the developers list

glass jasper
#

They don’t really care about this kind of stuff

jolly egret
#

well... right now its a license violation and they usually are quite good with that

glass jasper
#

report it

austere talon
#

i think they just don't know better

jolly egret
#

thanks vee

#

wtf i looked at it like 10 seconds ago and the repo is gone

#

did they make them private?

glass jasper
#

Yeah thx, im sorry i wont do that again

glass jasper
jolly egret
#

oh yeah

#

forgor if the main repo was though

#

the installer is gone too so its deleted, right?

austere talon
#

they took it all private

#

probably to have time to fix it

#

valid

#

cant complain

#

i've seen this guy before

#

i really think he just didnt know better

#

else he wouldnt have edited the gpl text 😭

#

i still have to be strict and enforce the license

not just for myself but also for everyone else who contributed, especially cause he stripped out all plugin authors

#

honestly even without knowledge about how licenses works, removing all plugin authors and claiming you're the author kinda is just fucked up

jolly egret
#

Yeah fucking up the headers is kinda just clueless behaviour but removing all the plugin authors is just fucked up

jolly egret
#

The org is gone and they removed their pfp on github

#

i sure wonder what they are doing

rugged spire
spring gorge
spring gorge
opaque silo
frail skyBOT
#
Bad Patches

None

Bad Webpack Finds

None

Bad Starts

None

Discord Errors

None

#
Bad Patches

ImageZoom (had no effect):
ID: 288229
Match: ```
/return.{1,200}.wrapper.{1,200}src:\i,/g


**__FakeNitro (had no effect):__**
ID: `938353`
Match: ```
/renderAttachments\(\i\){let{attachments:(\i).+?;/
Bad Webpack Finds
findComponentByCode(".MEDIA_MODAL_CLOSE", "responsive")
Bad Starts

None

Discord Errors

None

glass jasper
#

rip ig we aint updating tonight

jolly egret
#

uh oh

#

can discord please stop killing fake nitro

fossil inlet
#

guhh

#

time to fix

rugged spire
shut vineBOT
#

@fossil inlet, <t:1728624121:R>: fix companion erroring when extracting a patch with all:true

fossil inlet
# frail sky

from what ive done, it seems like discord has moved some things around in their code

#

cant be bothered to fix it

#

theres also a find that's broken that didnt get reported

fossil inlet
#

got it working but it feels wrong

#

||ping me if you want to see my shitty code||

#

||its also pushed to a branch on my fork if I don't respond||

rugged spire
still wasp
#

@rugged spire you did this

rugged spire
#

mantika a real BetterUploadButton hater

limber skiff
#

I'll fix the patches if no one does

#

gotta merge some stuff to main anyways

glass jasper
#

ViewIcons crashing on canary btw

limber skiff
#

send error

#

they really came back from break days to break 25 patches

glass jasper
limber skiff
#

neither am I

#

😔

#

don't worry about it

glass jasper
jolly egret
#

such is the maintainer life

austere talon
#

did they break everything again

charred monolithBOT
crude hearth
#

and tbh most people probably dont use it

limber skiff
#

I was being sarcastic

jolly egret
brazen bone
#

Is ImageModal the one when you click on an image?

fossil inlet
#

Yea

jolly egret
#

at least in my opinion

charred monolithBOT
limber skiff
fossil inlet
#

shitty fix
look inside
shitty code
wires

limber skiff
#

lol

charred monolithBOT
sterile oak
glass jasper
# charred monolith

They waited a month. I’ve been waiting half a year. People have been waiting longer. You can wait.

jolly egret
#

its besides the point, not everyone wants to use dev builds forever

#

its kinda expected that you get impatient at some point

sterile oak
#

last one is depression for sure

glass jasper
opaque silo
#

am I insane why is day night switcher needed

#

just set theme to auto and have your system do the switching

austere talon
#

ya

jolly egret
#

The changes hit stable btw

fossil inlet
#

Guhh

#

I will fix when I get home

limber skiff
#

ehh no need to hurry

#

I'll only push later

jolly egret
jagged reef
# charred monolith

lol this plugin is definitely chatgpted judging by the amount of random botty comments

jolly egret
#

new programmers have that problem too

#

though... i can't even tell you whats going on here there are multiple easier ways of doing what they did

fossil inlet
#

||oops double ping||

charred monolithBOT
glass jasper
jolly egret
#

bruh

#

what plugin is that that lets you see the integrations without sharing a mutual?

glass jasper
#

Vp showConnections

nimble pendantBOT
frail skyBOT
#
Bad Patches

ImageZoom (had no effect):
ID: 288229
Match: ```
/return.{1,200}.wrapper.{1,200}src:\i,/g


**__FakeNitro (had no effect):__**
ID: `938353`
Match: ```
/renderAttachments\(\i\){let{attachments:(\i).+?;/
Bad Webpack Finds
findComponentByCode(".MEDIA_MODAL_CLOSE", "responsive")
Bad Starts

None

Discord Errors

None

#
Bad Patches

ImageZoom (had no effect):
ID: 288229
Match: ```
/return.{1,200}.wrapper.{1,200}src:\i,/g


**__FakeNitro (had no effect):__**
ID: `938353`
Match: ```
/renderAttachments\(\i\){let{attachments:(\i).+?;/
Bad Webpack Finds
findComponentByCode(".MEDIA_MODAL_CLOSE", "responsive")
Bad Starts

None

Discord Errors

None

rugged spire
#

@fossil inlet you will PR your fix

fossil inlet
rugged spire
#

oh

#

why dont you just PR the FakeNitro attachments fix?

#

inb4 "I didn't fix that"

fossil inlet
#

@rugged spire it does fucky things for banners @placid wave as an example

rugged spire
#

i might be a dumbass

fossil inlet
#

||and other things that arent squares||

rugged spire
#

i tried to imagezoom on that image itself

charred monolithBOT
fossil inlet
#

@rugged spire you love

opaque silo
fossil inlet
opaque silo
#

ai to fix patches .,.,,..,

fossil inlet
#

oh

opaque silo
#

(it will be nin0 locked in a cage)

glass jasper
charred monolithBOT
scenic brook
fossil inlet
scenic brook
#

Hmm true

#

Oh well

sterile oak
bleak blaze
limber skiff
#

pronouns before clan now

charred monolithBOT
limber skiff
#

yeah this image modal change is pretty annoying to fix

fossil inlet
limber skiff
#

havent tried yet

dusk blaze
brazen bone
#

I would guess it means cockroach palace

#

Google translate says it's cookies, not palace though

austere talon
#

cookies

#

well technically it would also mean place

limber skiff
#

it's supposed to support

#

L is whether width and height exists

#

but when rendering the one with "img" tag it simply shows nothing

#

where is it 😭

#

it's the css breaking it

#

LOL this is definitely not a good fix

fossil inlet
limber skiff
#

341176

fossil inlet
#

Alr

limber skiff
#

it's just the css shitting it up

fossil inlet
#

Nice to know I wasn't going crazy the whole time blobcatcozy

limber skiff
#

I dont know how to fix tho

#

typing items is a pain in the ass

scenic brook
limber skiff
#

😭

fossil inlet
#

@limber skiff is it worth making a patch for this to fix it, it is causing crashes right now

limber skiff
#

view icons?

#

crash handler is catching it tho

fossil inlet
#

and all other ImageModal things

fossil inlet
limber skiff
#

if you find a way to fix

#

because it's just a css problem

fossil inlet
#

you can just pass a class

limber skiff
#

play with it in devtools ur gonna see

fossil inlet
#

horror

fossil inlet
#

@limber skiff I FIXED IT

#

FINALLY

#

I FUCKING HATE CSS

limber skiff
#

lmao

#

how did u do it

fossil inlet
#

position: inherit

#

they had it set to absolute

#

which fucked something up

limber skiff
#

do things not get huge?

#

like the vencord icon

fossil inlet
limber skiff
#

haha

#

nice

fossil inlet
#

is that huge?

limber skiff
#

nah

#

that's perfectly normal

#

did you add your own class or edit css

fossil inlet
#

you need a css class with position: inherit !important;

#

if you want this to be the fix

#

where would be the best place to put this class

limber skiff
#

ugh I have no idea

fossil inlet
#

do i just import a css file?

limber skiff
#

does that work?

fossil inlet
#

thats how all the plugins do it

limber skiff
#

yes but I mean outside a plugin

#

place it near discord.tsx

#

just leave it to me regardless tho

#

I'll commit later and add you as co author

fossil inlet
#

alr

limber skiff
#

I got everything done just missing the css fix

fossil inlet
#

i also have a pr for the fakenitro

#

do you want me to close that

limber skiff
#

ya

#

oh

#

if you dont mind sure

rugged spire
#

has that been fixed already

limber skiff
#

I would like do the patch in a different way too

fossil inlet
#

alr

limber skiff
#

thanks a lot regardless

charred monolithBOT
fossil inlet
#

np

#

@limber skiff guhhh

limber skiff
#

uhh

fossil inlet
#

a patch was also needed with the css fix

#

there was bad logic + bad css

limber skiff
#

huh

#

what bad logic

fossil inlet
#

guhh gtg now

#

will explain when i get back if you havent solved it yet

jolly egret
#

are you still working on a fix 😭

charred monolithBOT
rugged spire
#

why is it being discussed in a 16 months since last active thread

fossil inlet
#

@limber skiff from what i can tell now, with the modals fixed, there were two issues.

one: discord sets the position to absolute on all image modals, breaking it when no height/width is provided

two: discord does not pass the height or width if both are not there

limber skiff
#

what's the issue on second

fossil inlet
#

second just makes it really big and not respect any height/width if you dont pass both height and width

#

there is a simple patch for the second

limber skiff
#

it's just that boolean L right

fossil inlet
#

yea

#

i changed it from this

#

, L = !(null == p || 0 === p || null == I || 0 === I);

#

to L = !((null == p || 0 === p) && (null == I || 0 === I));

#

or you could pass style props to the other image modal

limber skiff
#

where are we passing only one

#

and why can't we pass both

#

seems wrong to just do one?

fossil inlet
#

if we set both, we can ruin the aspect ratio

limber skiff
#

yk at this point we could just make L always true

fossil inlet
#

its a patch either way

figure id keep as close as possible

limber skiff
#

I'll see

frail skyBOT
#
Bad Patches

ImageZoom (had no effect):
ID: 288229
Match: ```
/return.{1,200}.wrapper.{1,200}src:\i,/g


**__ShowHiddenChannels (had no effect):__**
ID: `557135`
Match: ```
/(?<=getCurrentClientVoiceChannelId\((\i)\.guild_id\);return)/

FakeNitro (had no effect):
ID: 938353
Match: ```
/renderAttachments(\i){let{attachments:(\i).+?;/

Bad Webpack Finds
findComponentByCode(".MEDIA_MODAL_CLOSE", "responsive")
Bad Starts

None

Discord Errors

None

#
Bad Patches

ImageZoom (had no effect):
ID: 288229
Match: ```
/return.{1,200}.wrapper.{1,200}src:\i,/g


**__ShowHiddenChannels (had no effect):__**
ID: `557135`
Match: ```
/(?<=getCurrentClientVoiceChannelId\((\i)\.guild_id\);return)/

FakeNitro (had no effect):
ID: 938353
Match: ```
/renderAttachments(\i){let{attachments:(\i).+?;/

Bad Webpack Finds
findComponentByCode(".MEDIA_MODAL_CLOSE", "responsive")
Bad Starts

None

Discord Errors

None

fossil inlet
#

SHC time 🎉

fossil inlet
#

its an easy fix

#
                {
                    // Do not show confirmation to join a voice channel when already connected to another if clicking on a hidden voice channel
                    match: /(?<=getBlockedUsersForVoiceChannel\((\i)\.id\);return)/,
                    replace: (_, channel) => `!$self.isHiddenChannel(${channel})&&`
                },
charred monolithBOT
limber skiff
#

okay I will fix stuff at maximum today

#

already took too long :)

fossil inlet
jolly egret
#

lets grow

fossil inlet
#

@limber skiff let me know when you fix the image-modal things, I'm curious how you'll fix it in the end

limber skiff
#

aight

charred monolithBOT
fossil inlet
#

hiiiii i really want a niche plugin that doxxes me

#

im sure they put effort into it, but this is just a stupid idea

jolly egret
#

why would anybody want that wtf

#

straight up just doxxes you for no reason

#

Not even useful for IRL friends because you're very likely to just be in the same area and why the weather of all things?

#

Like if you want to doxx yourself use this as a 3rd party plugin why would this be official 😭

charred monolithBOT
rugged spire
rugged spire
glass jasper
rugged spire
glass jasper
rugged spire
#

clueless

#

you know thats an issue if you aren't streaming

#

just don't set the location too accurate

frail skyBOT
#
Bad Patches

ImageZoom (had no effect):
ID: 288229
Match: ```
/return.{1,200}.wrapper.{1,200}src:\i,/g


**__ShowHiddenChannels (had no effect):__**
ID: `557135`
Match: ```
/(?<=getCurrentClientVoiceChannelId\((\i)\.guild_id\);return)/

FakeNitro (had no effect):
ID: 938353
Match: ```
/renderAttachments(\i){let{attachments:(\i).+?;/

Bad Webpack Finds
findComponentByCode(".MEDIA_MODAL_CLOSE", "responsive")
Bad Starts

None

Discord Errors

None

#
Bad Patches

ImageZoom (had no effect):
ID: 288229
Match: ```
/return.{1,200}.wrapper.{1,200}src:\i,/g


**__ShowHiddenChannels (had no effect):__**
ID: `557135`
Match: ```
/(?<=getCurrentClientVoiceChannelId\((\i)\.guild_id\);return)/

FakeNitro (had no effect):
ID: 938353
Match: ```
/renderAttachments(\i){let{attachments:(\i).+?;/

Bad Webpack Finds
findComponentByCode(".MEDIA_MODAL_CLOSE", "responsive")
Bad Starts

None

Discord Errors

None

jolly egret
#

now showhiddenchannels have a broken patch? incredible

rugged spire
#

discord will bomb it in 3 months anyways

jolly egret
#

please keep gatekeeping fixes we love it 😋

glass jasper
#

ong

rugged spire
#

i hate discord-types

#

so out of date

jolly egret
fossil inlet
#

theres a pr for that

jolly egret
#

neat

charred monolithBOT
#

Fix FakeNitro patch for rendering FakeStickers
This patch might be a little sensitive but it doesn't currently conflict with any other code in the module and I don't think it will
An alternative less sensitive patch would be a lookbehind for the function's name "renderAttachments"

To test this patch enable these 3 options and send a FakeNitro's FakeSticker
image
If the patch applied correctly it should ...

inland fable
#

RAHHHH

jolly egret
#

new plugin FixImagesQuality
Is a one line change to FakeNitro

#

Did you write that with AI? 😭

fossil inlet
#

by default that is the main branch

#

the last commit on dev is new plugin fix images quality

#

even if you are pr-ing to the dev branch

inland fable
#

the title did itself

#

WHAT

fossil inlet
#

it does not change the title

inland fable
#

I DIDNT EVEN NOTICE

#

MY COMMIT TITLE WAS FIX FAKENITRO PATCH

jolly egret
#

it shouldn't be anything to do with what branch you are targeting or their commits

inland fable
#

anyway i fixed title in the pr

fossil inlet
#

has anyone ever seen that before

inland fable
rugged spire
#

close devtools and reopen

charred monolithBOT
rugged spire
#

in theory?

mortal marlin
#

yeah. tho the only reason I actually decided to implement it is because I'm in way too many servers

jolly egret
#

unless you actually actively use like 50 servers? (wtf is wrong with you)

dusk blaze
limber skiff
#

view icons working as intended

#

totally not stretched

limber skiff
# limber skiff

@fossil inlet doesnt really work properly without setting both height and width now lol

#

ughh wtf did they do

limber skiff
#

img zoom patch fix

{
    find: ".contain,SCALE_DOWN:",
    replacement: {
        match: /\.slide,\i\),/g,
        replace: `$&id:"${ELEMENT_ID}",`
    }
},
#

@fossil inlet okay so basically the issue is:

when an image doesnt have width and height it uses the plain img element, which the css makes the image disappear. fixing the css and changing position to inherit has the issue of images being too big (from my testing)

#

and then, patching the boolean check for if an image has width and height to use their other component has the same issue with css, or if you do put a width or height (not both) then the image gets fucked up

#

(ImageZoom also doesnt work properly when it uses the plain img element)

#

so we need to fix images getting fucked up if you miss a height or width, fix them being too big, and also make it always use the custom component

#

(they bullied us on this one)

cunning canyon
#

imagezoom doesnt work when you switched image in carousel too

charred monolithBOT
rugged spire
fossil inlet
#

Even if that is the case

#

I can think of another way to easily patch it

charred monolithBOT
fossil inlet
#

@limber skiff my fix works fine for me Shrug

limber skiff
#

which

fossil inlet
#

also
in what file would i put the needed patch

fossil inlet
limber skiff
#

which fix

fossil inlet
#

the patch + css

limber skiff
#

send here

#

ill test again later

fossil inlet
#
        {
            find: "SCALE_DOWN:",
            replacement: {
                match: /!\(null==(\i)\|\|0===\i\|\|null==(\i)\|\|0===\i\)/,
                replace: (_, width, height) => `!((null == ${width} || 0 === ${width}) && (null == ${height} || 0 === ${height}))`
            }
        }

#

you could also patch the img component with a style prop

#

that has the width and/or height

charred monolithBOT
frail skyBOT
#
Bad Patches

ImageZoom (had no effect):
ID: 288229
Match: ```
/return.{1,200}.wrapper.{1,200}src:\i,/g

Bad Webpack Finds
findComponentByCode(".MEDIA_MODAL_CLOSE", "responsive")
Bad Starts

None

Discord Errors

None

#
Bad Patches

ImageZoom (had no effect):
ID: 288229
Match: ```
/return.{1,200}.wrapper.{1,200}src:\i,/g

Bad Webpack Finds
findComponentByCode(".MEDIA_MODAL_CLOSE", "responsive")
Bad Starts

None

Discord Errors

None

glass jasper
#

RIP imagezoom

jolly egret
#

viewicons and stuff are still borked

glass jasper
#

This ran on dev

#

They are fix on dev

jolly egret
#

i haven't been keeping up

rugged spire
#

time to merge in dev

glass jasper
fossil inlet
bleak blaze
#

it’s been almost a week now and viewicons, still not resolved yet

#

height and width go wild

swift fjord
#

be patient

bleak blaze
rugged spire
# charred monolith

@limber skiff I've just been trying this out today and I've found a few things that feel weird about the plugin
I wouldn't be surprised if these existed in the PronounDB plugin too

  1. The tooltip showing "Pronouns" feels off-center, because it includes the
  2. Pronouns are not cached by the plugin itself, so every time the client is restarted, pronouns do not show until the user profile is opened, defeating the point of the plugin

1 is trivial to solve, the second isn't so much. How do you propose solving that? Cache in DataStore every time it's rendered, if Discord can't provide pronouns then load from cache?

#

WHAT ARE YOU

#

must be for those annoying particles

limber skiff
#

idk about 2 tho

rugged spire
limber skiff
#

huh

#

oh so you mean the tooltip should not include the •

rugged spire
#

Yes

limber skiff
#

put it inside a div with the classes

#

and then both spans there

#

without the classes

#

and see if it works

#

check compact too

rugged spire
#

I never actually explained this but

#

I'm using timestampInline for the spacing

rugged spire
#

why?

#

ah

#

i see

#

the timestamp class itself adds this massive margin on both sides

#

should the • even be used in compact mode?

limber skiff
#

yeah

rugged spire
#

is that how it's always been?

#

oh also

#

how about server profiles?

limber skiff
#

wdym

rugged spire
#

server specific pronouns aren't considered

limber skiff
#

oh

#

that needs to be fixed

rugged spire
#

it just looks like pronoundb code was ripped out without even refactoring away from that system blobcatcozy

charred monolithBOT
#

@Vendicated merge

20 Eki 2024 Paz 17:43 tarihinde jamesbt365 @.***> şunu
yazdı:

Any interest in reopening and merging this? The default behaviour (left
click) without any plugins in my opinion should be listed as more people
are not going to be using BetterUploadButton than are using the plugin.


Reply to this email directly, view it on GitHub
https://github.com/Vendicated/Vencord/pull/2954#issuecomment-2425020666,
or unsubscribe
<https://github.com/n...

jolly egret
#

yeah vee MERGE (best way to get something merged is to to demand it)

torpid vessel
dusk blaze
crude hearth
charred monolithBOT
fossil inlet
#

@limber skiff have you fixed image zoom yet?

limber skiff
#

ya

charred monolithBOT
charred monolithBOT
#

Discord Account

.serious_person_not_found

What happens when the bug or crash occurs?

The bug happens then i wanna see someone pfp or banner but instead of image there appear grey rectangle
Screenshot_81

What is the expected behaviour?

I expect Vencord/Discord to makes avatars and banners in user profiles clickable and being able to download them

How do you recreate this bug or crash?

...

fossil inlet
#

?remind 23.5 hours open link on stickers

shut vineBOT
#

Alright @fossil inlet, in 23 hours and 30 minutes: open link on stickers

frail skyBOT
#
Bad Patches

ImageZoom (had no effect):
ID: 288229
Match: ```
/return.{1,200}.wrapper.{1,200}src:\i,/g

Bad Webpack Finds
findComponentByCode(".MEDIA_MODAL_CLOSE", "responsive")
Bad Starts

None

Discord Errors

None

#
Bad Patches

ImageZoom (had no effect):
ID: 288229
Match: ```
/return.{1,200}.wrapper.{1,200}src:\i,/g

Bad Webpack Finds
findComponentByCode(".MEDIA_MODAL_CLOSE", "responsive")
Bad Starts

None

Discord Errors

None

charred monolithBOT
fossil inlet
#

is it possible to specify the location vencord is loaded from in the command line for discord desktop

topaz thistle
#

Anyone here aware of discord desktop rendering video calls with some cursed electron feature or do they just use plain html elements as the web version does?

rugged spire
topaz thistle
#

Is that some electron feature that allows to render raw-data from native into the webview?

charred monolithBOT
shut vineBOT
#

@fossil inlet, <t:1729466835:R>: open link on stickers

frail skyBOT
#
Bad Patches

ImageZoom (had no effect):
ID: 288229
Match: ```
/return.{1,200}.wrapper.{1,200}src:\i,/g

Bad Webpack Finds
findComponentByCode(".MEDIA_MODAL_CLOSE", "responsive")
Bad Starts

None

Discord Errors

None

#
Bad Patches

ImageZoom (had no effect):
ID: 288229
Match: ```
/return.{1,200}.wrapper.{1,200}src:\i,/g

Bad Webpack Finds
findComponentByCode(".MEDIA_MODAL_CLOSE", "responsive")
Bad Starts

None

Discord Errors

None

opaque silo
#

not again 😭

jolly egret
#

we hadn't fixed them yet !!

charred monolithBOT
limber skiff
#

@fossil inlet can you put up together our tests and pr a fix

#

I'm busy to able to find something that works correctly and put it together

#

I'll still review and we can do necessary changes based on it

#

if you search for imagezoom in this channel you can find a fix for it's patch

#

but ideally view icons should work with imagezoom too

#

use the patch file I sent here as a starting point

fossil inlet
crude hearth
#

WHY IS IMAGE ZOOM STILL BROKEN

#

@austere talon fix or I will no longer donate to you

fossil inlet
#

you love

crude hearth
#

be faster

fossil inlet
crude hearth
#

GOOD

rugged spire
#

wow

#

did you know your arRPC server isn't running?

#

you are insane for having this toggled on

charred monolithBOT
crude hearth
#

@austere talon merge

jolly egret
#

megre

crude hearth
#

I think it looks good

#

yes you should merge

#

no need to review

fossil inlet
rugged spire
#

i don't like the fact a whole "API" was implemented for this

#

even though it was just a patch fix

fossil inlet
#

seriously

rugged spire
#

blobcatcozy idk

fossil inlet
#

imagemodals are used by more than one plugin

rugged spire
#

yop

fossil inlet
#

and without the patch, you cant set a min height or min width

rugged spire
#

was anything actually setting the min height/width

rugged spire
#

what

fossil inlet
#

for good reason

rugged spire
#

where

fossil inlet
#

view icons

rugged spire
#

ah

#

thanks for explaining to me like i am stupid (i am a bit)

jolly egret
fossil inlet
#

@rugged spire testing more

jolly egret
#

it failed the tests!!!

fossil inlet
#

and i need to add a size limit to server info as well

#

its massie without it

rugged spire
#

serverinfo needs a rewrite anyway

#

should i redo serverinfo to be more useful

crude hearth
#

what if imagezoom got merged instead

bleak blaze
#

Instead of pressing a button make something when hovering over the icon it’ll display information

#

similar to betterdiscord

rugged spire
bleak blaze
#

it’s way better than always manually pressing button

bleak blaze
austere talon
rugged spire
#

make some things less prominent and add stuff like features + some other values

#

i'm busy doing something called making my laptop useful again though

jolly egret
#

Will give it a little test if i remember too later

long stream
#

wait my plugin broke as well?

#

nooo

austere talon
jolly egret
#

actually nvm, don't understood what made it not work for me, i tried after a user reported it and it wasn't working properly but i guess it is

#

I'm guessing i am also capable of user error

fossil inlet
#

@limber skiff i swear im going crazy

long stream
#

my plugin is still going strong

jolly egret
#

gonna bribe discord devs to break it brb

long stream
#

smh

glass jasper
frail skyBOT
#
Bad Patches

ImageZoom (had no effect):
ID: 288229
Match: ```
/return.{1,200}.wrapper.{1,200}src:\i,/g


**__OpenInApp (had no effect):__**
ID: `740605`
Match: ```
/\i\.\i\.isProtocolRegistered\(\)(.{0,100})window.open/g
Bad Webpack Finds
findComponentByCode(".MEDIA_MODAL_CLOSE", "responsive")
findComponentByCode(".mutualFriendItem]")
Bad Starts

None

Discord Errors

None

jolly egret
#

more breakage

frail skyBOT
#
Bad Patches

ImageZoom (had no effect):
ID: 288229
Match: ```
/return.{1,200}.wrapper.{1,200}src:\i,/g


**__OpenInApp (had no effect):__**
ID: `740605`
Match: ```
/\i\.\i\.isProtocolRegistered\(\)(.{0,100})window.open/g
Bad Webpack Finds
findComponentByCode(".MEDIA_MODAL_CLOSE", "responsive")
findComponentByCode(".mutualFriendItem]")
Bad Starts

None

Discord Errors

None

austere talon
#

the new modal stuff seems like a bitch to work with

charred monolithBOT
fossil inlet
#

the open in app fix looks easy

#

i just dont know how to use it to test it

fossil inlet
#

LMFAO

fossil inlet
#

ignore that

#

its just a hardcoded true

#

when it should be false

austere talon
#

your pr didnt update any other plugins

fossil inlet
austere talon
#

you didnt add width / height to any of them

fossil inlet
#

didnt need them

austere talon
austere talon
#

anyway another thing

#

this in imageZoom

#

pushes the image modal down so it's off centre

fossil inlet
austere talon
#

cause it didnt happen before

fossil inlet
austere talon
#

it also happens on stock discord images

fossil inlet
austere talon
fossil inlet
#

@austere talon do you plan to merge to main sometime tonight or tomorrow

austere talon
#

GUHHH

#

pickle

#

how to webpack find this 😭