Skip to content

fix: embeds mobile view - #17

Merged
mezotv merged 3 commits into
mainfrom
fix/embeds/mobile-view
Jul 20, 2025
Merged

fix: embeds mobile view#17
mezotv merged 3 commits into
mainfrom
fix/embeds/mobile-view

Conversation

@tobias-hay

@tobias-hay tobias-hay commented Jul 20, 2025

Copy link
Copy Markdown
Member
  • fix(embed): remove thumbnail on mobile & all fields non inline
  • fix(embed): registry build
  • style(embed): apply styles

@tobias-hay tobias-hay changed the title Fix: Fix Embeds Mobile View fix: Fix Embeds Mobile View Jul 20, 2025
@mezotv
mezotv requested a review from Copilot July 20, 2025 16:46

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes mobile responsiveness issues in Discord embed components by hiding thumbnails on mobile devices and ensuring embed fields stack vertically instead of using grid layout on smaller screens.

  • Hide thumbnail images on mobile devices using responsive classes
  • Change field wrapper layout from grid to flex column on mobile
  • Update registry build with the responsive styling changes

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
registry/discord-embeds/discord-embeds.tsx Added responsive classes to hide thumbnails on mobile and change field wrapper layout to flex column
public/r/discord-embed.json Updated registry content with the new responsive styling changes

Comment thread registry/discord-embeds/discord-embeds.tsx
@mezotv mezotv changed the title fix: Fix Embeds Mobile View fix: embeds mobile view Jul 20, 2025
@mezotv
mezotv merged commit 46bd3e5 into main Jul 20, 2025
2 checks passed
@mezotv
mezotv deleted the fix/embeds/mobile-view branch July 20, 2025 16:51
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