Add ordering to menu items

backend/app/models/spree/backend_configuration.rb

@@ -24,6 +24,8 @@ class BackendConfiguration < Preferences::Configuration
     class MenuItem
       attr_reader :icon, :label, :partial, :condition, :sections, :url
 
+      attr_accessor :position
+
       # @param sections [Array<Symbol>] The sections which are contained within
       #   this admin menu section.
       # @param icon [String] The icon to draw for this menu item
@@ -35,13 +37,16 @@ class MenuItem
       # @param partial [String] A partial to draw within this menu item for use
       #   in declaring a submenu
       # @param url [String] A url where this link should send the user to
+      # @param position [Integer] The position in which the menu item should render
+      #   nil will cause the item to render last
       def initialize(
         sections,
         icon,
         condition: nil,
         label: nil,
         partial: nil,
-        url: nil
+        url: nil,
+        position: nil
       )
 
         @condition = condition || -> { true }
@@ -50,6 +55,7 @@ def initialize(
         @label = label || sections.first
         @partial = partial
         @url = url
+        @position = position
       end
     end
 
@@ -65,6 +71,11 @@ def initialize(
     #
     # @!attribute menu_items
     #   @return [Array<Spree::BackendConfiguration::MenuItem>]
+    #
+    # Positioning can be determined by setting the position attribute to
+    # an Integer or nil. Menu Items will be rendered with smaller lower values
+    # first and higher values last. A position value of nil will cause the menu
+    # item to be rendered at the end of the list.
     attr_writer :menu_items
 
     # Return the menu items which should be drawn in the menu
@@ -77,40 +88,46 @@ def menu_items
           ORDER_TABS,
           'shopping-cart',
           condition: -> { can?(:admin, Spree::Order) },
+          position: 0
         ),
         MenuItem.new(
           PRODUCT_TABS,
           'th-large',
           condition: -> { can?(:admin, Spree::Product) },
-          partial: 'spree/admin/shared/product_sub_menu'
-        ),
-        MenuItem.new(
-          CONFIGURATION_TABS,
-          'wrench',
-          condition: -> { can?(:admin, Spree::Store) },
-          label: :settings,
-          partial: 'spree/admin/shared/settings_sub_menu',
-          url: :admin_stores_path
+          partial: 'spree/admin/shared/product_sub_menu',
+          position: 1
         ),
         MenuItem.new(
           PROMOTION_TABS,
           'bullhorn',
           partial: 'spree/admin/shared/promotion_sub_menu',
           condition: -> { can?(:admin, Spree::Promotion) },
-          url: :admin_promotions_path
+          url: :admin_promotions_path,
+          position: 2
         ),
         MenuItem.new(
           STOCK_TABS,
           'cubes',
           condition: -> { can?(:admin, Spree::StockItem) },
           label: :stock,
-          url: :admin_stock_items_path
+          url: :admin_stock_items_path,
+          position: 3
         ),
         MenuItem.new(
           USER_TABS,
           'user',
           condition: -> { Spree.user_class && can?(:admin, Spree.user_class) },
-          url: :admin_users_path
+          url: :admin_users_path,
+          position: 4
+        ),
+        MenuItem.new(
+          CONFIGURATION_TABS,
+          'wrench',
+          condition: -> { can?(:admin, Spree::Store) },
+          label: :settings,
+          partial: 'spree/admin/shared/settings_sub_menu',
+          url: :admin_stores_path,
+          position: 5
         )
       ]
     end

backend/app/views/spree/admin/shared/_tabs.html.erb

@@ -1,4 +1,4 @@
-<% Spree::Backend::Config.menu_items.each do |menu_item| %>
+<% Spree::Backend::Config.menu_items.sort_by { |item| item.position || Float::INFINITY }.each do |menu_item| %>
   <% if instance_exec(&menu_item.condition) %>
     <%=
       tab(