Skip to content

Update emojis list to show positive intent first

Robert Hunt requested to merge 363356-positive-intent-first into master

What does this MR do and why?

This MR adds a simple MVC structure to sort emojis by positive intent first. The initial implementation makes it so :thumbsup*: and :slight_smile:/:slightly_smiling_face: appear first. It can easily be expanded to others.

This does not affect the initial list of emojis that are shown. It only affects the search results.

Screenshots or screen recordings

Screen_Recording_2022-05-30_at_15.27.32

Area Tested
Epic description
Epic emoji reaction
Epic comment emoji
Epic comment emoji reaction
---------------------------------------- ------
Issue description
Issue emoji reaction
Issue comment emoji
Issue comment emoji reaction
---------------------------------------- ------
MR description
MR emoji reaction
MR comment emoji
MR comment emoji reaction
---------------------------------------- ------
Personal snippets reaction
Personal snippets creation description
Personal snippets comment emoji
Personal snippets comment emoji reaction
---------------------------------------- ------
Project snippets reaction
Project snippets creation description
Project snippets comment emoji
Project snippets comment emoji reaction

How to set up and validate locally

  1. Visit an issue, MR, epic or snippet
  2. Test that the positive intent emoji are higher up than the negative intent emoji in:
    • The emoji selection within the WYSIWYG editors
    • The reactions/awards at the top
    • The reactions/awards on the comments

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #363356 (closed)

Edited by Robert Hunt

Merge request reports