katalyst-navigation 1.3.0 → 1.3.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 368bcecb2264a1a536d2bbeb6c05f2dfa6762dcd693a215db21ef93db4da26f8
-  data.tar.gz: 83ef6762cddd830df2acc4ef1404c8037211c4de4b3dbe5d8b5501bb4cf42fd7
+  metadata.gz: 7355ef157efbf9879accdafa63efa35f02815aa29be531d9e4f73289c130e433
+  data.tar.gz: 32cb8173a0f397d3b36491b35687b4a4848d5e7deed554742e5310d2a33b7a35
 SHA512:
-  metadata.gz: a4e9965e0b7e49c055e405d90dd13dcb764f2b4b1b0fb62191347eb066e784dea5c1ee069b9a848fe283d87da1241fa840663d8152a92fe2b1f73607bae11b9b
-  data.tar.gz: df13a7692f558a2eb5937a4e85d7137540d175ca6be500a743bfbd1caa53fc4fbba1be0acc27cc6bedf665009a8ff58439bc1d2fc35ae76028059ec0ea0614f4
+  metadata.gz: 52ec419f941ed4f1fb101206552c4bfde580a9c9afe8a50b63ca68ea2e569f6356a2feba27230bd9f7a6fec29f7c571703e7faee22007609aab393a0b21c7ba0
+  data.tar.gz: 0dbf4578c6b223b5fc6ab8f506cf4e45a392811ad181272352a64bdae83242367f927c7fa673349247f791b69387d08072c8f55b5609fa19d637880f2bf81c71

data/app/assets/javascripts/utils/navigation/editor/item.js

@@ -42,7 +42,7 @@ export default class Item {
    * @returns {number} logical nesting depth of node in menu
    */
   get depth() {
-    return parseInt(this.node.dataset[`navigationDepth`]);
+    return parseInt(this.node.dataset[`navigationDepth`]) || 0;
   }
 
   get #depthInput() {

data/app/assets/stylesheets/katalyst/navigation/editor/_item-actions.scss

@@ -22,8 +22,7 @@
     }
   }
 
-  [role="img"][value="link"],
-  [role="img"][value="title"] {
+  [role="img"][title="Type"] {
     width: 1.5rem;
     height: 1.5rem;
     display: grid;
@@ -44,18 +43,18 @@
       line-height: 1.125rem;
       text-align: center;
     }
+  }
 
-    &[value="link"] {
-      &::before {
-        content: "#";
-      }
-    }
+  [role="img"][value="heading"]::before {
+    content: "H";
+  }
 
-    &[value="title"] {
-      &::before {
-        content: "T";
-      }
-    }
+  [role="img"][value="link"]::before {
+    content: "#";
+  }
+
+  [role="img"][value="button"]::before {
+    content: "#";
   }
 
   [role="img"][value="invisible"] {

data/app/assets/stylesheets/katalyst/navigation/editor/_new-items.scss

@@ -28,6 +28,10 @@
       position: unset;
     }
 
+    &[data-item-type="heading"]:before {
+      background-image: url("data:image/svg+xml,%3Csvg viewBox='0 0 64 64' xmlns='http://www.w3.org/2000/svg'%3E%3Cstyle%3Etext %7B font: 72px serif %7D%3C/style%3E%3Ctext x='7' y='56' textLength='48'%3EH%3C/text%3E%3C/svg%3E%0A");
+    }
+
     &[data-item-type="link"]:before {
       background-image: url("data:image/svg+xml,%3Csvg viewBox='0 0 48 48' xmlns='http://www.w3.org/2000/svg'%3E%3Cg clip-path='url(%23clip0_67_1273)'%3E%3Cpath d='M16 22H32V26H16V22ZM40.2 24H44C44 18.48 39.52 14 34 14H26V17.8H34C37.42 17.8 40.2 20.58 40.2 24ZM7.8 24C7.8 20.58 10.58 17.8 14 17.8H22V14H14C8.48 14 4 18.48 4 24C4 29.52 8.48 34 14 34H22V30.2H14C10.58 30.2 7.8 27.42 7.8 24ZM38 24H34V30H28V34H34V40H38V34H44V30H38V24Z' /%3E%3C/g%3E%3Cdefs%3E%3CclipPath id='clip0_67_1273'%3E%3Crect width='48' height='48' /%3E%3C/clipPath%3E%3C/defs%3E%3C/svg%3E");
     }

data/app/controllers/katalyst/navigation/items_controller.rb

@@ -10,6 +10,10 @@ module Katalyst
         render locals: { item: @menu.items.build(type: new_item_params) }
       end
 
+      def edit
+        render locals: { item: @item }
+      end
+
       def create
         item = @menu.items.build(item_params)
         if item.save
@@ -19,10 +23,6 @@ module Katalyst
         end
       end
 
-      def edit
-        render locals: { item: @item }
-      end
-
       def update
         @item.attributes = item_params
 

data/app/controllers/katalyst/navigation/menus_controller.rb

@@ -9,10 +9,22 @@ module Katalyst
         render locals: { menus: menus, sort: sort }
       end
 
+      def show
+        menu = Menu.find(params[:id])
+
+        render locals: { menu: menu }
+      end
+
       def new
         render locals: { menu: Menu.new }
       end
 
+      def edit
+        menu = Menu.find(params[:id])
+
+        render locals: { menu: menu }
+      end
+
       def create
         @menu = Menu.new(menu_params)
 
@@ -23,18 +35,6 @@ module Katalyst
         end
       end
 
