Skip to content

Components v2#146

Merged
doluk merged 8 commits intomahtoid:masterfrom
FroostySnoowman:master
Jan 8, 2026
Merged

Components v2#146
doluk merged 8 commits intomahtoid:masterfrom
FroostySnoowman:master

Conversation

@FroostySnoowman
Copy link
Contributor

@FroostySnoowman FroostySnoowman commented Dec 1, 2025

This implements Components v2. A highly requested feature by the users of chat-exporter.

Demo: https://youtu.be/9Bn6yD9AmVI
Test File: https://mystb.in/f133a50c5f09a85ff5

@FroostySnoowman
Copy link
Contributor Author

FroostySnoowman commented Dec 1, 2025

Tests export with everything flawless.
export.html

@mahtoid
Copy link
Owner

mahtoid commented Dec 19, 2025

@FroostySnoowman
Code wise, everything seems fine. Only a few things, but nothing worth holding up PR for.

The only problem for me is styling is way too different from the direction the product sets, we go for 1:1 with Discord without corners cut (as best as we can).

The stuff for spoilers on hover, the Accessory thumbnail, buttons etc.
It's up to you if you want to take another go at that or leave to me and I'll try get to over the Christmas period.

@doluk
Copy link
Collaborator

doluk commented Dec 20, 2025

  • italic text in textDisplay isn't working (looks like it is consuming a linebreak too much)
image
  • default selection in selects isn't working
image
  • media galery is missing the spoiler text (works for normal files in a message). It also shouldn't show the description/alt of an item marked as spoiler
image

Expect the visual issues. I can confirm it works well with discord.py and pycord

Copy link
Collaborator

@doluk doluk left a comment

Choose a reason for hiding this comment

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

Last minor changes to flow and look and we can merge. Great work!

Copy link
Collaborator

@doluk doluk left a comment

Choose a reason for hiding this comment

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

lgtm

@doluk doluk merged commit c2b3328 into mahtoid:master Jan 8, 2026
2 checks passed
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