Merge pull request #156 from JulieWinchester/fix-remove-label-sheet-header

Fix setting removeLabelSheetHeader and header sheet icon toggling

Author: megahead11

src/lang/en.json

@@ -73,8 +73,8 @@
         "autoHideBottom": "Auto Hide Bottom",
         "autoHideBottomHint": "Hide bottom UI(player&hotbar) when actor shows on stage.",
         "suppressMacroHotbar": "Show Player List and Macro Hotbar When Stage is Suppressed",
-        "removeLabelSheetHeader": "Remove label from the header character sheet",
-        "removeLabelSheetHeaderHint": "Remove label from the header character sheet, Useful for little screen and mobile",
+        "removeLabelSheetHeader": "Remove Text Labels From Actor Sheet Header",
+        "removeLabelSheetHeaderHint": "Removes text labels from actor sheet header buttons, useful for small screens and mobile devices. Button will still be displayed with icon.",
         "ignoreMessagesToChat": "Ignore Theatre Messages From Chat",
         "ignoreMessagesToChatHint": "Ignores messages sent during Theatre Mode from chat. Disabling this message will make them appear again.",
         "quoteType": "Quote Type",

src/scripts/Theatre.js

@@ -8402,8 +8402,8 @@ export class Theatre {
       newText = removeLabel;
     }
     ev.currentTarget.innerHTML = Theatre.isActorStaged(actor)
-      ? `<i class="fas fa-theater-masks"></i>${newText}`
-      : `<i class="fas fa-mask"></i>${newText}`;
+      ? `<i class="fas fa-mask"></i>${newText}`
+      : `<i class="fas fa-theater-masks"></i>${newText}`;
   }
 
   static _getTheatreId(actor) {

src/scripts/theatre_main.js

@@ -23,24 +23,27 @@ Handlebars.registerHelper("resprop", function (propPath, hash) {
  */
 Hooks.on("getActorSheetHeaderButtons", (app, buttons) => {
   if (!game.user.isGM && game.settings.get("theatre", "gmOnly")) return;
+  const removeLabelSheetHeader = game.settings.get(Theatre.SETTINGS, "removeLabelSheetHeader");
 
   let theatreButtons = [];
   if (app.object.isOwner) {
     // only prototype actors
     if (!app.object.token) {
       theatreButtons.push({
-        label: "Theatre.UI.Config.Theatre",
+        label: removeLabelSheetHeader ? "" : "Theatre.UI.Config.Theatre",
         class: "configure-theatre",
         icon: "fas fa-user-edit",
         onclick: (ev) => Theatre.onConfigureInsert(ev, app.object.sheet),
       });
     }
     theatreButtons.push({
-      label: Theatre.isActorStaged(app.object) ? "Theatre.UI.Config.RemoveFromStage" : "Theatre.UI.Config.AddToStage",
+      label: removeLabelSheetHeader
+        ? ""
+        : (Theatre.isActorStaged(app.object) ? "Theatre.UI.Config.RemoveFromStage" : "Theatre.UI.Config.AddToStage"),
       class: "add-to-theatre-navbar",
-      icon: "fas fa-theater-masks",
+      icon: Theatre.isActorStaged(app.object) ? "fas fa-mask" : "fas fa-theater-masks",
       onclick: (ev) => {
-        Theatre.onAddToNavBar(ev, app.object.sheet);
+        Theatre.onAddToNavBar(ev, app.object.sheet, removeLabelSheetHeader);
       },
     });
   }