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

feat(widget-builder): Create blank widget preview for templates #83509

Merged

Conversation

nikkikapadia
Copy link
Member

Before when we didn't have a template selected the widget preview would error out. The mockup had a default or blank preview that I've implemented. Now when you open templates you will see the blank preview. I've also done some quick fixes for the draggable widget preview. TLDR customizing templates was not allowing draggable previews due to ref issues.

Here's a before and after:

Before After
image image

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 15, 2025
@nikkikapadia nikkikapadia marked this pull request as ready for review January 15, 2025 16:07
@nikkikapadia nikkikapadia requested a review from a team as a code owner January 15, 2025 16:07
Copy link

codecov bot commented Jan 15, 2025

Bundle Report

Changes will increase total bundle size by 31.05MB (100.0%) ⬆️⚠️, exceeding the configured threshold of 5%.

Bundle name Size Change
app-webpack-bundle-array-push 31.05MB 31.05MB (100%) ⬆️⚠️

@nikkikapadia nikkikapadia merged commit 8857e10 into master Jan 15, 2025
42 checks passed
@nikkikapadia nikkikapadia deleted the nikki/feat/widget-builder-templates-blank-preview branch January 15, 2025 16:33
andrewshie-sentry pushed a commit that referenced this pull request Jan 22, 2025
Before when we didn't have a template selected the widget preview would
error out. The mockup had a default or blank preview that I've
implemented. Now when you open templates you will see the blank preview.
I've also done some quick fixes for the draggable widget preview. TLDR
customizing templates was not allowing draggable previews due to `ref`
issues.

Here's a before and after:
| Before | After |
|--------|--------|
| <img width="1285" alt="image"
src="https://github.com/user-attachments/assets/f288c319-dde9-4806-8d52-5ea9d542e854"
/> | <img width="1296" alt="image"
src="https://github.com/user-attachments/assets/137a5e16-37eb-4155-a4c6-86a9cc2d026c"
/> |
@github-actions github-actions bot locked and limited conversation to collaborators Jan 31, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Scope: Frontend Automatically applied to PRs that change frontend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants