ENH: Dialog for weapon range

src/module/item/entity.js

@@ -16,8 +16,8 @@ export class OseItem extends Item {
   }
 
   static chatListeners(html) {
-    html.on('click', '.card-buttons button', this._onChatCardAction.bind(this));
+    html.on("click", ".card-buttons button", this._onChatCardAction.bind(this));
-    html.on('click', '.item-name', this._onChatCardToggleContent.bind(this));
+    html.on("click", ".item-name", this._onChatCardToggleContent.bind(this));
   }
 
   getChatData(htmlOptions) {
@@ -34,11 +34,7 @@ export class OseItem extends Item {
       props.push(data.qualities);
     }
     if (this.data.type == "spell") {
-      props.push(
+      props.push(`${data.class} ${data.lvl}`, data.range, data.duration);
-        `${data.class} ${data.lvl}`,
-        data.range,
-        data.duration
-      );
     }
     if (data.hasOwnProperty("equipped")) {
       props.push(data.equipped ? "Equipped" : "Not Equipped");
@@ -50,14 +46,53 @@ export class OseItem extends Item {
   }
 
   rollWeapon(skipDialog) {
-    let isNPC = this.actor.data.type != 'character';
+    let isNPC = this.actor.data.type != "character";
-    if (this.data.data.missile && !isNPC) {
+    const data = this.data.data;
-      this.actor.rollAttack({type: 'missile', label: this.name, dmg: this.data.data.damage}, {event: {ctrlKey: skipDialog}});
+    let type = "raw";
-    } else if (this.data.data.melee && !isNPC) {
+    if (data.missile && data.melee && !isNPC) {
-      this.actor.rollAttack({type: 'melee', label: this.name, dmg: this.data.data.damage}, {event: {ctrlKey: skipDialog}});
+      // Dialog
-    } else {
+      new Dialog({
-      this.actor.rollAttack({type: 'raw', label: this.name, dmg: this.data.data.damage}, {event: {ctrlKey: skipDialog}});
+        title: "Choose Attack Range",
+        content: '',
+        buttons: {
+          melee: {
+            icon: '<i class="fas fa-fist-raised"></i>',
+            label: "Melee",
+            callback: () => {
+              this.actor.rollAttack(
+                { type: "melee", label: this.name, dmg: this.data.data.damage },
+                { event: { ctrlKey: skipDialog } }
+              );
+            },
+          },
+          missile: {
+            icon: '<i class="fas fa-bullseye"></i>',
+            label: "Missile",
+            callback: () => {
+              this.actor.rollAttack(
+                {
+                  type: "missile",
+                  label: this.name,
+                  dmg: this.data.data.damage,
+                },
+                { event: { ctrlKey: skipDialog } }
+              );
+            },
+          },
+        },
+        default: "melee",
+      }).render(true);
+      return true;
+    } else if (data.missile && !isNPC) {
+      type = "missile";
+    } else if (data.melee && !isNPC) {
+      type = "melee";
     }
+    this.actor.rollAttack(
+      { type: type, label: this.name, dmg: this.data.data.damage },
+      { event: { ctrlKey: skipDialog } }
+    );
+
     return true;
   }
 
@@ -74,7 +109,7 @@ export class OseItem extends Item {
       ...{
         rollData: {
           type: "Formula",
-          blindroll: this.data.data.blindroll
+          blindroll: this.data.data.blindroll,
         },
       },
     };
@@ -96,7 +131,7 @@ export class OseItem extends Item {
    * @return {Promise}
    */
   async roll({ skipDialog = false } = {}) {
-    if (this.data.type == 'weapon') {
+    if (this.data.type == "weapon") {
       if (this.rollWeapon(skipDialog)) return;
     }
     // Basic template rendering data
@@ -111,7 +146,7 @@ export class OseItem extends Item {
       hasDamage: this.hasDamage,
       isSpell: this.data.type === "spell",
       hasSave: this.hasSave,
-      config: CONFIG.OSE
+      config: CONFIG.OSE,
     };
 
     // Render the chat card template
@@ -157,7 +192,6 @@ export class OseItem extends Item {
     }
   }
 
-
   static async _onChatCardAction(event) {
     event.preventDefault();
 
@@ -180,7 +214,9 @@ export class OseItem extends Item {
     // Get the Item
     const item = actor.getOwnedItem(card.dataset.itemId);
     if (!item) {
-      return ui.notifications.error(`The requested item ${card.dataset.itemId} no longer exists on Actor ${actor.name}`)
+      return ui.notifications.error(
+        `The requested item ${card.dataset.itemId} no longer exists on Actor ${actor.name}`
+      );
     }
 
     // Get card targets
@@ -192,12 +228,13 @@ export class OseItem extends Item {
     // Attack and Damage Rolls
     if (action === "damage") await item.rollDamage({ event });
     else if (action === "formula") await item.rollFormula({ event });
-
     // Saving Throws for card targets
     else if (action == "save") {
       if (!targets.length) {
-        ui.notifications.warn(`You must have one or more controlled Tokens in order to use this option.`);
+        ui.notifications.warn(
-        return button.disabled = false;
+          `You must have one or more controlled Tokens in order to use this option.`
+        );
+        return (button.disabled = false);
       }
       for (let t of targets) {
         await t.rollSave(button.dataset.save, { event });
@@ -209,7 +246,6 @@ export class OseItem extends Item {
   }
 
   static _getChatCardActor(card) {
-
     // Case 1 - a synthetic actor from a Token
     const tokenKey = card.dataset.tokenId;
     if (tokenKey) {
@@ -230,8 +266,11 @@ export class OseItem extends Item {
   static _getChatCardTargets(card) {
     const character = game.user.character;
     const controlled = canvas.tokens.controlled;
-    const targets = controlled.reduce((arr, t) => t.actor ? arr.concat([t.actor]) : arr, []);
+    const targets = controlled.reduce(
-    if ( character && (controlled.length === 0) ) targets.push(character);
+      (arr, t) => (t.actor ? arr.concat([t.actor]) : arr),
+      []
+    );
+    if (character && controlled.length === 0) targets.push(character);
     return targets;
   }
 }