trestle 0.9.5 → 0.9.6

data/app/views/trestle/flash/_flash.html.erb

@@ -2,6 +2,6 @@
   <%= render "trestle/flash/alert", html_class: "alert-success", icon: icon("alert-icon far fa-check-circle"), alert: normalize_flash_alert(flash[:message]) %>
 <% elsif flash[:error] -%>
   <%= render layout: "trestle/flash/alert", locals: { html_class: "alert-danger", icon: icon("alert-icon far fa-times-circle"), alert: normalize_flash_alert(flash[:error]) } do %>
-    <%= render "trestle/flash/debug" if debug_form_errors? %>
+    <%= render "trestle/flash/debug", errors: instance.errors if debug_form_errors? %>
   <% end %>
 <% end -%>

data/config/routes.rb

@@ -1,5 +1,5 @@
 Trestle::Engine.routes.draw do
-  Trestle.admins.each do |name, admin|
+  Trestle.registry.each do |admin|
     instance_eval(&admin.routes)
   end
 

data/frontend/css/components/_alerts.scss

@@ -32,13 +32,18 @@
 
 .alert-icon {
   margin-right: $alert-padding-x;
+
   font-size: 40px;
+  width: 40px;
+
+  text-align: center;
 
   @include media-breakpoint-down(sm) {
     margin-left: $alert-padding-x * 0.25;
     margin-right: $alert-padding-x * 0.75;
 
     font-size: 32px;
+    width: 32px;
   }
 }
 
@@ -74,10 +79,16 @@
   border-color: $border;
 
   a {
+    color: $color;
+
     &:hover, &:focus {
       color: mix($color, $background, 80%);
     }
   }
+
+  code {
+    color: $color;
+  }
 }
 
 .alert-success {
@@ -88,6 +99,26 @@
   @include alert-variant($alert-danger-bg, $alert-danger-border, $alert-danger-color);
 }
 
+.alert-info {
+  @include alert-variant($alert-info-bg, $alert-info-border, $alert-info-color);
+}
+
+.alert-warning {
+  @include alert-variant($alert-warning-bg, $alert-warning-border, $alert-warning-color);
+}
+
+.alert-light {
+  @include alert-variant($alert-light-bg, $alert-light-border, $alert-light-color);
+}
+
+.alert-dark {
+  @include alert-variant($alert-dark-bg, $alert-dark-border, $alert-dark-color);
+}
+
+.alert-primary {
+  @include alert-variant($alert-primary-bg, $alert-primary-border, $alert-primary-color);
+}
+
 
 // Debug errors
 

data/frontend/css/components/_avatar.scss

@@ -41,3 +41,17 @@
     font-weight: 500;
   }
 }
+
+.avatar-sm {
+  width: 24px !important;
+  height: 24px !important;
+}
+
+.avatar-lg {
+  width: 64px !important;
+  height: 64px !important;
+
+  .avatar-fallback {
+    font-size: 1.5rem;
+  }
+}

data/frontend/css/components/_buttons.scss

