-
Notifications
You must be signed in to change notification settings - Fork 135
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
[Merl 1149] Add TextControl field #1560
Conversation
🦋 Changeset detectedLatest commit: e03a555 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
📦 Next.js Bundle Analysis for @faustwp/getting-started-exampleThis analysis was generated by the Next.js Bundle Analysis action. 🤖
|
Page | Size (compressed) |
---|---|
global |
242.65 KB (🟡 +25 B) |
Details
The global bundle is the javascript bundle that loads alongside every page. It is in its own category because its impact is much higher - an increase to its size means that every page on your website loads slower, and a decrease means every page loads faster.
Any third party scripts you have added directly to your app using the <script>
tag are not accounted for in this analysis
If you want further insight into what is behind the changes, give @next/bundle-analyzer a try!
Thank you for your work on this, @theodesp! I had some questions on expected behavior to clarify with you on this item, as well as a related question about the associated RFC and some assumptions I may be making. Below, I've tested out the block in my example, following the instructions above. I'm noting the text control field, which looks as expected based on the WordPress TextControl component. 👍 My concern now is whether the TextControl field is behaving as expected when clicking away from the block in the editor, and when examining the block contents on the front-end post. Screen.Recording.2023-09-05.at.3.57.48.PM.movWhen clicking away from the block in the editor, I seem to just see the attribute message default from the Is this block working as expected? If not, I have a follow-up question on testing. In my
I have it here as type: "string" because in the RFC here, this seems to resolve to the correct type for the Here is my testing plugin's
|
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.
Thank you for your thorough walkthrough of expected behavior and my questions on the RFC, @theodesp! This LGTM! 😊 ✨
Tasks
Description
registerFaustBlock
Edit function now handlestext
control fields.Related Issue(s):
#1522
Testing
Follow testing instruction from this PR:
#1559
type: text
.Screenshots
Documentation Changes
Dependant PRs