Port admin views to Bootstrap 4

Author: mvz

Manifest.txt

@@ -173,6 +173,7 @@ app/views/admin/settings/feedback.html.erb
 app/views/admin/settings/index.html.erb
 app/views/admin/settings/write.html.erb
 app/views/admin/shared/_edit.html.erb
+app/views/admin/shared/_flash.html.erb
 app/views/admin/shared/_menu.html.erb
 app/views/admin/shared/_twitter_alert.html.erb
 app/views/admin/sidebar/_available.html.erb

app/assets/stylesheets/administration_structure.css.scss

@@ -58,10 +58,6 @@ tr .action {
 .page-title { display:inline-block }
 /* Classes for menu icons */
 
-.btn-action {
-  padding: 4px;
-}
-
 #article_title,
 #page_title {
   font-size: 2.3em;

app/assets/stylesheets/sidebar_admin.css.scss

@@ -27,10 +27,4 @@
   h3 {
     margin-top: 0;
   }
-  .deletion_link {
-    position: absolute;
-    top: 20px;
-    right: 20px;
-    display: inline-block;
-  }
 }

app/helpers/admin/base_helper.rb

@@ -35,11 +35,9 @@ def plugin_options(kind)
   end
 
   def show_actions(item)
-    tag.div(class: "action", style: "") do
-      safe_join [button_to_edit(item),
-                 button_to_delete(item),
-                 button_to_short_url(item)], " "
-    end
+    safe_join [button_to_edit(item),
+               button_to_delete(item),
+               button_to_short_url(item)], " "
   end
 
   def display_pagination(collection, cols, _first = "", _last = "")
@@ -53,34 +51,42 @@ def display_pagination(collection, cols, _first = "", _last = "")
   def button_to_edit(item)
     link_to(t("generic.edit"),
             { action: "edit", id: item.id },
-            { class: "btn btn-primary btn-xs btn-action" })
+            { class: "btn btn-primary btn-sm" })
   end
 
   def button_to_delete(item)
     confirm_text = t("admin.shared.destroy.are_you_sure",
                      element: item.class.model_name.human.downcase)
     link_to(t("generic.delete"),
             { action: "destroy", id: item.id },
-            { class: "btn btn-danger btn-xs btn-action", method: :delete,
+            { class: "btn btn-danger btn-sm", method: :delete,
               data: { confirm: confirm_text } })
   end
 
   def button_to_short_url(item)
     return "" if item.short_url.nil?
 
     link_to(t("generic.short_url"), item.short_url,
-            class: "btn btn-success btn-xs btn-action")
+            class: "btn btn-success btn-sm")
   end
 
   def twitter_available?(blog, user)
     blog.has_twitter_configured? && user.has_twitter_configured?
   end
 
+  def top_menu_item(name, url)
+    if current_page? url
+      tag.li(link_to(name, "#", class: "active nav-link"), class: "nav-item")
+    else
+      tag.li(link_to(name, url, class: "nav-link"), class: "nav-item")
+    end
+  end
+
   def menu_item(name, url)
     if current_page? url
-      tag.li(link_to(name, "#"), class: "active")
+      link_to(name, "#", class: "active dropdown-item")
     else
-      tag.li(link_to(name, url))
+      link_to(name, url, class: "dropdown-item")
     end
   end
 end

app/helpers/admin/feedback_helper.rb

@@ -23,19 +23,19 @@ def show_feedback_actions(item, context = "listing")
   def button_to_edit_comment(item)
     link_to(t("generic.edit"),
             { controller: "admin/feedback", action: "edit", id: item.id },
-            { class: "btn btn-primary btn-xs btn-action" })
+            { class: "btn btn-primary btn-sm" })
   end
 
   def button_to_delete_comment(item)
     link_to(t("generic.delete"),
             { controller: "admin/feedback", action: "destroy", id: item.id },
-            { class: "btn btn-danger btn-xs btn-action" })
+            { class: "btn btn-danger btn-sm" })
   end
 
   def button_to_conversation(item)
     link_to(t("generic.conversation"),
             { controller: "admin/feedback", action: "article", id: item.article_id },
-            { class: "btn btn-default btn-xs btn-action" })
+            { class: "btn btn-default btn-sm" })
   end
 
   def change_status(item, context = "listing")
@@ -46,7 +46,7 @@ def change_status(item, context = "listing")
     link_to(link_text,
             { controller: "admin/feedback", action: "change_state",
               id: item.id, context: context },
-            { class: "btn btn-#{button_type} btn-xs btn-action",
+            { class: "btn btn-#{button_type} btn-sm",
               method: :post, remote: true })
   end
 end