Remove element border reset (#6093)

* Remove the border update in handleDrop. Fixes GrapesJS/mjml#338

* Allow custom parser options in component text view `disableEditing`

* Format

src/common/index.ts

@@ -1,4 +1,5 @@
 import Backbone from 'backbone';
+import { HTMLParserOptions } from '../parser/config/config';
 export { default as $ } from '../utils/cash-dom';
 
 interface NOOP {}
@@ -13,6 +14,8 @@ export type DisableOptions = { fromMove?: boolean };
 
 export type LocaleOptions = { locale?: boolean };
 
+export type WithHTMLParserOptions = { parserOptions?: HTMLParserOptions };
+
 export type RemoveOptions = Backbone.Silenceable;
 
 export type EventHandler = Backbone.EventHandler;

src/dom_components/model/Components.ts

@@ -1,6 +1,6 @@
 import { isEmpty, isArray, isString, isFunction, each, includes, extend, flatten, keys } from 'underscore';
 import Component from './Component';
-import { AddOptions, Collection, OptionAsDocument } from '../../common';
+import { AddOptions, Collection } from '../../common';
 import { DomComponentsConfig } from '../config/config';
 import EditorModel from '../../editor/model/Editor';
 import ComponentManager from '..';
@@ -15,7 +15,7 @@ import {
 } from './types';
 import ComponentText from './ComponentText';
 import ComponentWrapper from './ComponentWrapper';
-import { ComponentsEvents } from '../types';
+import { ComponentsEvents, ParseStringOptions } from '../types';
 import { isSymbolInstance, isSymbolRoot, updateSymbolComps } from './SymbolUtils';
 
 export const getComponentIds = (cmp?: Component | Component[] | Components, res: string[] = []) => {
@@ -252,11 +252,11 @@ Component> {
     return new model(attrs, options) as Component;
   }
 
-  parseString(value: string, opt: AddOptions & OptionAsDocument & { temporary?: boolean; keepIds?: string[] } = {}) {
+  parseString(value: string, opt: ParseStringOptions = {}) {
     const { em, domc, parent } = this;
     const asDocument = opt.asDocument && parent?.is('wrapper');
     const cssc = em.Css;
-    const parsed = em.Parser.parseHtml(value, { asDocument });
+    const parsed = em.Parser.parseHtml(value, { asDocument, ...opt.parserOptions });
     let components = parsed.html;
 
     if (asDocument) {

src/dom_components/types.ts

@@ -1,3 +1,4 @@
+import { AddOptions, OptionAsDocument, WithHTMLParserOptions } from '../common';
 import Component from './model/Component';
 
 export enum ActionLabelComponents {
@@ -16,6 +17,10 @@ export interface SymbolInfo {
   relatives: Component[];
 }
 
+export interface ParseStringOptions extends AddOptions, OptionAsDocument, WithHTMLParserOptions {
+  keepIds?: string[];
+}
+
 export enum ComponentsEvents {
   /**
    * @event `component:add` New component added.

src/dom_components/view/ComponentTextView.ts

@@ -1,5 +1,5 @@
 import { bindAll } from 'underscore';
-import { AddOptions, DisableOptions, ObjectAny } from '../../common';
+import { AddOptions, DisableOptions, ObjectAny, WithHTMLParserOptions } from '../../common';
 import RichTextEditorModule from '../../rich_text_editor';
 import RichTextEditor from '../../rich_text_editor/model/RichTextEditor';
 import { off, on } from '../../utils/dom';
@@ -108,7 +108,7 @@ export default class ComponentTextView<TComp extends ComponentText = ComponentTe
    * Disable element content editing
    * @private
    * */
-  async disableEditing(opts: DisableOptions = {}) {
+  async disableEditing(opts: DisableOptions & WithHTMLParserOptions = {}) {
     const { model, rte, activeRte, em } = this;
     // There are rare cases when disableEditing is called when the view is already removed
     // so, we have to check for the model, this will avoid breaking stuff.

src/utils/Droppable.ts

@@ -215,7 +215,6 @@ export default class Droppable {
     const { dragContent } = this;
     const dt = (ev as DragEvent).dataTransfer;
     const content = this.getContentByData(dt).content;
-    (ev.target as HTMLElement).style.border = '';
     content && dragContent && dragContent(content);
     this.endDrop(!content, ev);
   }