bullet_train-themes 1.6.38 → 1.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0d036819f37f9789169076a640dea2a48aa359d89b9da4761b21c88d8821bd23
-  data.tar.gz: ebf6e916af7b1d33c4677257ca8c8952960266ef08bfc9e13ccf644d9c5f567d
+  metadata.gz: b6e56d92ce9cb17c2287a03b49e00355b1428c2f6d7224e04e0cf9b4683cf709
+  data.tar.gz: ca2781cdf07fc92979dd3414359fdfbe3c7ece80e4291da52c2d48bf8668ea60
 SHA512:
-  metadata.gz: fb947112389d96417169181f642ab1d1753d8bffb29a87b0e5a24d4474efba1bb13bb10dc700996bbd33745ed305d3c560d606a114e9fece9cb976579ae4ab0e
-  data.tar.gz: f35ac27fec13323de44b02cf83d82c07bf6e873b5b830e404d9311adb49be736c75a2f33b7e92f39d8efc6bfd96b4b37b978230b17ea824729ca3555fc89c9e0
+  metadata.gz: 4e062ce511f5edaa75e42d1bff78996c2646b4c5b384aa7e3b4a91cfbf04213f2b474d9a0162b02e91c0bfe6b1fbd670c82225982e2f0a77cd2d52b6927ea6f8
+  data.tar.gz: 890d0f162b865abb20bef7e8f5baf0593ebbb8159fcceedc36b976bf3ead31da12c762c94f27a6e6b054def1308c6621ccafa92cd39aca7f8fb08c4932c7a14e

data/app/views/themes/base/fields/_ckeditor.html.erb

@@ -9,9 +9,10 @@ stimulus_controller = 'fields--ckeditor'
 
 form ||= current_fields_form
 options ||= {}
-options = options.merge({ data: {"#{stimulus_controller}-target": 'field' }})
 other_options ||= {}
 
+# TODO: Remove this if it's unneeded.
+options = options.merge({ data: {"#{stimulus_controller}-target": 'field' }})
 %>
 
 <%= render 'shared/fields/field', form: form, method: method, options: options, other_options: other_options do %>

data/app/views/themes/base/fields/_password_field.html.erb

@@ -6,6 +6,7 @@ options ||= {}
 
 strong_passwords = Object.const_defined?("BulletTrain::Configuration") ? BulletTrain::Configuration.strong_passwords : true
 
+# TODO: This should be `bt_options`
 if options[:show_strength_indicator] && strong_passwords
   options = options.merge({ data: {"action": "keyup->fields--password#estimateStrength" }})
 end
@@ -15,6 +16,8 @@ other_options ||= {}
 
 <div data-controller="<%= stimulus_controller %>">
   <%= render 'shared/fields/field', form: form, method: method, helper: :password_field, options: options, other_options: other_options %>
+
+  <%# TODO: This should be `bt_options` %>
   <% if options[:show_strength_indicator] %>
     <% # TODO This is the wrong place to define this kind of style, but I couldn't make this work in the `:help` content buffer. %>
     <div data-<%= stimulus_controller %>-target="strengthIndicator" class="hidden mt-1.5 text-xs text-red-500"></div>

data/app/views/themes/base/fields/_phone_field.html.erb

@@ -3,9 +3,9 @@ stimulus_controller = 'fields--phone'
 
 form ||= current_fields_form
 options ||= {}
-options = options.merge({type: 'tel', data: {method: method, "#{stimulus_controller}-target": 'field' }})
 other_options ||= {}
 
+options = options.merge({type: 'tel', data: {method: method, "#{stimulus_controller}-target": 'field' }})
 %>
 
 <div data-controller="<%= stimulus_controller %>">

data/app/views/themes/base/fields/_super_select.html.erb

@@ -6,14 +6,16 @@ options ||= {}
 other_options ||= {}
 select2_options ||= nil
 html_options ||= {}
+choices_url ||= nil
+
 html_options[:id] ||= form.field_id(method)
 html_options[:class] = "form-control select2 #{html_options[:class]}".strip
-choices_url ||= nil
+
 if choices_url.nil?
   choices ||= options_for(form, method)&.invert
 else
   # If we have a choices_url, we're going to lazy load the choices.  However, we still want to show the currently selected option.
-  method_without_id = method.to_s.gsub("_id", "")
+  method_without_id = method.to_s.gsub("_id", "") # TODO: We should use /_id$/.
   if current_selection = form.object.send(method_without_id)
     choices = [[current_selection.label_string, current_selection.id]]
   else
@@ -24,6 +26,8 @@ end
 other_options[:use_browser_time_zone] = true if other_options[:use_browser_time_zone].nil?
 use_browser_time_zone = other_options[:use_browser_time_zone]
 
+# TODO: Change name for `wrapper_options`, maybe to simply `wrapper`.
+# This seems to be internal, so we shouldn't need to write docs or a deprecation message for it.
 wrapper_options ||= {}
 wrapper_options[:data] ||= {}
 wrapper_options[:data][:controller] ||= ""

data/app/views/themes/base/fields/_trix_editor.html.erb

@@ -2,6 +2,8 @@
 form ||= current_fields_form
 labels = labels_for(form, method)
 options ||= {}
+other_options ||= {}
+
 options[:class] = "formatted_content trix-content #{options[:class]}"
 options[:placeholder] ||= labels.placeholder if labels.placeholder
 options[:data] ||= {}
@@ -16,7 +18,6 @@ options[:data][:topics] ||= []
 # current_team.scaffolding_things.map { |scaffolding_thing|
 #   {key: scaffolding_thing.name, value: scaffolding_thing.id, protocol: 'bullettrain', model: 'scaffolding/things', id: scaffolding_thing.id, label: scaffolding_thing.name, photo: photo_for(scaffolding_thing)}
 # }.to_json
-other_options ||= {}
 %>
 
 <div class="<%= 'trix-hide-toolbar' if other_options[:hide_toolbar] %>">

data/lib/bullet_train/themes/version.rb

@@ -1,5 +1,5 @@
 module BulletTrain
   module Themes
-    VERSION = "1.6.38"
+    VERSION = "1.7.0"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bullet_train-themes
 version: !ruby/object:Gem::Version
-  version: 1.6.38
+  version: 1.7.0
 platform: ruby
 authors:
 - Andrew Culver
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-03-20 00:00:00.000000000 Z
+date: 2024-03-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: standard