Skip to content

feat: CSS Snippet Manager#345

Closed
ArjixWasTaken wants to merge 10 commits into
Vendicated:mainfrom
ArjixWasTaken:css-snippet-manager
Closed

feat: CSS Snippet Manager#345
ArjixWasTaken wants to merge 10 commits into
Vendicated:mainfrom
ArjixWasTaken:css-snippet-manager

Conversation

@ArjixWasTaken
Copy link
Copy Markdown
Contributor

@ArjixWasTaken ArjixWasTaken commented Dec 16, 2022

About

Uses the message accessories API to add the following buttons Apply, Update, Remove.

  • Apply: Saves the code snippet in the plugin settings and appends it to the head.
  • Update: Appears when a message (of an installed snippet) has been edited, clicking it would simply use the new code.
  • Remove: Removes the snippet from the plugin settings and from the head.
Screenshots

TODO:

  • Implement the settings UI
  • Figure out why the elements do not re-render until the mouse cursor does not intersect with it.

Comment thread src/plugins/cssSnippetManager.tsx Outdated
Comment thread src/plugins/cssSnippetManager.tsx Outdated
Comment thread src/plugins/cssSnippetManager.tsx Outdated
@ArjixWasTaken ArjixWasTaken marked this pull request as ready for review December 18, 2022 20:11
@ArjixWasTaken
Copy link
Copy Markdown
Contributor Author

ArjixWasTaken commented Dec 18, 2022

image
image

@ArjixWasTaken ArjixWasTaken changed the title CSS Snippet Manager feat: CSS Snippet Manager Dec 19, 2022
Repository owner deleted a comment from 144reasons Dec 19, 2022
Repository owner deleted a comment from ArjixWasTaken Dec 19, 2022
Comment thread src/plugins/cssSnippetManager.tsx Outdated
Comment thread src/plugins/cssSnippetManager.tsx Outdated
Comment thread src/plugins/cssSnippetManager.tsx Outdated
Comment thread src/plugins/cssSnippetManager.tsx Outdated
Comment thread src/plugins/cssSnippetManager.tsx Outdated
Comment thread src/plugins/cssSnippetManager.tsx Outdated
Comment thread src/plugins/cssSnippetManager.tsx Outdated
Comment thread src/plugins/cssSnippetManager.tsx Outdated
@ArjixWasTaken
Copy link
Copy Markdown
Contributor Author

image

@ArjixWasTaken
Copy link
Copy Markdown
Contributor Author

ven will now merge

@ArjixWasTaken
Copy link
Copy Markdown
Contributor Author

Note: Because ven requested that all the snippets are in one style node, you can no longer view the entire CSS if you have too many snippets, the browser just gives up and has ... after some point

@ArjixWasTaken
Copy link
Copy Markdown
Contributor Author

the linter is retarded...

image
image
image

@Vap0r1ze
Copy link
Copy Markdown
Contributor

its not the linter its you, you didnt type it

Comment thread src/plugins/cssSnippetManager.tsx Outdated
@ArjixWasTaken
Copy link
Copy Markdown
Contributor Author

its not the linter its you, you didnt type it

I typed it, typescript recognizes it and does not complain.

@Vap0r1ze
Copy link
Copy Markdown
Contributor

The Guild type does not have a channels property idk what to tell you, you can ctrl + click getGuild to find that out

@ArjixWasTaken
Copy link
Copy Markdown
Contributor Author

ArjixWasTaken commented Dec 25, 2022

The Guild type does not have a channels property idk what to tell you, you can ctrl + click getGuild to find that out

Then you must be blind, cause I did Guild & { channels } as the type.

Edit: I was just dum dum for not looking at the line number the linter complained for...

ArjixWasTaken and others added 2 commits December 25, 2022 22:23
Co-authored-by: Justice Almanzar <superdash993@gmail.com>
@ArjixWasTaken
Copy link
Copy Markdown
Contributor Author

ven will merge

@SammCheese
Copy link
Copy Markdown
Contributor

Ven will not merge

@ArjixWasTaken
Copy link
Copy Markdown
Contributor Author

@SammCheese 😭

@SammCheese
Copy link
Copy Markdown
Contributor

Cursed to be unmerged. Such is the fate of every PR on Vencord

@ArjixWasTaken
Copy link
Copy Markdown
Contributor Author

I have sent the assassins, if you don't merge within 5 business days your life is over.

@C10udburst
Copy link
Copy Markdown
Contributor

I have sent the assassins, if you don't merge within 5 business days your life is over.

he will not. (he forgot what the merge PR button looks like)

@ArjixWasTaken ArjixWasTaken deleted the css-snippet-manager branch January 18, 2023 17:24
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.

6 participants