- Issue created by @lauriii
- Assigned to Gauravvv
- Status changed to Postponed
about 2 months ago 8:15am 19 July 2024 - 🇫🇮Finland lauriii Finland
We probably need ✨ Contextual form values need to be integrated with Redux Active before this can be worked on.
- Issue was unassigned.
- 🇫🇮Finland lauriii Finland
An additional consideration for improving performance is to only re-render the parts that were changed. There's a similar improvements in the works for Layout Builder: ✨ Only reload updated part of Layout Builder element Needs work .
- 🇧🇪Belgium Wim Leers Ghent 🇧🇪🇪🇺
+1, that seemed an evident implied requirement. 😇
- 🇬🇧United Kingdom catch
One thing about #5/6, the layout builder issue is trying to scale down from the entire layout to a block.
However theoretically we could scale down to an individual prop (i.e. one field value, one static prop etc.) within a component. Would require being able to identify exactly what has changed vs not (possibly down to field columns) and also being able to replace the HTML at that level of granularity.
- 🇧🇪Belgium Wim Leers Ghent 🇧🇪🇪🇺
#7:
However theoretically we could scale down to an individual prop (i.e. one field value, one static prop etc.) within a component.
Yep.
Would require being able to identify exactly what has changed vs not (possibly down to field columns) and also being able to replace the HTML at that level of granularity.
Yep.
But if it worked, it would be less for the browser to re-render etc.
Yep.
Happy to say that we're way ahead of you on this one: 🌱 [META] Real-time preview: supporting back-end infrastructure Needs work 😄
- 🇫🇮Finland lauriii Finland
This will be at least partially solved by ✨ Contextual form values need to be integrated with Redux Active based on @bnjmnm.
- Assigned to lauriii
- 🇧🇪Belgium Wim Leers Ghent 🇧🇪🇪🇺
Actually, having seen ✨ Contextual form values need to be integrated with Redux Active in action, I think this is already implemented there? 😅
@lauriii, do you want to rescope this issue to handle #5 specifically? Because right now, the entire preview is updated — i.e. the entire component tree is re-rendered.
So I think rescoping this to would make sense.
(The even more efficient thing @catch is talking about in #7 requires the AST work in 🌱 [META] Real-time preview: supporting back-end infrastructure Needs work to be done, what I'm proposing here would be an easily achievable intermediary step.)
- Issue was unassigned.
- 🇧🇪Belgium Wim Leers Ghent 🇧🇪🇪🇺
Issue rescoped 👍
Crafted a fairly detailed proposed resolution.
- Status changed to Active
about 1 month ago 2:59pm 26 July 2024 - 🇧🇪Belgium Wim Leers Ghent 🇧🇪🇪🇺
✨ Contextual form values need to be integrated with Redux Active is in! 🥳