Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nixcord: Check if using Vencord or vesktop (or both) #821

Closed
wants to merge 0 commits into from

Conversation

ari-rs
Copy link
Contributor

@ari-rs ari-rs commented Jan 29, 2025

nixcord has the option of using discord + vencord and vesktop before the module only worked on vencord this should now apply theming to both vencord and vesktop

Fixes #802

@ari-rs ari-rs changed the title Check if nixcord is using Vencord or vesktop (or both) nixcord: Check if using Vencord or vesktop (or both) Jan 29, 2025
trueNAHO added a commit to trueNAHO/stylix that referenced this pull request Jan 30, 2025
Closes: danth#802
Fixes: e594886 ("nixcord: init (danth#767)")
Link: danth#821

Co-authored-by: NAHO <[email protected]>
Reviewed-by: NAHO <[email protected]>
Copy link
Collaborator

@trueNAHO trueNAHO left a comment

Choose a reason for hiding this comment

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

Thanks for the patch.

For some reason I am unable to push to your master branch. I fixed the formatting and reduced the lib.mkMerge scope in #822. Can you approve that this new PR still works as expected?

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.

Change the nixcord module to be compatible with the vesktop client
2 participants