- Issue created by @jessebaker
- 🇮🇳India nayana_mvr
Hi @jessebaker, I tried to reproduce this issue in Drupal cms, but I'm getting some error. Steps followed:-
- Installed Drupal CMS (starshot) and Experience Builder module and enabled it.
- Created a new field in Basic page content type of field type Experience Builder.
- In the manage dispaly, the new field was in disabled section.
- Dragged the field to enabled section and tried to save configuration.
- Got the following error.
Please let me know if I'm missing some steps here. Also please update the Issue Summary with steps to reproduce.
- Merge request !327Issue #3475484: When the height of the preview changes, it moves up and down in the viewport → (Merged) created by Unnamed author
- 🇬🇧United Kingdom jessebaker
This MR !327 doesn't address the problem but instead removes the functionality that causes the problem. At first this seemed to be the wrong solution but actually gave me an idea!
In ✨ [later phase] Total canvas size should be dynamic based on browser viewport size Needs review functionality was added to make the canvas height dynamic based on the content. However in this MR some of that functionality was removed - and yet a small padding was still present because of the CSS. Surely I could just add additional padding in CSS!
Given that the "dynamic" height was just adding extra height/width equal to 80% of the viewport's height/width it occured to me that I could use vh and vw units in CSS to achieve the same thing and remove the JS (and the resize observer!) that was making that calculation.
-
balintbrews →
committed 958d5000 on 0.x authored by
gauravvvv →
Issue #3475484 by jessebaker, gauravvvv, balintbrews: When the height of...
-
balintbrews →
committed 958d5000 on 0.x authored by
gauravvvv →
Automatically closed - issue fixed for 2 weeks with no activity.