diff options
Diffstat (limited to 'src/components/emoji-input')
| -rw-r--r-- | src/components/emoji-input/emoji-input.js | 2 | ||||
| -rw-r--r-- | src/components/emoji-input/suggestor.js | 96 |
2 files changed, 47 insertions, 51 deletions
diff --git a/src/components/emoji-input/emoji-input.js b/src/components/emoji-input/emoji-input.js index 08c2f60b..d2e4a21a 100644 --- a/src/components/emoji-input/emoji-input.js +++ b/src/components/emoji-input/emoji-input.js @@ -8,7 +8,7 @@ import { take } from 'lodash' * Intended usage is: * <emoji-input v-model="something"> * <input v-model="something"/> - * </emoji-input> + * </EmojiInput> * * Works only with <input> and <textarea>. Intended to use with only one nested * input. It will find first input or textarea and work with that, multiple diff --git a/src/components/emoji-input/suggestor.js b/src/components/emoji-input/suggestor.js index eb70f299..95f14633 100644 --- a/src/components/emoji-input/suggestor.js +++ b/src/components/emoji-input/suggestor.js @@ -21,65 +21,61 @@ export default function suggest (data) { } } -function suggestEmoji (emojis) { - return input => { - const noPrefix = input.toLowerCase().substr(1) - return emojis - .filter(({ displayText }) => displayText.toLowerCase().startsWith(noPrefix)) - .sort((a, b) => { - let aScore = 0 - let bScore = 0 +export const suggestEmoji = (emojis) => input => { + const noPrefix = input.toLowerCase().substr(1) + return emojis + .filter(({ displayText }) => displayText.toLowerCase().startsWith(noPrefix)) + .sort((a, b) => { + let aScore = 0 + let bScore = 0 - // Make custom emojis a priority - aScore += Number(!!a.imageUrl) * 10 - bScore += Number(!!b.imageUrl) * 10 + // Make custom emojis a priority + aScore += a.imageUrl ? 10 : 0 + bScore += b.imageUrl ? 10 : 0 - // Sort alphabetically - const alphabetically = a.displayText > b.displayText ? 1 : -1 + // Sort alphabetically + const alphabetically = a.displayText > b.displayText ? 1 : -1 - return bScore - aScore + alphabetically - }) - } + return bScore - aScore + alphabetically + }) } -function suggestUsers (users) { - return input => { - const noPrefix = input.toLowerCase().substr(1) - return users.filter( - user => - user.screen_name.toLowerCase().startsWith(noPrefix) || - user.name.toLowerCase().startsWith(noPrefix) +export const suggestUsers = (users) => input => { + const noPrefix = input.toLowerCase().substr(1) + return users.filter( + user => + user.screen_name.toLowerCase().startsWith(noPrefix) || + user.name.toLowerCase().startsWith(noPrefix) - /* taking only 20 results so that sorting is a bit cheaper, we display - * only 5 anyway. could be inaccurate, but we ideally we should query - * backend anyway - */ - ).slice(0, 20).sort((a, b) => { - let aScore = 0 - let bScore = 0 + /* taking only 20 results so that sorting is a bit cheaper, we display + * only 5 anyway. could be inaccurate, but we ideally we should query + * backend anyway + */ + ).slice(0, 20).sort((a, b) => { + let aScore = 0 + let bScore = 0 - // Matches on screen name (i.e. user@instance) makes a priority - aScore += a.screen_name.toLowerCase().startsWith(noPrefix) * 2 - bScore += b.screen_name.toLowerCase().startsWith(noPrefix) * 2 + // Matches on screen name (i.e. user@instance) makes a priority + aScore += a.screen_name.toLowerCase().startsWith(noPrefix) ? 2 : 0 + bScore += b.screen_name.toLowerCase().startsWith(noPrefix) ? 2 : 0 - // Matches on name takes second priority - aScore += a.name.toLowerCase().startsWith(noPrefix) - bScore += b.name.toLowerCase().startsWith(noPrefix) + // Matches on name takes second priority + aScore += a.name.toLowerCase().startsWith(noPrefix) ? 1 : 0 + bScore += b.name.toLowerCase().startsWith(noPrefix) ? 1 : 0 - const diff = bScore * 10 - aScore * 10 + const diff = (bScore - aScore) * 10 - // Then sort alphabetically - const nameAlphabetically = a.name > b.name ? 1 : -1 - const screenNameAlphabetically = a.screen_name > b.screen_name ? 1 : -1 + // Then sort alphabetically + const nameAlphabetically = a.name > b.name ? 1 : -1 + const screenNameAlphabetically = a.screen_name > b.screen_name ? 1 : -1 - return diff + nameAlphabetically + screenNameAlphabetically - /* eslint-disable camelcase */ - }).map(({ screen_name, name, profile_image_url_original }) => ({ - displayText: screen_name, - detailText: name, - imageUrl: profile_image_url_original, - replacement: '@' + screen_name + ' ' - })) - /* eslint-enable camelcase */ - } + return diff + nameAlphabetically + screenNameAlphabetically + /* eslint-disable camelcase */ + }).map(({ screen_name, name, profile_image_url_original }) => ({ + displayText: screen_name, + detailText: name, + imageUrl: profile_image_url_original, + replacement: '@' + screen_name + ' ' + })) + /* eslint-enable camelcase */ } |
