Skip to content

SendTimestamps: Redesign modal, replace deprecated forms - #4718

Open
ggtylerr wants to merge 5 commits into
Vendicated:mainfrom
ggtylerr:main
Open

ggtylerr wants to merge 5 commits into
Vendicated:mainfrom
ggtylerr:main

Conversation

@ggtylerr

@ggtylerr ggtylerr commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Describe your Changes

The current modal for SendTimestamps seemed a little clunky after giving it a recent look, so I figured I'd redesign it. Instead of having a dropdown list, it now has a grid of buttons that the user can quickly look through and use. The preview has been replaced with just the raw markdown code (since the preview is already shown to the user.) There's also now an option to copy instead of directly inserting.

Also noticed the Form tags it previously had is now deprecated, so all previous ones now use Vencord's Components instead.

Some quick notes:

  • Not sure how to style the timestamps in the buttons as just regular text and remove the tooltip. Custom styles may override it.
  • Text wrap in the buttons looks a little wonky. The only half decent solution I can think of is either reducing the font size or increasing the modal size, both of which leads to mixed results imo. So right now it's just at the standard md modal size at 0.9em font size.
  • No idea why the codebox isn't formatted properly in heading. Tried paragraph too and it's the same thing.

Screenshots (if applicable)

image

Checklist before submitting

  • I have read the contribution rules and made sure this pull request complies with it
  • This pull request was written by me, and not an AI agent.
  • I understand that if any of the above conditions are not met, this pull request will be closed and I will be banned from future contributions without further warning

@ggtylerr

Copy link
Copy Markdown
Contributor Author

Latest commit also fixes up the plugin description.

Before:

image

After:

image

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.

1 participant