Skip to content
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

Add fitted preview on playground panel #2133

Merged
merged 11 commits into from
Feb 13, 2025

Conversation

burieberry
Copy link
Contributor

@burieberry burieberry commented Feb 10, 2025

  • Display fitted format preview in playground panel
  • Needed to update the existing fitted-format-gallery to match the new format specs

An issue I'm facing is that any place where the cards are pre-rendered, I'm not seeing the changes I made. Might be a caching issue -- will try again.

fitted-preview

Copy link

github-actions bot commented Feb 10, 2025

Host Test Results

    1 files  ±0      1 suites  ±0   23m 58s ⏱️ -1s
763 tests ±0  760 ✔️ ±0  3 💤 ±0  0 ±0 
768 runs  ±0  765 ✔️ ±0  3 💤 ±0  0 ±0 

Results for commit aac8553. ± Comparison against base commit aa779f8.

♻️ This comment has been updated with latest results.

@burieberry burieberry force-pushed the cs-7914-playground-fitted-format branch from b23e604 to 1c27a68 Compare February 11, 2025 23:05
@burieberry burieberry marked this pull request as ready for review February 12, 2025 00:11
@burieberry burieberry requested a review from a team February 12, 2025 00:11
@backspace
Copy link
Contributor

Build #10247 - Cardstack - Percy | Visual testing as a service 2025-02-12 14-06-16

The Percy diffs suggest some unintentional changes elsewhere, or is it a weird merge situation?

@burieberry
Copy link
Contributor Author

The Percy diffs suggest some unintentional changes elsewhere, or is it a weird merge situation?

I updated the BasicFitted component based on the new designs, which is why everything looks different. I'm checking with design whether this is intentional.

Copy link
Contributor

@backspace backspace left a comment

Choose a reason for hiding this comment

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

Assuming the changes shown in Percy are acceptable, looks good!

style={{setContainerSize spec}}
/>
</li>
{{/each}}
Copy link
Contributor

Choose a reason for hiding this comment

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

I appreciate this abstraction 🎉

@burieberry burieberry merged commit 77a8b47 into main Feb 13, 2025
51 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.

2 participants