From 2f1a601b58dc6b2571da583987b5db6d38d1e2c4 Mon Sep 17 00:00:00 2001
From: Pascal Wilbrink <pascal.wilbrink@alliander.com>
Date: Mon, 28 Nov 2022 08:50:58 +0100
Subject: [PATCH] Fixed plain compare list

Signed-off-by: Pascal Wilbrink <pascal.wilbrink@alliander.com>
---
 src/plain-compare-list.ts                     | 29 ++++++++++--------
 .../plain-compare-list.test.snap.js           | 30 ++-----------------
 2 files changed, 20 insertions(+), 39 deletions(-)

diff --git a/src/plain-compare-list.ts b/src/plain-compare-list.ts
index 78ffb4e71c..750a5f2b84 100644
--- a/src/plain-compare-list.ts
+++ b/src/plain-compare-list.ts
@@ -60,10 +60,7 @@ export class PlainCompareList extends LitElement {
 
   render(): TemplateResult {
     return html`
-      <div class="container">
-        <div class="flex"></div>
-        ${this.renderFilterCheckbox()}
-      </div>
+      ${this.renderFilterCheckbox()}
       <div class="container container--alt">
         <div class="list__container list__container--left">
           <h3 class="mdc-dialog__title">${this.leftHandTitle}</h3>
@@ -83,8 +80,8 @@ export class PlainCompareList extends LitElement {
       ${this.leftHandObject && this.rightHandObject
         ? html`
             ${renderDiff(
-              this.leftHandObject,
               this.rightHandObject,
+              this.leftHandObject,
               this.filterMutables ? this.filterToIgnore : {}
             )}
           `
@@ -93,13 +90,21 @@ export class PlainCompareList extends LitElement {
   }
 
   protected renderFilterCheckbox(): TemplateResult {
-    return html`<mwc-formfield label="${translate('compare.filterMutables')}">
-      <mwc-checkbox
-        ?checked=${this.filterMutables}
-        @change=${() => (this.filterMutables = !this.filterMutables)}
-      >
-      </mwc-checkbox>
-    </mwc-formfield> `;
+    if (this.filterToIgnore) {
+      return html`
+        <div class="container">
+          <div class="flex"></div>
+          <mwc-formfield label="${translate('compare.filterMutables')}">
+            <mwc-checkbox
+              ?checked=${this.filterMutables}
+              @change=${() => (this.filterMutables = !this.filterMutables)}
+            >
+            </mwc-checkbox>
+          </mwc-formfield>
+        </div>
+      `;
+    }
+    return html``;
   }
 
   static styles = css`
diff --git a/test/unit/__snapshots__/plain-compare-list.test.snap.js b/test/unit/__snapshots__/plain-compare-list.test.snap.js
index afcc997b0c..7445726ad3 100644
--- a/test/unit/__snapshots__/plain-compare-list.test.snap.js
+++ b/test/unit/__snapshots__/plain-compare-list.test.snap.js
@@ -2,15 +2,7 @@
 export const snapshots = {};
 
 snapshots["Plain Compare List Empty list looks like the latest snapshot"] = 
-`<div class="container">
-  <div class="flex">
-  </div>
-  <mwc-formfield label="[compare.filterMutables]">
-    <mwc-checkbox checked="">
-    </mwc-checkbox>
-  </mwc-formfield>
-</div>
-<div class="container container--alt">
+`<div class="container container--alt">
   <div class="list__container list__container--left">
     <h3 class="mdc-dialog__title">
     </h3>
@@ -24,15 +16,7 @@ snapshots["Plain Compare List Empty list looks like the latest snapshot"] =
 /* end snapshot Plain Compare List Empty list looks like the latest snapshot */
 
 snapshots["Plain Compare List Basic List looks like the latest snapshot"] = 
-`<div class="container">
-  <div class="flex">
-  </div>
-  <mwc-formfield label="[compare.filterMutables]">
-    <mwc-checkbox checked="">
-    </mwc-checkbox>
-  </mwc-formfield>
-</div>
-<div class="container container--alt">
+`<div class="container container--alt">
   <div class="list__container list__container--left">
     <h3 class="mdc-dialog__title">
       Project doc
@@ -48,15 +32,7 @@ snapshots["Plain Compare List Basic List looks like the latest snapshot"] =
 /* end snapshot Plain Compare List Basic List looks like the latest snapshot */
 
 snapshots["Plain Compare List Basic List Has a subtitlte, so looks like the latest snapshot"] = 
-`<div class="container">
-  <div class="flex">
-  </div>
-  <mwc-formfield label="[compare.filterMutables]">
-    <mwc-checkbox checked="">
-    </mwc-checkbox>
-  </mwc-formfield>
-</div>
-<div class="container container--alt">
+`<div class="container container--alt">
   <div class="list__container list__container--left">
     <h3 class="mdc-dialog__title">
       Project doc