💥 rename 'tray' action to 'toggle-tray'

francisashley · Dec 16, 2019 · 83eca2d · 83eca2d
1 parent c297a3e
commit 83eca2d
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/src/mde.js b/src/mde.js
@@ -15,7 +15,7 @@ import tracer from "./utils/tracer.js";
 
     defaults(options, {
       stateId: "global",
-      actions: ["reload", "tray"],
+      actions: ["reload", "toggle-tray"],
       hardReload: true,
       actionsCorner: "tr"
     });
@@ -121,15 +121,15 @@ import tracer from "./utils/tracer.js";
         ActionBar({
           corner: state.get("action-bar.corner"),
           showReload: state.get("action-bar").includes("reload"),
-          showTray: state.get("action-bar").includes("tray"),
+          showTray: state.get("action-bar").includes("toggle-tray"),
           shouldRefreshCache: state.get("reload.refreshCache"),
           trayIsOpen: state.getCache("tray.open"),
           onToggleTray: toggleTray
         })
       );
 
       // render tray
-      if (state.get("action-bar").includes("tray")) {
+      if (state.get("action-bar").includes("toggle-tray")) {
         crel(
           root,
           Tray({

diff --git a/src/mde.test.js b/src/mde.test.js
@@ -67,3 +67,15 @@ test("MDE console.assert()'s to tray", () => {
       .innerHTML
   ).toBe("2");
 });
+
+test("MDE shows correct actions", () => {
+  mobileDevEnvironment({
+    actions: ["reload", "toggle-tray"]
+  });
+
+  expect(document.querySelector("#mde-action-bar #mde-reload")).toBeTruthy();
+  expect(document.querySelector("#mde-action-bar #mde-toggle-tray")).toBeTruthy();
+});
+
+test.todo("MDE shows actions tray on top left");
+test.todo("MDE shows actions tray on top right");