diff options
author | Mike Greiling <mike@pixelcog.com> | 2018-11-19 10:00:49 +0300 |
---|---|---|
committer | GitLab Release Tools Bot <robert+release-tools@gitlab.com> | 2018-11-30 06:56:18 +0300 |
commit | 05472a43102c8cb52abce11f16584f9f6fd756d7 (patch) | |
tree | c01e44149fd76d591ccf9298c9f458deac65485b /app | |
parent | 2de43a823333d04ac3c4da48a9716759fa9e6c5f (diff) |
Merge branch 'fix-not-render-emoji' into 'master'
Fix not render emoji in filter dropdown
See merge request gitlab-org/gitlab-ce!23112
(cherry picked from commit 8dd89f03061aa646bcda5055fa96376fb9e30759)
428b220a Fix not render emoji in filter dropdown
59e5bad1 Fix spec to catch not rendering emoji
Diffstat (limited to 'app')
-rw-r--r-- | app/assets/javascripts/filtered_search/dropdown_emoji.js | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/app/assets/javascripts/filtered_search/dropdown_emoji.js b/app/assets/javascripts/filtered_search/dropdown_emoji.js index af7936a92fb..d9a4d06b549 100644 --- a/app/assets/javascripts/filtered_search/dropdown_emoji.js +++ b/app/assets/javascripts/filtered_search/dropdown_emoji.js @@ -69,10 +69,13 @@ export default class DropdownEmoji extends FilteredSearchDropdown { // Replace empty gl-emoji tag to real content const dropdownItems = [...this.dropdown.querySelectorAll('.filter-dropdown-item')]; dropdownItems.forEach(dropdownItem => { - const name = dropdownItem.querySelector('.js-data-value').innerText; - const emojiTag = this.glEmojiTag(name); - const emojiElement = dropdownItem.querySelector('gl-emoji'); - emojiElement.outerHTML = emojiTag; + const valueElement = dropdownItem.querySelector('.js-data-value'); + if (valueElement !== null) { + const name = valueElement.innerText; + const emojiTag = this.glEmojiTag(name); + const emojiElement = dropdownItem.querySelector('gl-emoji'); + emojiElement.outerHTML = emojiTag; + } }); } |