From 96c1cb023e533d9e7294af6b8e53a4f49685bea8 Mon Sep 17 00:00:00 2001
From: U~man <ewolfwood@yandex.com>
Date: Sat, 11 Jul 2020 17:04:31 +0200
Subject: [PATCH] ENH: Delete lang

---
 src/module/actor/character-sheet.js           | 43 +++++++++++++------
 src/scss/character.scss                       |  7 +++
 .../actors/partials/character-notes-tab.html  | 10 ++++-
 3 files changed, 46 insertions(+), 14 deletions(-)

diff --git a/src/module/actor/character-sheet.js b/src/module/actor/character-sheet.js
index 3de618c..fffd902 100644
--- a/src/module/actor/character-sheet.js
+++ b/src/module/actor/character-sheet.js
@@ -108,11 +108,11 @@ export class OseActorSheetCharacter extends OseActorSheet {
     if (data.config.encumbrance == "detailed") {
       if (weight > data.encumbrance.max) {
         data.data.movement.base = 0;
-      } else if (weight > (800 + delta)) {
+      } else if (weight > 800 + delta) {
         data.data.movement.base = 30;
-      } else if (weight > (600 + delta)) {
+      } else if (weight > 600 + delta) {
         data.data.movement.base = 60;
-      } else if (weight > (400 + delta)) {
+      } else if (weight > 400 + delta) {
         data.data.movement.base = 90;
       } else {
         data.data.movement.base = 120;
@@ -145,10 +145,9 @@ export class OseActorSheetCharacter extends OseActorSheet {
     }
   }
 
-  
   async _chooseLang() {
     let choices = CONFIG.OSE.languages;
-    
+
     let templateData = { choices: choices },
       dlg = await renderTemplate(
         "/systems/ose/templates/actors/dialogs/lang-create.html",
@@ -165,7 +164,7 @@ export class OseActorSheetCharacter extends OseActorSheet {
             icon: '<i class="fas fa-check"></i>',
             callback: (html) => {
               resolve({
-                choice: html.find('select[name="choice"]').val()
+                choice: html.find('select[name="choice"]').val(),
               });
             },
           },
@@ -182,22 +181,33 @@ export class OseActorSheetCharacter extends OseActorSheet {
   _pushLang(table) {
     const data = this.actor.data.data;
     let update = duplicate(data[table]);
-    this._chooseLang().then(dialogInput => {
+    this._chooseLang().then((dialogInput) => {
       const name = CONFIG.OSE.languages[dialogInput.choice];
       console.log(name);
       if (update.value) {
         update.value.push(name);
       } else {
-        update = {value: [name]};
+        update = { value: [name] };
       }
       let newData = {};
       newData[table] = update;
-      return this.actor.update({data: newData}).then(() => {
+      return this.actor.update({ data: newData }).then(() => {
         this.render(true);
       });
     });
   }
 
+  _popLang(table, lang) {
+    const data = this.actor.data.data;
+    let update = data[table].value.filter((el) => el != lang);
+    console.log(update);
+    let newData = {};
+    newData[table] = {value: update};
+    return this.actor.update({ data: newData }).then(() => {
+      this.render(true);
+    });
+  }
+
   /* -------------------------------------------- */
 
   async _onQtChange(event) {
@@ -236,8 +246,7 @@ export class OseActorSheetCharacter extends OseActorSheet {
       this.actor.deleteOwnedItem(li.data("itemId"));
       li.slideUp(200, () => this.render(false));
     });
-    
-    // Delete Inventory Item
+
     html.find(".item-push").click((ev) => {
       event.preventDefault();
       const header = event.currentTarget;
@@ -245,6 +254,16 @@ export class OseActorSheetCharacter extends OseActorSheet {
       this._pushLang(table);
     });
 
+    html.find(".item-pop").click((ev) => {
+      event.preventDefault();
+      const header = event.currentTarget;
+      const table = header.dataset.array;
+      this._popLang(
+        table,
+        $(event.currentTarget).closest(".item").data("lang")
+      );
+    });
+
     html.find(".item-create").click((event) => {
       event.preventDefault();
       const header = event.currentTarget;
@@ -317,7 +336,7 @@ export class OseActorSheetCharacter extends OseActorSheet {
       }
     });
 
-    html.find("button[data-action='modifiers']").click(ev => {
+    html.find("button[data-action='modifiers']").click((ev) => {
       this._onShowModifiers(ev);
     });
 
diff --git a/src/scss/character.scss b/src/scss/character.scss
index 24ac905..9d6831d 100644
--- a/src/scss/character.scss
+++ b/src/scss/character.scss
@@ -42,6 +42,13 @@
             padding: 5px;
             li {
               margin: 0;
+              &:hover .item-controls {
+                display: flex;
+              }
+              .item-controls {
+                flex: 0 0 20px;
+                display:none;
+              }
             }
           }
         }
diff --git a/src/templates/actors/partials/character-notes-tab.html b/src/templates/actors/partials/character-notes-tab.html
index 84549f8..3d99fe2 100644
--- a/src/templates/actors/partials/character-notes-tab.html
+++ b/src/templates/actors/partials/character-notes-tab.html
@@ -13,8 +13,14 @@
                 </div>
                 <ol>
                     {{#each data.languages.value as |lang|}}
-                    <li>
-                        {{lang}}
+                    <li class="item flexrow" data-lang="{{lang}}">
+                        <div class="item-name">
+                            {{lang}}
+                        </div>
+                        <div class="item-controls">
+                            <a class="item-control item-pop" data-array="languages" title="{{localize 'OSE.Del'}}"><i
+                                class="fa fa-trash"></i></a>
+                        </div>
                     </li>
                     {{/each}}
                 </ol>