lib/ransack/helpers/form_helper.rb in ransack-1.8.0 vs lib/ransack/helpers/form_helper.rb in ransack-1.8.1

@@ -95,10 +95,18 @@
           @hide_indicator = @options.delete(:hide_indicator) ||
                             Ransack.options[:hide_sort_order_indicators]
           @default_order  = @options.delete :default_order
         end
 
+        def up_arrow
+          '▼'.freeze
+        end
+
+        def down_arrow
+          '▲'.freeze
+        end
+
         def name
           [ERB::Util.h(@label_text), order_indicator]
           .compact
           .join(' '.freeze)
           .html_safe
@@ -188,19 +196,18 @@
             @default_order
           end
 
           def order_indicator
             return if @hide_indicator || no_sort_direction_specified?
-            direction_arrow
+            if @current_dir == 'desc'.freeze
+              up_arrow
+            else
+              down_arrow
+            end
           end
 
           def no_sort_direction_specified?(dir = @current_dir)
-            !['asc'.freeze, 'desc'.freeze].freeze.include?(dir)
-          end
-
-          def direction_arrow
-            return Constants::DESC_ARROW if @current_dir == 'desc'.freeze
-            Constants::ASC_ARROW
+            dir != 'asc'.freeze && dir != 'desc'.freeze
           end
 
           def direction_text(dir)
             return 'asc'.freeze if dir == 'desc'.freeze
             'desc'.freeze