@@ -1,5 +1,4 @@
 .btn {
-  border-width: 0;
   box-shadow: none;
 
   &.loading {

data/frontend/css/components/_datepicker.scss

@@ -13,6 +13,7 @@
 
   &:hover, &:focus {
     color: #aaa;
+    text-decoration: none;
   }
 
   &::before {

data/frontend/css/components/_modal.scss

@@ -24,6 +24,8 @@
   }
 
   &.background {
+    overflow-y: hidden;
+
     .modal-dialog {
       transform: scale(0.9);
       opacity: 0.75;

data/frontend/css/components/_tags.scss

@@ -16,6 +16,8 @@
 }
 
 a.tag {
+  font-weight: inherit;
+
   &:hover, &:focus {
     text-decoration: none;
     color: $tag-hover-color;

data/frontend/css/components/_timestamp.scss

@@ -4,7 +4,7 @@
 
   small {
     display: block;
-    color: #666;
+    opacity: 0.75;
   }
 
   &.timestamp-inline,

data/frontend/css/layout/_sidebar.scss

@@ -16,11 +16,13 @@
   height: $header-height;
 
   display: flex;
-  align-items: center;
+  align-items: stretch;
 
   .navbar-toggler {
     outline: none;
 
+    align-self: center;
+
     margin-left: $grid-gutter-width / 2;
     margin-right: $grid-gutter-width / 2;
 
@@ -44,7 +46,6 @@
   display: flex;
   align-items: center;
 
-  height: 100%;
   padding: 10px 15px;
 
   font-size: 1.4rem;

data/frontend/css/variables/_bootstrap.scss

@@ -7,6 +7,7 @@ $theme-colors: (
   "warning": #f3bd71,
   "danger":  #e17572,
   "light":   #bbbbbb,
+  "dark":    #444444,
 
   // Backwards compatibility
   "default": #bbbbbb

data/frontend/css/variables/_trestle.scss

@@ -101,6 +101,26 @@ $alert-danger-bg:      #de7471;
 $alert-danger-color:   white;
 $alert-danger-border:  #d85956;
 
+$alert-info-bg:        darken(theme-color("info"), 10%);
+$alert-info-color:     white;
+$alert-info-border:    darken($alert-info-bg, 10%);
+
+$alert-warning-bg:     darken(theme-color("warning"), 5%);
+$alert-warning-color:  white;
+$alert-warning-border: darken($alert-warning-bg, 5%);
+
+$alert-light-bg:       theme-color("light");
+$alert-light-color:    $body-color;
+$alert-light-border:   darken($alert-light-bg, 5%);
+
+$alert-dark-bg:        theme-color("dark");
+$alert-dark-color:     white;
+$alert-dark-border:    darken($alert-dark-bg, 10%);
+
+$alert-primary-bg:     darken(theme-color("primary"), 5%);
+$alert-primary-color:  white;
+$alert-primary-border: darken($alert-primary-bg, 10%);
+
 
 // Modals
 

data/frontend/js/components/dialog.js

@@ -20,14 +20,19 @@ function createElement () {
     focus: false
   })
 
-  // Remove dialog elements once hidden
   $el.on('hidden.bs.modal', function () {
+    // Restore modal-open class on body if background modals still visible
+    if ($el.prevAll('.modal.show').length) {
+      $(document.body).addClass('modal-open')
+    }
+
+    // Remove dialog elements once hidden
     $el.remove()
   })
 
   // Restore the next visible modal to the foreground
   $el.on('hide.bs.modal', function () {
-    $el.prevAll('.modal.in').last().removeClass('background')
+    $el.prevAll('.modal.show').last().removeClass('background')
   })
 
   // Set X-Trestle-Dialog header on AJAX requests initiated from the dialog
@@ -120,7 +125,7 @@ export default class Dialog {
     this.$el.modal('show')
 
     // Background any existing visible modals
-    this.$el.prevAll('.modal.in').addClass('background')
+    this.$el.prevAll('.modal.show').addClass('background')
   }
 
   hide () {

data/frontend/theme/trestle/theme/_defaults.scss

@@ -8,7 +8,7 @@ $theme-colors: (
 $theme-color-interval: 8% !default;
 
 // The yiq lightness value that determines when the lightness of color changes from "dark" to "light". Acceptable values are between 0 and 255.
-$yiq-contrasted-threshold: 150 !default;
+$yiq-contrasted-threshold: 175 !default;
 
 // Customize the light and dark text colors for use in our YIQ color contrast function.
 $yiq-text-dark:  #212529 !default;
@@ -115,9 +115,9 @@ $badge-focus-width: $input-btn-focus-width !default;
 
 // Alerts
 
-$alert-bg-level:     -10 !default;
-$alert-border-level: -9 !default;
-$alert-color-level:  6 !default;
+$alert-bg-level:     2 !default;
+$alert-border-level: 3 !default;
+$alert-color-level:  -12 !default;
 
 
 // Progress bars

data/lib/generators/trestle/install/templates/_theme.scss

@@ -9,5 +9,5 @@
 //
 //   $theme-colors: (
 //     "primary":   #337ab7,
-//     "secondary": #b3b53d
+//     "secondary": #719dc3
 //   );

data/lib/generators/trestle/install/templates/trestle.rb.erb

@@ -15,7 +15,7 @@ Trestle.configure do |config|
   #
   # config.site_logo_small = "logo-small.png"
 
-  # Speficy a favicon to be used within the admin.
+  # Specify a favicon to be used within the admin.
   #
   # config.favicon = "favicon.ico"
 
@@ -103,6 +103,11 @@ Trestle.configure do |config|
   #
   # config.persistent_params << :query
 
+  # List of methods to try calling on an instance when displayed by the `display` helper.
+  # Defaults to [:display_name, :full_name, :name, :title, :username, :login, :email].
+  #
+  # config.display_methods.unshift(:admin_label)
+
   # Customize the default adapter class used by all admin resources.
   # See the documentation on Trestle::Adapters::Adapter for details on
   # the adapter methods that can be customized.

data/lib/trestle/adapters/active_record_adapter.rb

@@ -60,6 +60,8 @@ module Trestle
             Attribute::Association.new(column.name, class: -> { reflection.klass }, name: name, polymorphic: reflection.polymorphic?, type_column: reflection.foreign_type)
           elsif column.name.end_with?("_type") && (name = column.name.sub(/_type$/, '')) && (reflection = model.reflections[name])
             # Ignore type columns for polymorphic associations
+          elsif enum_column?(column)
+            Attribute.new(column.name, :enum, values: enum_values(column))
           else
             Attribute.new(column.name, column.type, array_column?(column) ? { array: true } : {})
           end
@@ -81,6 +83,20 @@ module Trestle
       def array_column?(column)
         column.respond_to?(:array?) && column.array?
       end
+
+      def enum_column?(column)
+        model.defined_enums.key?(column.name)
+      end
+
+      def enum_values(column)
+        model.defined_enums[column.name].map { |key, value|
+          [value, enum_human_name(column, key)]
+        }
+      end
+
+      def enum_human_name(column, value)
+        human_attribute_name([column.name.pluralize, value].join("."), default: value.humanize)
+      end
     end
   end
 end

data/lib/trestle/form/automatic.rb

@@ -32,6 +32,8 @@ module Trestle
                 datetime_field attribute.name
               when :boolean
                 check_box attribute.name
+              when :enum
+                collection_radio_buttons attribute.name, attribute.options[:values] || [], :first, :last
               when :json, :jsonb
                 value = instance.public_send(attribute.name)
                 text_area attribute.name, value: value.try(:to_json)

data/lib/trestle/form/field.rb

@@ -45,7 +45,7 @@ module Trestle
       end
 
       def readonly?
-        options[:readonly] || admin.readonly?
+        options[:readonly] || admin.try(:readonly?)
       end
 
       def normalize_options!

data/lib/trestle/form/renderer.rb

@@ -4,6 +4,9 @@ require "action_view/helpers"
 module Trestle
   class Form
     class Renderer
+      def self.ruby2_keywords(*)
+      end unless respond_to?(:ruby2_keywords, true)
+
       include ::ActionView::Context
       include ::ActionView::Helpers::CaptureHelper
 
@@ -40,7 +43,7 @@ module Trestle
         concat(result)
       end
 
-      def method_missing(name, *args, &block)
+      ruby2_keywords def method_missing(name, *args, &block)
         target = @form.respond_to?(name) ? @form : @template
 
         if block_given? && !RAW_BLOCK_HELPERS.include?(name)

data/lib/trestle/registry.rb

@@ -0,0 +1,61 @@
+module Trestle
+  class Registry
+    include Enumerable
+
+    # The admins hash is left exposed for backwards compatibility
+    attr_reader :admins
+
+    def initialize
+      reset!
+    end
+
+    def each(&block)
+      @admins.values.sort_by(&:admin_name).each(&block)
+    end
+
+    def reset!
+      @admins = {}
+      @models = {}
+    end
+
+    def empty?
+      none?
+    end
+
+    def register(admin, register_model: true)
+      @admins[admin.admin_name] = admin
+
+      if register_model && register_admin_for_model_loookup?(admin)
+        @models[admin.model.name] ||= admin
+      end
+
+      admin
+    end
+
+    def lookup_admin(admin)
+      # Given object is already an admin class
+      return admin if admin.is_a?(Class) && admin < Trestle::Admin
+
+      @admins[admin.to_s]
+    end
+    alias lookup lookup_admin
+
+    def lookup_model(model)
+      # Lookup each class in the model's ancestor chain
+      while model
+        admin = @models[model.name]
+        return admin if admin
+
+        model = model.superclass
+      end
+
+      # No admin found
+      nil
+    end
+
+  private
+    def register_admin_for_model_loookup?(admin)
+      admin.respond_to?(:model) && !(admin.respond_to?(:singular?) && admin.singular?)
+    end
+  end
+end

data/lib/trestle/reloader.rb

@@ -26,7 +26,7 @@ module Trestle
     end
 
     def clear
-      Trestle.admins = {}
+      Trestle.registry.reset!
     end
 
     def load_paths

data/lib/trestle/table/automatic.rb

@@ -14,6 +14,13 @@ module Trestle
           case attribute.type
           when :association
             Column.new(attribute.association_name, sort: false)
+          when :enum
+            Column.new(attribute.name, align: :center, sort: attribute.name) do |instance|
+              if value = instance.public_send(attribute.name)
+                full_attribute_name = [attribute.name.to_s.pluralize, value].join(".")
+                status_tag(admin.human_attribute_name(full_attribute_name, default: value.humanize))
+              end
+            end
           else
             Column.new(attribute.name, link: index.zero?, align: (:center if [:datetime, :boolean].include?(attribute.type)))
           end

data/lib/trestle/version.rb

@@ -1,3 +1,3 @@
 module Trestle
-  VERSION = "0.9.5"
+  VERSION = "0.9.6"
 end

data/lib/trestle.rb

@@ -23,27 +23,29 @@ module Trestle
   require_relative "trestle/tab"
   require_relative "trestle/table"
   require_relative "trestle/admin"
+  require_relative "trestle/registry"
   require_relative "trestle/resource"
 
-  mattr_accessor :admins
-  self.admins = {}
+  # The registry records all active Trestle admins and facilitates lookups.
+  mattr_accessor :registry
+  self.registry = Registry.new
 
-  def self.admin(name, options={}, &block)
-    register(Admin::Builder.create(name, options, &block))
+  class << self
+    # Expose registry methods on Trestle module
+    delegate :register, :lookup, :lookup_model, :admins, to: :registry
   end
 
-  def self.resource(name, options={}, &block)
-    register(Resource::Builder.create(name, options, &block))
+  # Builds and registers a new plain admin
+  def self.admin(name, **options, &block)
+    register(Admin::Builder.create(name, options, &block))
   end
 
-  def self.register(admin)
-    self.admins[admin.admin_name] = admin
+  # Builds and registers a new admin resource
+  def self.resource(name, register_model: true, **options, &block)
+    register(Resource::Builder.create(name, options, &block), register_model: register_model)
   end
 
-  def self.lookup(admin)
-    return admin if admin.is_a?(Class) && admin < Trestle::Admin
-    self.admins[admin.to_s]
-  end
+  # Configuration methods
 
   def self.config
     @configuration ||= Configuration.new
@@ -53,14 +55,27 @@ module Trestle
     config.configure(&block)
   end
 
+  # Builds the global navigation by combining the menu options from the
+  # Trestle configuration along with menu blocks from admin resources.
   def self.navigation(context)
-    blocks = config.menus + admins.values.map(&:menu).compact
+    blocks = config.menus + registry.map(&:menu).compact
     Navigation.build(blocks, context)
   end
 
+  # Returns the I18n fallbacks for the given locale.
+  #
+  # This is used from within a Sprockets asset (JavaScript)
+  # to determine which locale files to include.
+  #
+  # Examples
+  #
+  #   Trestle.i18n_fallbacks("pt-BR") => ["pt-BR", "pt"]
+  #   Trestle.i18n_fallbacks("ca") => ["ca", "es-ES", "es"] %>
+  #
+  # Returns an array of locale Strings.
   def self.i18n_fallbacks(locale=I18n.locale)
     if I18n.respond_to?(:fallbacks)
-      I18n.fallbacks[locale]
+      I18n.fallbacks[locale].map(&:to_s)
     elsif locale.to_s.include?("-")
       fallback = locale.to_s.split("-").first
       [locale, fallback]

data/package.json

@@ -1,6 +1,6 @@
 {
   "name": "trestle",
-  "version": "0.9.0",
+  "version": "0.9.6",
   "description": "A modern, responsive admin framework for Ruby on Rails",
   "homepage": "https://trestle.io",
   "repository": "https://github.com/TrestleAdmin/trestle.git",
@@ -13,36 +13,33 @@
     "watch": "webpack --mode development --watch"
   },
   "devDependencies": {
-    "@babel/core": "^7.6.0",
-    "@babel/preset-env": "^7.6.0",
-    "autoprefixer": "^9.6.1",
+    "@babel/core": "^7.14.6",
+    "@babel/preset-env": "^7.14.7",
+    "autoprefixer": "^10.2.6",
     "babel-loader": "^8.0.6",
-    "copy-webpack-plugin": "^5.0.4",
-    "css-loader": "^3.2.0",
-    "expose-loader": "^0.7.5",
-    "file-loader": "^6.0.0",
-    "mini-css-extract-plugin": "^0.9.0",
-    "node-sass": "^4.14.1",
-    "optimize-css-assets-webpack-plugin": "^5.0.3",
-    "postcss-loader": "^3.0.0",
-    "sass-loader": "^8.0.0",
-    "uglify-js": "^3.9.3",
-    "uglifyjs-webpack-plugin": "^2.2.0",
-    "url-loader": "^4.1.0",
-    "webpack": "^4.40.2",
-    "webpack-cli": "^3.3.9"
+    "copy-webpack-plugin": "^9.0.1",
+    "css-loader": "^5.2.6",
+    "css-minimizer-webpack-plugin": "^3.0.2",
+    "expose-loader": "^3.0.0",
+    "mini-css-extract-plugin": "^2.0.0",
+    "node-sass": "^7.0.1",
+    "postcss": "^8.3.5",
+    "postcss-loader": "^6.1.1",
+    "sass-loader": "^12.1.0",
+    "webpack": "^5.42.0",
+    "webpack-cli": "^4.7.2"
   },
   "dependencies": {
-    "@fortawesome/fontawesome-free": "^5.11.1",
-    "@rails/ujs": "^6.0.3",
+    "@fortawesome/fontawesome-free": "^5.15.3",
+    "@rails/ujs": "^6.1.3-1",
     "bootstrap": "^4.5.0",
-    "bootstrap-confirmation2": "^4.1.0",
+    "bootstrap-confirmation2": "^4.2.1",
     "bs-custom-file-input": "^1.3.2",
     "flatpickr": "^4.6.2",
-    "jquery": "^3.5.1",
+    "jquery": "^3.6.0",
     "magnific-popup": "^1.1.0",
     "popper.js": "^1.15.0",
     "select2": "^4.0.10",
-    "select2-theme-bootstrap4": "^0.2.0-beta.4"
+    "select2-theme-bootstrap4": "^1.0.0"
   }
 }