Skip to content

Feat - Separate embed settings and add multiple embeds#667

Merged
7w1 merged 8 commits intoSableClient:devfrom
nushea:muliple-previews
Apr 20, 2026
Merged

Feat - Separate embed settings and add multiple embeds#667
7w1 merged 8 commits intoSableClient:devfrom
nushea:muliple-previews

Conversation

@nushea
Copy link
Copy Markdown
Contributor

@nushea nushea commented Apr 12, 2026

Description

This PR proposes the moving of embed settings to their own category under General:
image

Additionally it adds multiple embeds in one message:
image

Showing them all is however just a setting that may be turned off (it is on by default) since the embeds respect the aspect ratio of the thumbnail so a list of embeds may be uneven which might appear unaesthetic to some, as per this example:
image

Fixes #

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings

AI disclosure:

  • Partially AI assisted (clarify which code was AI assisted and briefly explain what it does).
  • Fully AI generated (explain what all the generated code does in moderate detail).

My pathological avoiding anxiety has empowered me to create this instead of doing a more important task.

@nushea nushea requested review from 7w1 and hazre as code owners April 12, 2026 08:00
@7w1 7w1 enabled auto-merge April 20, 2026 15:52
auto-merge was automatically disabled April 20, 2026 17:54

Head branch was pushed to by a user without write access

@nushea
Copy link
Copy Markdown
Contributor Author

nushea commented Apr 20, 2026

The merge has somehow been broken by #695 but ive fixed it so its good now

@7w1 7w1 enabled auto-merge April 20, 2026 17:56
@7w1 7w1 added this pull request to the merge queue Apr 20, 2026
Merged via the queue into SableClient:dev with commit 4e9c349 Apr 20, 2026
12 checks passed
@nushea nushea deleted the muliple-previews branch April 20, 2026 18:30
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.

2 participants