-      def show
-        menu = Menu.find(params[:id])
-
-        render locals: { menu: menu }
-      end
-
-      def edit
-        menu = Menu.find(params[:id])
-
-        render locals: { menu: menu }
-      end
-
       # PATCH /admins/navigation_menus/:slug
       def update
         menu = Menu.find(params[:id])
@@ -42,7 +42,8 @@ module Katalyst
         menu.attributes = menu_params
 
         unless menu.valid?
-          return render :show, locals: { menu: menu }, status: :unprocessable_entity
+          return render turbo_stream: helpers.navigation_editor_errors(menu: menu),
+                        status:       :unprocessable_entity
         end
 
         case params[:commit]

data/app/helpers/katalyst/navigation/editor/errors.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+module Katalyst
+  module Navigation
+    module Editor
+      class Errors < Base
+        def build(**options)
+          turbo_frame_tag dom_id(menu, :errors) do
+            next unless menu.errors.any?
+
+            tag.div(class: "navigation-errors", **options) do
+              tag.h2("Errors in navigation") +
+                tag.ul(class: "errors") do
+                  menu.errors.each do |error|
+                    concat(tag.li(error.message))
+                  end
+                end
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/app/helpers/katalyst/navigation/editor/menu.rb

@@ -14,6 +14,7 @@ module Katalyst
         def build(options)
           form_with(model: menu, **default_options(id: menu_form_id, **options)) do |form|
             concat hidden_input
+            concat errors
             concat(capture { yield form })
           end
         end
@@ -26,6 +27,10 @@ module Katalyst
           tag.input(type: "hidden", name: "#{Item::ATTRIBUTES_SCOPE}[id]")
         end
 
+        def errors
+          Editor::Errors.new(self, menu).build
+        end
+
         def default_options(options)
           add_option(options, :data, :controller, MENU_CONTROLLER)
           add_option(options, :data, :action, ACTIONS)

data/app/helpers/katalyst/navigation/editor_helper.rb

@@ -25,7 +25,13 @@ module Katalyst
         Editor::List.new(self, menu).items(item)
       end
 
-      # Gene
+      # Generate a turbo stream fragment that will show structural errors to the user.
+      def navigation_editor_errors(menu:, **options)
+        turbo_stream.replace(dom_id(menu, :errors),
+                             Editor::Errors.new(self, menu).build(**options))
+      end
+
+      # Generate a new item template.
       def navigation_editor_new_item(item:, menu: item.menu, **options, &block)
         Editor::NewItem.new(self, menu).build(item, **options, &block)
       end

data/app/models/katalyst/navigation/menu.rb

@@ -69,6 +69,11 @@ module Katalyst
         self
       end
 
+      # Required for testing items validation
+      def items_attributes
+        draft_version&.nodes&.as_json
+      end
+
       # Updates the current draft version with new structure. Attributes should be structural information about the
       # items, e.g. `{index => {id:, depth:}` or `[{id:, depth:}]`.
       #
@@ -105,6 +110,8 @@ module Katalyst
 
         attribute :nodes, Types::NodesType.new, default: -> { [] }
 
+        validate :ensure_items_exists
+
         def items
           # support building menus in memory
           # requires that items are added in order and index and depth are set
@@ -112,12 +119,18 @@ module Katalyst
 
           items = parent.items.where(id: nodes.map(&:id)).index_by(&:id)
           nodes.map do |node|
-            item       = items[node.id]
-            item.index = node.index
-            item.depth = node.depth
-            item
+            items[node.id]&.tap do |item|
+              item.index = node.index
+              item.depth = node.depth
+            end
           end
         end
+
+        private
+
+        def ensure_items_exists
+          parent.errors.add(:items, :missing_item) unless items.all?(&:present?)
+        end
       end
     end
   end

data/app/views/katalyst/navigation/menus/_item.html.erb

@@ -2,7 +2,7 @@
   <div class="tree" data-invisible="<%= !item.visible? %>">
     <%= builder.accordion_actions %>
 
-    <span role="img" value="<%= item.url.present? ? "link" : "title" %>" title="Type"></span>
+    <span role="img" value="<%= item.model_name.param_key %>" title="Type"></span>
     <h4 class="title" title="<%= item.title %>"><%= item.title %></h4>
     <span role="img" value="invisible" title="Hidden"></span>
   </div>

data/config/locales/en.yml

@@ -1,4 +1,8 @@
 en:
+  activerecord:
+    errors:
+      messages:
+        missing_item: Items are missing or invalid
   views:
     katalyst:
       navigation:

data/lib/katalyst/navigation/version.rb

@@ -2,6 +2,6 @@
 
 module Katalyst
   module Navigation
-    VERSION = "1.3.0"
+    VERSION = "1.3.2"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: katalyst-navigation
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.3.2
 platform: ruby
 authors:
 - Katalyst Interactive
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-10-25 00:00:00.000000000 Z
+date: 2022-11-08 00:00:00.000000000 Z
 dependencies: []
 description:
 email:
@@ -39,6 +39,7 @@ files:
 - app/controllers/katalyst/navigation/items_controller.rb
 - app/controllers/katalyst/navigation/menus_controller.rb
 - app/helpers/katalyst/navigation/editor/base.rb
+- app/helpers/katalyst/navigation/editor/errors.rb
 - app/helpers/katalyst/navigation/editor/item.rb
 - app/helpers/katalyst/navigation/editor/list.rb
 - app/helpers/katalyst/navigation/editor/menu.rb