Render emoji like a sticker

This commit is contained in:
Josh Perez 2021-10-06 13:37:53 -04:00 committed by GitHub
parent 1466f9f1b2
commit 7c9b8e919c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
8 changed files with 97 additions and 30 deletions

View File

@ -44,27 +44,33 @@ span.emoji-inner {
}
img.emoji {
width: 1.4em;
height: 1.4em;
margin-bottom: -5px;
margin-left: 2px;
margin-right: 2px;
vertical-align: baseline;
width: 1.4em;
}
img.emoji.small {
width: 1.8em;
height: 1.8em;
width: 32px;
height: 32px;
}
img.emoji.medium {
width: 2em;
height: 2em;
width: 36px;
height: 36px;
}
img.emoji.large {
width: 2.5em;
height: 2.5em;
width: 40px;
height: 40px;
}
img.emoji.jumbo {
width: 3.5em;
height: 3.5em;
img.emoji.extra-large {
width: 48px;
height: 48px;
}
img.emoji.max {
width: 56px;
height: 56px;
}
// we need these, or we'll make conversation items too big in the left-nav

View File

@ -418,6 +418,10 @@
padding-bottom: 3px;
}
.module-message__container--emoji {
padding-top: 0;
}
.module-message__container--outgoing {
@include light-theme {
background-color: $color-ultramarine;

View File

@ -35,7 +35,16 @@ story.add('Skin Color Modifier', () => {
story.add('Jumbo', () => {
const props = createProps({
text: '😹😹😹',
sizeClass: 'jumbo',
sizeClass: 'max',
});
return <Emojify {...props} />;
});
story.add('Extra Large', () => {
const props = createProps({
text: '😹😹😹',
sizeClass: 'extra-large',
});
return <Emojify {...props} />;

View File

@ -192,6 +192,22 @@ story.add('Plain Message', () => {
return renderBothDirections(props);
});
story.add('Emoji Messages', () => (
<>
<Message {...createProps({ text: '😀' })} />
<br />
<Message {...createProps({ text: '😀😀' })} />
<br />
<Message {...createProps({ text: '😀😀😀' })} />
<br />
<Message {...createProps({ text: '😀😀😀😀' })} />
<br />
<Message {...createProps({ text: '😀😀😀😀😀' })} />
<br />
<Message {...createProps({ text: '😀😀😀😀😀😀😀' })} />
</>
));
story.add('Delivered', () => {
const props = createProps({
direction: 'outgoing',

View File

@ -63,7 +63,8 @@ import {
CustomColorType,
} from '../../types/Colors';
import { createRefMerger } from '../../util/refMerger';
import { emojiToData } from '../emoji/lib';
import { emojiToData, getEmojiCount } from '../emoji/lib';
import { isEmojiOnlyText } from '../../util/isEmojiOnlyText';
import type { SmartReactionPicker } from '../../state/smart/ReactionPicker';
import { getCustomColorStyle } from '../../util/getCustomColorStyle';
import { offsetDistanceModifier } from '../../util/popperUtil';
@ -618,6 +619,11 @@ export class Message extends React.PureComponent<Props, State> {
return null;
}
const isEmojiOnly = Boolean(
text && isEmojiOnlyText(text) && getEmojiCount(text) < 6
);
const isStickerLike = isSticker || isEmojiOnly;
return (
<MessageMetadata
deletedForEveryone={deletedForEveryone}
@ -628,7 +634,7 @@ export class Message extends React.PureComponent<Props, State> {
i18n={i18n}
id={id}
isShowingImage={this.isShowingImage()}
isSticker={isSticker}
isSticker={isStickerLike}
isTapToViewExpired={isTapToViewExpired}
showMessageDetail={showMessageDetail}
status={status}
@ -2325,6 +2331,7 @@ export class Message extends React.PureComponent<Props, State> {
isTapToView,
isTapToViewExpired,
isTapToViewError,
text,
} = this.props;
const { isSelected } = this.state;
@ -2333,17 +2340,24 @@ export class Message extends React.PureComponent<Props, State> {
const width = this.getWidth();
const isShowingImage = this.isShowingImage();
const isEmojiOnly =
text && isEmojiOnlyText(text) && getEmojiCount(text) < 6;
const isStickerLike = isSticker || isEmojiOnly;
const containerClassnames = classNames(
'module-message__container',
isGIF(attachments) ? 'module-message__container--gif' : null,
isSelected && !isSticker ? 'module-message__container--selected' : null,
isSticker ? 'module-message__container--with-sticker' : null,
!isSticker ? `module-message__container--${direction}` : null,
isSelected && !isStickerLike
? 'module-message__container--selected'
: null,
isStickerLike ? 'module-message__container--with-sticker' : null,
!isStickerLike ? `module-message__container--${direction}` : null,
isEmojiOnly ? 'module-message__container--emoji' : null,
isTapToView ? 'module-message__container--with-tap-to-view' : null,
isTapToView && isTapToViewExpired
? 'module-message__container--with-tap-to-view-expired'
: null,
!isSticker && direction === 'outgoing'
!isStickerLike && direction === 'outgoing'
? `module-message__container--outgoing-${conversationColor}`
: null,
isTapToView && isAttachmentPending && !isTapToViewExpired
@ -2363,7 +2377,7 @@ export class Message extends React.PureComponent<Props, State> {
const containerStyles = {
width: isShowingImage ? width : undefined,
};
if (!isSticker && direction === 'outgoing') {
if (!isStickerLike && direction === 'outgoing') {
Object.assign(containerStyles, getCustomColorStyle(customColor));
}

View File

@ -26,7 +26,13 @@ import * as log from '../../logging/log';
export const skinTones = ['1F3FB', '1F3FC', '1F3FD', '1F3FE', '1F3FF'];
export type SkinToneKey = '1F3FB' | '1F3FC' | '1F3FD' | '1F3FE' | '1F3FF';
export type SizeClassType = '' | 'small' | 'medium' | 'large' | 'jumbo';
export type SizeClassType =
| ''
| 'small'
| 'medium'
| 'large'
| 'extra-large'
| 'max';
export type EmojiSkinVariation = {
unified: string;
@ -315,19 +321,22 @@ export function getSizeClass(str: string): SizeClassType {
const emojiCount = getEmojiCount(str);
if (emojiCount > 8) {
return '';
if (emojiCount === 1) {
return 'max';
}
if (emojiCount > 6) {
return 'small';
if (emojiCount === 2) {
return 'extra-large';
}
if (emojiCount > 4) {
return 'medium';
}
if (emojiCount > 2) {
if (emojiCount === 3) {
return 'large';
}
return 'jumbo';
if (emojiCount === 4) {
return 'medium';
}
if (emojiCount === 5) {
return 'small';
}
return '';
}
data.forEach(emoji => {

View File

@ -1,8 +1,8 @@
// Copyright 2021 Signal Messenger, LLC
// SPDX-License-Identifier: AGPL-3.0-only
import * as grapheme from './grapheme';
import { getEmojiCount } from '../components/emoji/lib';
import { isEmojiOnlyText } from './isEmojiOnlyText';
type FontSizes = {
diameter: number;
@ -33,7 +33,7 @@ export function getFittedFontSize(
const sizes = getFontSizes(bubbleSize);
let candidateFontSize = sizes.text;
if (grapheme.count(text) === 1 && getEmojiCount(text) === 1) {
if (isEmojiOnlyText(text) && getEmojiCount(text) === 1) {
candidateFontSize = sizes.singleEmoji;
}

View File

@ -0,0 +1,9 @@
// Copyright 2021 Signal Messenger, LLC
// SPDX-License-Identifier: AGPL-3.0-only
import * as grapheme from './grapheme';
import { getEmojiCount } from '../components/emoji/lib';
export function isEmojiOnlyText(text: string): boolean {
return grapheme.count(text) === getEmojiCount(text);
}