[IMP] SavePlugin is now necessary for dragAndDrop

Why is that the case? ¯\_(ツ)_/¯

Author: thjo-odoo

addons/html_builder/static/src/builder.xml

@@ -18,8 +18,8 @@
                     <t t-else="">FS</t>
                     <!-- TODO remove FS -->
                 </button>
-                <button t-if="editor.shared.savePlugin" type="button" t-on-click="discard" class="o-hb-btn btn" data-action="cancel" title="Tip: Esc to preview" accesskey="j">Discard</button>
-                <button t-if="editor.shared.savePlugin" type="button" t-on-click="save" class="o-hb-btn btn btn-success px-3" data-action="save" accesskey="s">Save</button>
+                <button t-if="!props.getExternalScrollableAncestor?.()" type="button" t-on-click="discard" class="o-hb-btn btn" data-action="cancel" title="Tip: Esc to preview" accesskey="j">Discard</button>
+                <button t-if="!props.getExternalScrollableAncestor?.()" type="button" t-on-click="save" class="o-hb-btn btn btn-success px-3" data-action="save" accesskey="s">Save</button>
             </div>
         </div>
         <div class="o-snippets-tabs position-relative grid px-2 my-2" style="--columns: 3; --gap: 0.25rem">

addons/mass_mailing_egg/static/src/builder/mass_mailing_builder.js

@@ -22,7 +22,7 @@ export class MassMailingBuilder extends Component {
         // TODO EGGMAIL: copied from website, check if something needs to be changed here
         const pluginsToRemove = [
             "BuilderFontPlugin", // Makes call to Google API (can't be used for emails)
-            "SavePlugin",
+            // "SavePlugin",
             "SaveSnippetPlugin",
             "AnchorPlugin",
         ];