batch removeElement, fixes overtake bug, but does not solve it

src/browserApp/DirectDomAdapter.ts

@@ -91,6 +91,9 @@ export class DirectDomAdapter implements DocumentObjectModelAdapter {
 
                 case 'innerHTML':
                     return this.setContentToSync(command.elementId, command.value as string)
+
+                case 'removeElement':
+                    return this.removeSync(command.elementId)
 
                 case 'setStyleTo':
                     return this.setStyleToSync(command.elementId, command.value as string|Style)
@@ -306,6 +309,9 @@ export class DirectDomAdapter implements DocumentObjectModelAdapter {
     }
 
     public async remove(id: string): Promise<void> {
+        this.removeSync(id)
+    }
+    public removeSync(id: string): void {
         const element: HTMLElement|null = this.getElement(id)
         if (!element) {
             util.logWarning(`DirectDomAdapter::remove(..) failed to get element with id '${id}'.`)

src/core/RenderManager.ts

@@ -158,7 +158,8 @@ export class RenderManager {
   public remove(id: string, priority: RenderPriority = RenderPriority.NORMAL): Promise<void> {
     return this.runOrSchedule(new Command({
       priority: priority,
-      command: () => dom.remove(id)
+      command: () => dom.remove(id),
+      batchParameters: {elementId: id, method: 'removeElement', value: ''}
     }))
   }
 
@@ -301,7 +302,8 @@ export class RenderManager {
     }))
   }
 
-  public async runOrSchedule<T>(command: Command): Promise<T> { // only public for unit tests
+  /** only public for unit tests */
+  public async runOrSchedule<T>(command: Command): Promise<T> {
     const combinedCommand: Command|undefined = this.tryToCombineWithQueuedCommands(command)
     if (combinedCommand) {
       return combinedCommand.promise.get()
@@ -335,7 +337,8 @@ export class RenderManager {
     return command.promise.get()
   }
 
-  public addCommand(command: Command): void { // only public for unit tests
+  /** only public for unit tests */
+  public addCommand(command: Command): void {
     let i = 0
     for(; i < this.commands.length; i++) {
       const queuedCommand: Command = this.commands[i]

src/core/domAdapter.ts

@@ -25,7 +25,7 @@ export type MouseEventResultAdvanced = {
   targetPathElementIds: string[]
 }
 
-export type BatchMethod = 'appendChildTo'|'addContentTo'|'addElementsTo'|'addElementTo'|'setElementsTo'|'setElementTo'|'innerHTML'|'setStyleTo'|'addStyleTo'|'addClassTo'|'removeClassFrom'
+export type BatchMethod = 'appendChildTo'|'addContentTo'|'addElementsTo'|'addElementTo'|'setElementsTo'|'setElementTo'|'innerHTML'|'removeElement'|'setStyleTo'|'addStyleTo'|'addClassTo'|'removeClassFrom'
 
 export let dom: DocumentObjectModelAdapter
 

src/electronApp/ElectronIpcDomAdapter.ts

@@ -108,6 +108,9 @@ export class ElectronIpcDomAdapter implements DocumentObjectModelAdapter {
 
           case 'innerHTML':
             return `document.getElementById('${command.elementId}').innerHTML='${command.value}';`// TODO: fails if innerHTML includes "'"
+
+          case 'removeElement':
+            return `document.getElementById('${command.elementId}').remove();`
 
           case 'setStyleTo':
             if (typeof command.value === 'string') {