Skip to content

refactor: improve canvas preview and layout consistency #23

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Mar 13, 2025

Conversation

benjaminpreiss
Copy link
Collaborator

No description provided.

- Refactor CanvasPreview component to support "tiny" and "post" variants
- Remove redundant padding and styling from CanvasAndReplies
- Clean up Reply component layout and remove duplicate timestamp
- Adjust Header component styling for better visual consistency
- Update ProfileButton dimensions to use tailwind spacing units
- Optimize preview rendering by separating and sorting elements

The changes improve the overall visual consistency and component
organization while adding more flexible preview options for different
canvas display contexts.
Refactors the canvas preview component to display content in a structured layout:
- Primary image/content shown full width
- Secondary images as scrollable thumbnails
- Text content displayed below with optional line limit
- Extracts PreviewFrame component for reusability
- Add padding and spacing between elements
- Style reply buttons with consistent design
- Only show secondary apps container when there are apps
- Move text content preview inside padding container
- Improve layout and visual hierarchy of reply section
@marcus-pousette marcus-pousette marked this pull request as ready for review March 13, 2025 13:33
@marcus-pousette marcus-pousette self-requested a review as a code owner March 13, 2025 13:33
@marcus-pousette marcus-pousette merged commit 36c8576 into master Mar 13, 2025
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