adding support for glyph helper custom classes

seyhunak · Jan 14, 2017 · 07e9ac2 · 07e9ac2
1 parent d3776dd
commit 07e9ac2
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/README.md b/README.md
@@ -599,6 +599,8 @@ Glyph:
 <%= glyph(:pencil) %> <i class="icon-pencil"></i>
 
 <%= glyph(:pencil, {tag: :span}) %> <span class="icon-pencil"></span>
+
+<%= glyph(:pencil, {class: 'foo'}) %> <i class="icon-pencil foo"></i>
 ```
 
 ### I18n Internationalization Support
@@ -616,7 +618,7 @@ Please see CONTRIBUTERS.md for contributors list
 
 
 ## About Me
-Senior Software Developer @INGBank 
+Senior Software Developer @INGBank
 Istanbul / Turkey
 
 

diff --git a/app/helpers/glyph_helper.rb b/app/helpers/glyph_helper.rb
@@ -9,12 +9,13 @@ module GlyphHelper
   # glyph(:lock, {tag: :span})
   # # => <span class="icon-lock"></span>
   def glyph(*names)
-    options = (names.last.kind_of?(Hash)) ? names.pop : {}
-    names.map! { |name| name.to_s.gsub('_','-') }
+    options = names.last.kind_of?(Hash) ? names.pop : {}
+    names.map! { |name| name.to_s.tr('_', '-') }
     names.map! do |name|
       name =~ /pull-(?:left|right)/ ? name : "glyphicon glyphicon-#{name}"
     end
     options[:tag] = options[:tag] ||= :i
-    content_tag options[:tag], nil, :class => names
+    names.push options[:class]
+    content_tag options[:tag], nil, class: names
   end
 end