PRO-7551 do not show editing UI of subwidgets when live previewing a widget (#4910)

Author: boutell

modules/@apostrophecms/area/ui/apos/apps/AposAreas.js

@@ -20,17 +20,19 @@ export default function() {
     widgetsRendering++;
   });
 
-  apos.bus.$on('widget-rendered', function() {
+  apos.bus.$on('widget-rendered', options => {
     widgetsRendering--;
-    createAreaAppsAndRunPlayersIfDone();
+    createAreaAppsAndRunPlayersIfDone(options);
   });
 
   apos.bus.$on('refreshed', function() {
     createAreaAppsAndRunPlayersIfDone();
   });
 
-  function createAreaAppsAndRunPlayersIfDone() {
-    createAreaApps();
+  function createAreaAppsAndRunPlayersIfDone({ edit = true } = {}) {
+    if (edit) {
+      createAreaApps();
+    }
     if (widgetsRendering === 0) {
       apos.util.runPlayers();
     }

modules/@apostrophecms/widget-type/ui/apos/components/AposWidgetEditor.vue

@@ -338,7 +338,9 @@ function guessOrigin(area) {
   // side of the screen will least obscure the widget
   const rect = area.$el.getBoundingClientRect();
   const cx = (rect.right - rect.left) / 2 + rect.left;
-  if (cx >= (window.innerWidth / 2)) {
+  // Favor the right hand side slightly because rich text
+  // subwidgets in centered areas are more intuitive that way
+  if (cx >= (window.innerWidth * 0.55)) {
     return 'left';
   } else {
     return 'right';

modules/@apostrophecms/widget-type/ui/apos/mixins/AposWidgetMixin.js

@@ -83,7 +83,7 @@ export default {
         // in the DOM before hinting that it might be time to prepare
         // sub-area editors and run players
         setTimeout(function() {
-          apos.bus.$emit('widget-rendered');
+          apos.bus.$emit('widget-rendered', { edit: !aposLivePreview });
         }, 0);
       } catch (e) {
         this.rendered = '<p>Unable to render this widget.</p>';