-
Notifications
You must be signed in to change notification settings - Fork 3
Temp/change preview dimension #464
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
Temp/change preview dimension #464
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
...ugin/src/messaging/pluginMessage/containerToPluginMessage/PreviewDimensionResponseMessage.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just left some code clean-up comments, but overall, I understand the concern, and the fix feels okay with the current approach 🙏, even if, as you said, it had more sense to make it like the modal method.
packages/field-plugin/src/createFieldPlugin/createPluginActions/callbackQueue.ts
Outdated
Show resolved
Hide resolved
packages/field-plugin/src/createFieldPlugin/createPluginActions/callbackQueue.ts
Outdated
Show resolved
Hide resolved
packages/field-plugin/src/createFieldPlugin/createPluginActions/createPluginActions.ts
Outdated
Show resolved
Hide resolved
packages/field-plugin/src/createFieldPlugin/createPluginActions/createPluginActions.ts
Outdated
Show resolved
Hide resolved
Hey @Dawntraoz 🙌 Thanks for reviewing it! 🫶 I'll apply the changes, now, so we can unlock the task 🙏 |
What?
Possible solution for: https://github.com/storyblok/field-plugin/pull/463/files#r2231327295
Screen.Recording.2025-07-25.at.12.44.52.mov
PS: Tests should be fixed and some created.