Skip to content

feat(Plugin): InvisibleChat#349

Merged
Vendicated merged 22 commits into
Vendicated:mainfrom
SammCheese:invisible-chat
Jan 7, 2023
Merged

feat(Plugin): InvisibleChat#349
Vendicated merged 22 commits into
Vendicated:mainfrom
SammCheese:invisible-chat

Conversation

@SammCheese
Copy link
Copy Markdown
Contributor

Port of Invisible Chat for Vencord.

Features and Design are kept to a minimum for now.

image
image

Comment thread src/plugins/invisiblechat/components/EncryptionModal.tsx Outdated
Co-authored-by: Ven <vendicated@riseup.net>
@Vendicated
Copy link
Copy Markdown
Owner

Using divs with hardcoded colour for text is a bad idea, because it means zero theme support and might even look terrible on stock light theme. Please use Form components (Forms.FormText, Forms.FormDivider, etc) instead

@Vendicated
Copy link
Copy Markdown
Owner

the library is also pretty big so it'd be good to fetch it on demand from a cdn like unpkg or cdnjs

Comment thread src/plugins/invisiblechat/index.tsx
Comment thread src/plugins/invisiblechat/index.tsx Outdated
@SammCheese
Copy link
Copy Markdown
Contributor Author

you will merge

@SammCheese
Copy link
Copy Markdown
Contributor Author

now you will merge

@ArjixWasTaken
Copy link
Copy Markdown
Contributor

now you will merge

:husk:

@SammCheese SammCheese requested review from ArjixWasTaken and Vendicated and removed request for ArjixWasTaken and Vendicated December 20, 2022 19:15
@SammCheese
Copy link
Copy Markdown
Contributor Author

SammCheese commented Dec 20, 2022

Bruh GitHub Mobile is horrid

@SammCheese SammCheese requested review from Vendicated and removed request for ArjixWasTaken December 20, 2022 21:36
@SammCheese
Copy link
Copy Markdown
Contributor Author

VEN

@ArjixWasTaken
Copy link
Copy Markdown
Contributor

instead of merging upstream changes, I'd suggest rebasing

@SammCheese
Copy link
Copy Markdown
Contributor Author

How why

@SammCheese SammCheese removed the request for review from Vendicated December 22, 2022 17:53
Comment thread src/plugins/invisiblechat/index.tsx Outdated
Comment thread src/plugins/invisiblechat/index.tsx Outdated
Comment thread src/plugins/invisiblechat/index.tsx Outdated
Comment thread src/plugins/invisiblechat/index.tsx
Comment thread src/plugins/invisiblechat/index.tsx Outdated
Comment thread src/plugins/invisiblechat/index.tsx Outdated
Comment thread src/utils/dependencies.ts Outdated
@SammCheese
Copy link
Copy Markdown
Contributor Author

@Vendicated venven

@ArjixWasTaken
Copy link
Copy Markdown
Contributor

ven will not merge

@SammCheese
Copy link
Copy Markdown
Contributor Author

SammCheese commented Dec 26, 2022

WAAAAA :trollface:

@SammCheese
Copy link
Copy Markdown
Contributor Author

@Vendicated it's been 10 days.

@ArjixWasTaken
Copy link
Copy Markdown
Contributor

We are doomed, we should vote for megu to be in charge.

@SammCheese
Copy link
Copy Markdown
Contributor Author

Actually Agreed

Copy link
Copy Markdown
Owner

@Vendicated Vendicated left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay looks good from my side now, but I did a bunch of changes so please see if you still like everything

@Vendicated Vendicated changed the title feet(Plugin): InvisibleChat feat(Plugin): InvisibleChat Jan 7, 2023
@Vendicated Vendicated merged commit 06aa72c into Vendicated:main Jan 7, 2023
@SammCheese SammCheese deleted the invisible-chat branch January 7, 2023 21:53
EtorixDev added a commit to EtorixDev/Vencord that referenced this pull request Aug 20, 2025
* PR.

* Update definitions.tsx

* Typo + cleanup.

* Revert breaking change.

* Fix image extension flaw + embed context flaw.

* Fix typo.

* Label.

* Video + label fixes.

* Label improvement.

* Content type improvements.

---------

Co-authored-by: thororen <78185467+thororen1234@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants