diff options
author | Eric Eastwood <contact@ericeastwood.com> | 2017-03-06 20:59:26 +0300 |
---|---|---|
committer | Eric Eastwood <contact@ericeastwood.com> | 2017-03-07 00:19:33 +0300 |
commit | 96cbad23378e492f2564a8ec4121b7b3548a7df3 (patch) | |
tree | 1f39f33940e914965d2f288f7f4d99fd592acd1d /lib/gitlab/emoji.rb | |
parent | f602efea65c2a816c7e29be546d2eb412fe538cc (diff) |
Fix up @DouweM review
Diffstat (limited to 'lib/gitlab/emoji.rb')
-rw-r--r-- | lib/gitlab/emoji.rb | 28 |
1 files changed, 23 insertions, 5 deletions
diff --git a/lib/gitlab/emoji.rb b/lib/gitlab/emoji.rb index 3ef92abd3cf..42703545c4f 100644 --- a/lib/gitlab/emoji.rb +++ b/lib/gitlab/emoji.rb @@ -1,8 +1,6 @@ module Gitlab module Emoji extend self - @emoji_unicode_version = JSON.parse(File.read(File.absolute_path(File.dirname(__FILE__) + '/../../node_modules/emoji-unicode-version/emoji-unicode-version-map.json'))) - @emoji_aliases = JSON.parse(File.read(File.join(Rails.root, 'fixtures', 'emojis', 'aliases.json'))) def emojis Gemojione.index.instance_variable_get(:@emoji_by_name) @@ -21,7 +19,7 @@ module Gitlab end def emojis_aliases - @emoji_aliases + @emoji_aliases ||= JSON.parse(File.read(Rails.root.join('fixtures', 'emojis', 'aliases.json'))) end def emoji_filename(name) @@ -33,7 +31,12 @@ module Gitlab end def emoji_unicode_version(name) - @emoji_unicode_version[name] + @emoji_unicode_versions_by_name ||= JSON.parse(File.read(Rails.root.join('node_modules', 'emoji-unicode-version', 'emoji-unicode-version-map.json'))) + @emoji_unicode_versions_by_name[name] + end + + def normalize_emoji_name(name) + emojis_aliases[name] || name end def emoji_image_tag(name, src) @@ -46,7 +49,22 @@ module Gitlab emoji_info = emojis[emoji_name] emoji_fallback_image_source = ActionController::Base.helpers.url_to_image("emoji/#{emoji_info['name']}.png") emoji_fallback_sprite_class = "emoji-#{emoji_name}" - "<gl-emoji #{force_fallback && sprite ? "class='emoji-icon #{emoji_fallback_sprite_class}'" : ""} data-name='#{emoji_name}' #{image ? "data-fallback-src='#{emoji_fallback_image_source}'" : ""} #{sprite ? "data-fallback-sprite-class='#{emoji_fallback_sprite_class}'" : ""} data-unicode-version='#{emoji_unicode_version(emoji_name)}'>#{force_fallback && sprite === false ? emoji_image_tag(emoji_name, emoji_fallback_image_source) : emoji_info['moji']}</gl-emoji>" + + data = { + name: emoji_name, + unicode_version: emoji_unicode_version(emoji_name) + } + data[:fallback_src] = emoji_fallback_image_source if image + data[:fallback_sprite_class] = emoji_fallback_sprite_class if sprite + ActionController::Base.helpers.content_tag 'gl-emoji', + class: ("emoji-icon #{emoji_fallback_sprite_class}" if force_fallback && sprite), + data: data do + if force_fallback && !sprite + emoji_image_tag(emoji_name, emoji_fallback_image_source) + else + emoji_info['moji'] + end + end end end end |