Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add emoji picker to chat composer #5091

Conversation

nucleartux
Copy link
Contributor

@nucleartux nucleartux commented Sep 2, 2024

Closes #5059

How it looks:

image

How it works:

emoji.mov

I'm not sure about the design, though—particularly the button style and position.

}

interface IProps {
state: EmojiPickerState
close: () => void
showAbove?: boolean
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure this is the best naming and approach


import {atoms as a} from '#/alf'
import {textInputWebEmitter} from '../TextInput.web'
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have to move this out of TextInput because it is now used everywhere that EmojiPicker is used.

Comment on lines +148 to +162
<Button
onPress={e => {
e.currentTarget.measure((_fx, _fy, _width, _height, px, py) => {
openPicker?.({top: py, left: px, right: px, bottom: py})
})
}}
style={[a.p_xs, {marginTop: 3}]}
label={_(msg`Open emoji picker`)}
accessibilityHint={_(msg`Open emoji picker`)}
variant="ghost"
shape="round"
size="small"
color="primary">
<EmojiSmile size="lg" />
</Button>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy pasted from the post composer and changed size a bit

@estrattonbailey estrattonbailey changed the base branch from main to eric/web-chat-emoji September 6, 2024 20:59
Copy link
Member

@estrattonbailey estrattonbailey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is awesome! Thanks for this :D gonna merge into a base and maybe tweak a couple things, but otherwise looks great 👍

@estrattonbailey estrattonbailey merged commit 4984f4e into bluesky-social:eric/web-chat-emoji Sep 6, 2024
6 checks passed
@nucleartux
Copy link
Contributor Author

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

(likely good first issue) Chats should have an emoji selector
2 participants