card 1.98.3 → 1.99.0

data/mod/pointer/spec/set/abstract/pointer/options_api_spec.rb

@@ -1,6 +1,7 @@
 RSpec.describe Card::Set::Abstract::Pointer::OptionsApi do
   def define_options_rule type_id, content
-    Card.create! name: %i[pointer type options], type_id: type_id, content: content
+    Card.create! name: %i[pointer type content_options],
+                 type_id: type_id, content: content
   end
 
   let :sample_pointer do

data/mod/pointer/spec/set/self/input_options_spec.rb

@@ -1,6 +1,6 @@
 describe Card::Set::Self::InputOptions do
   it "loads the self set" do
-    expect(Card[:input, :right, :options].item_names).to contain_exactly(
+    expect(Card[:input_type, :right, :content_options].item_names).to contain_exactly(
       "radio", "checkbox", "select", "multiselect", "list", "ace editor", "filtered list",
       "prosemirror editor", "tinymce editor", "text area", "text field", "calendar"
     )

data/mod/rules/set/right/self.rb

@@ -0,0 +1,3 @@
+def prototype_default_card
+  left
+end

data/mod/rules/set/rule/bar_view.rb

@@ -0,0 +1,88 @@
+format :html do
+  bar_cols 6, 6
+  info_bar_cols 5, 4, 3
+
+  def existing_rule_card
+    @existing_rule_card ||= find_existing_rule_card
+  end
+
+  view :bar, unknown: true do
+    voo.hide :bar_nav unless existing_rule_card
+    super()
+  end
+
+  view :expanded_bar, unknown: true do
+    super()
+  end
+
+  view :one_line_content,
+       wrap: { div: { class: "text-muted one-line" } }, unknown: true do
+    return render_mini_unknown unless existing_rule_card
+
+    with_nest_mode :compact do
+      one_line_content
+    end
+  end
+
+  view :raw_one_line_content,
+       wrap: { div: { class: "text-muted one-line" } }, unknown: true do
+    return render_mini_unknown unless existing_rule_card
+
+    raw_one_line_content
+  end
+
+  view :bar_bottom, unknown: true do
+    if nest_mode == :edit
+      current_rule_form
+    else
+      nest existing_rule_card, view: :core
+    end
+  end
+
+  view :bar_middle, unknown: true do
+    rule_info
+  end
+
+  view :bar_left, unknown: true do
+    super()
+  end
+
+  view :bar_right, unknown: true do
+    voo.show?(:bar_bottom) ? rule_info : rule_short_content
+  end
+
+  def rule_short_content
+    return "" unless existing_rule_card
+
+    nest existing_rule_card, { view: :one_line_content },
+         set_context: card.name.trunk_name
+  end
+
+  def bar_title
+    return super() if voo.show? :full_name
+
+    if existing_rule_card && voo.show?(:toggle)
+      link_to_view bar_title_toggle_view, card.rule_setting_title
+    else
+      card.rule_setting_title
+    end
+  end
+
+  # LOCALIZE
+  def rule_info
+    return wrap_with(:em, "no existing #{setting_link} rule") unless existing_rule_card
+
+    wrap_with :span,
+              "#{rule_setting_link} rule that applies to "\
+              "#{rule_set_link existing_rule_card}"
+  end
+
+  def rule_setting_link
+    link_to_card card.rule_setting, card.rule_setting_name
+  end
+
+  def rule_set_link existing_rule
+    count = link_to_card [card.rule_set, :by_name], card.rule_set.count
+    "#{link_to_card card.rule_set, existing_rule.trunk&.label&.downcase} (#{count})"
+  end
+end

data/mod/rules/set/rule/bridge_rules_editor.rb

@@ -14,20 +14,15 @@ format :html do
     edit_rule_title
   end
 
-  view :overlay_rule_help, unknown: true, perms: :none, cache: :never do
-    # wrap_with :div, class: "help-text rule-instruction d-flex justify-content-between"
-    # do
-
-    # output [wrap_with(:div, rule_based_help), setting_link]
-    # end
-    popover_link([rule_based_help, setting_link].join(" "))
+  view :help_text, unknown: true, cache: :never do
+    wrap_help_text [rule_based_help, setting_link].join(" ")
   end
 
   def setting_link
     wrap_with :div, class: "ml-auto" do
-      # FIXME: hardcoded count
-      link_to_card card.rule_setting_name, " (37 #{card.rule_setting_title} rules)",
-                   class: "text-muted", target: "wagn_setting"
+      link_to_card card.rule_setting_name,
+                  " (#{card.rule_setting.count} #{card.rule_setting_title} rules)",
+                   class: "text-muted"
     end
   end
 end

data/mod/rules/set/rule/editor.rb

@@ -1,19 +1,11 @@
 format :html do
   attr_accessor :rule_context
 
-  def current_rule force_reload=true
-    @current_rule = nil if force_reload
-    @current_rule ||= begin
-      rule = determine_current_rule
-      reload_rule rule
-    end
-  end
-
-  def determine_current_rule
-    existing = find_existing_rule_card
-    return existing if existing
-
-    Card.new name: "#{Card[:all].name}+#{card.rule_user_setting_name}"
+  view :rule_edit, cache: :never, unknown: true,
+                   wrap: { modal: { size: :large,
+                                    title: :edit_rule_title,
+                                    footer: "" } } do
+    current_rule_form form_type: :modal
   end
 
   view :rule_help, unknown: true, perms: :none, cache: :never do
@@ -22,62 +14,6 @@ format :html do
     end
   end
 
-  view :show_rule, cache: :never, unknown: true do
-    return "No Current Rule" if card.new_card?
-
-    voo.items[:view] ||= :link
-    output [
-      show_rule_set(card.rule_set),
-      _render_core
-    ]
-  end
-
-  def show_rule_set set
-    wrap_with :div, class: "rule-set" do
-      %(<label>Applies to</label> #{link_to_card set.name, set.label}:)
-    end
-  end
-
-  view :quick_edit, unknown: true, template: :haml, wrap: :slot do
-    setting_title + short_help_text + quick_editor
-  end
-
-  def quick_form
-    card_form :update,
-              "data-slot-selector": ".set-info.card-slot",
-              success: { view: :quick_edit_success }  do
-      quick_editor
-    end
-  end
-
-  def set_info notify_change=nil
-    wrap true, class: "set-info" do
-      haml :set_info, notify_change: notify_change
-    end
-  end
-
-  def undo_button
-    link_to "undo", method: :post, rel: "nofollow", class: "btn btn-secondary ml-2 btn-sm btn-reduced-padding slotter",
-                    remote: true,
-                    "data-slot-selector": ".card-slot.quick_edit-view",
-                    path: { action: :update,
-                            revert_actions: [card.last_action_id],
-                            revert_to: :previous }
-  end
-
-  view :quick_edit_success do
-    set_info true
-  end
-
-  def quick_editor
-    if card.right.codename == :default
-      @edit_rule_success = {}
-      rules_type_formgroup
-    else
-      rule_content_formgroup
-    end
-  end
-
   view :rule_bridge_link, unknown: true do
     opts = bridge_link_opts(class: "edit-rule-link nav-link",
                             "data-toggle": "pill",
@@ -86,57 +22,37 @@ format :html do
     link_to_view(:overlay_rule, (setting_title + short_help_text), opts)
   end
 
-  def setting_title
-    card.name.tag.tr "*", ""
-  end
-
-  def short_help_text
-    "<div class=\"help-text\">#{card.short_help_text}</div>"
-  end
-
-  def rule_content_container
-    wrap_with :div, class: "rule-content-container" do
-      wrap_with(:span, class: "closed-content content") { yield }
-    end
+  def edit_link_view
+    :rule_edit
   end
 
-  def link_to_open_rule
-    setting_title = card.name.tag.tr "*", ""
-    link_to_view :open_rule, setting_title, class: "edit-rule-link"
-  end
-
-  def closed_rule_content rule_card
-    return "" unless rule_card
-
-    nest rule_card, { view: :closed_content }, set_context: card.name.trunk_name
+  def edit_rule_title
+    output [
+      wrap_with(:h5, setting_title, class: "title font-weight-bold")
+      # render_overlay_rule_help
+    ]
   end
 
-  def open_rule_setting_links
-    wrap_with :div, class: "rule-setting" do
-      [link_to_closed_rule, link_to_all_rules]
+  def current_rule
+    if params[:assign]
+      card
+    elsif (existing = find_existing_rule_card)
+      existing
+    else
+      card
     end
   end
 
-  def link_to_all_rules
-    link_to_card card.rule_setting_name, "all #{card.rule_setting_title} rules",
-                 class: "setting-link", target: "wagn_setting"
+  def quick_editor
+    rule_content_formgroup
   end
 
-  def link_to_closed_rule
-    link_to_view :closed_rule, card.rule_setting_title,
-                 class: "close-rule-link"
+  def setting_title
+    card.name.tag.tr "*", ""
   end
 
-  def reload_rule rule
-    return rule unless (card_args = params[:card])
-
-    if card_args[:name] && card_args[:name].to_name.key != rule.key
-      Card.new card_args
-    else
-      rule = rule.refresh
-      rule.assign_attributes card_args
-      rule.include_set_modules
-    end
+  def short_help_text
+    "<div class=\"help-text\">#{card.short_help_text}</div>"
   end
 
   def rule_set_description
@@ -149,7 +65,7 @@ format :html do
     success = @edit_rule_success
     wrap_type_formgroup do
       type_field(
-        href: path(mark: success[:id], view: :rule_form, type_reload: true), # view: success[:view]
+        href: path(mark: success[:id], view: :rule_form, assign: true),
         class: "type-field rule-type-field live-type-field",
         "data-remote" => true
       )
@@ -157,15 +73,26 @@ format :html do
   end
 
   def rule_content_formgroup
-    formgroup "content", editor: "content", help: false do
-      content_field true
-    end
+    _render_content_formgroup hide: :conflict_tracker
   end
 
   def current_set_key
     card.new_card? ? Card.quick_fetch(:all).name.key : card.rule_set_key
   end
 
+  def left_type_for_nest_editor_set_selection
+    return super unless card.is_template?
+
+    case Card.fetch_id(card.rule_set_pattern_name) # set_pattern_id
+    when TypeID
+      card.rule_set.anchor_name
+    when SelfID
+      card.rule_set.anchor.type_name
+    else
+      super
+    end
+  end
+
   private
 
   def find_existing_rule_card

data/mod/rules/set/rule/html_views.rb

@@ -0,0 +1,13 @@
+format :html do
+  view :core do
+    # Rule cards that are searches are usual right structures and refer to the left
+    # in the search query. In that case the search doesn't work
+    # properly in the context of the rule card itself.  Hence we show the query syntax
+    # and not the search result.
+    if card.type_id == Card::SearchTypeID
+      render_raw
+    else
+      super()
+    end
+  end
+end

data/mod/rules/set/rule/quick_editor.rb

@@ -0,0 +1,39 @@
+format :html do
+  view :quick_edit, unknown: true, wrap: :slot do
+    quick_edit
+  end
+
+  view :quick_edit_success do
+    set_info true
+  end
+
+  def quick_edit
+    haml :quick_edit
+  end
+
+  def quick_form
+    card_form :update, quick_form_opts do
+      quick_editor
+    end
+  end
+
+  def quick_form_opts
+    { "data-slot-selector": ".set-info.card-slot",
+      success: { view: :quick_edit_success } }
+  end
+
+  def set_info notify_change=nil
+    wrap true, class: "set-info" do
+      haml :set_info, notify_change: notify_change
+    end
+  end
+
+  def undo_button
+    link_to "undo", method: :post, rel: "nofollow", class: "btn btn-secondary ml-2 btn-sm btn-reduced-padding slotter",
+                    remote: true,
+                    "data-slot-selector": ".card-slot.quick_edit-view",
+                    path: { action: :update,
+                            revert_actions: [card.last_action_id],
+                            revert_to: :previous }
+  end
+end

data/mod/rules/set/rule/{editor → quick_editor}/set_info.haml

@@ -1,6 +1,6 @@
 .form-group.form-control-feedback
   %label
-    set
+    Set
   %p.text-muted
     = find_existing_rule_card&.left&.label || "--"
   - if notify_change

data/mod/rules/set/rule/rule_form/buttons.rb

@@ -19,7 +19,9 @@ format :html do
 
     delete_opts = {
       confirm: delete_confirm(args[:fallback_set]),
-      success: @edit_rule_success,
+      # success: @edit_rule_success,
+      no_success: true,
+      "data-slotter-mode": "silent-success",
       class: "_close-#{form_type}-on-success"
     }
     delete_opts["data-slot-selector"] = slot_selector if args[:slot_selector]

data/mod/rules/set/rule/rule_form/form_elements.rb

@@ -10,7 +10,7 @@ format :html do
   def rule_set_formgroup
     tag = @rule_context.rule_user_setting_name
     narrower = []
-    option_list "set" do
+    option_list "Set" do
       rule_set_options.map do |set_name, state|
         rule_set_radio_button set_name, tag, state, narrower
       end
@@ -33,7 +33,7 @@ format :html do
   end
 
   def option_list title
-    formgroup title, editor: "set", class: "col-xs-6", help: false do
+    formgroup title, input: "set", class: "col-xs-6", help: false do
       wrap_with :ul do
         wrap_each_with(:li, class: "radio") { yield }
       end

data/mod/rules/set/rule/rule_form/rule_set_radio.rb

@@ -18,7 +18,7 @@ class RuleSetRadio
 
     rule_radio do
       radio_text = "#{@set_name}+#{@tag}"
-      radio_button :name, radio_text, checked: checked?, warning: warning
+      radio_button :name, radio_text, checked: false, warning: warning
     end
   end
 
@@ -51,14 +51,11 @@ class RuleSetRadio
     when :current
       "(current)"
     when :overwritten, :exists
-      link_to_card "#{@set_name}+#{@card.rule_user_setting_name}", "(#{@state})"
+      link_to_card "#{@set_name}+#{@card.rule_user_setting_name}", "(#{@state})",
+                   target: "_blank"
     end
   end
 
-  def checked?
-    [@set_name, true].include? @format.selected_rule_set
-  end
-
   def warning
     if @set_name == "*all"
       "This rule will affect all cards! Are you sure?"