Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Add existing reactions below message #2940

Merged
merged 3 commits into from
May 2, 2019
Merged

Conversation

jryans
Copy link
Collaborator

@jryans jryans commented May 2, 2019

This change both displays the existing reactions and also allows you to increment by clicking on them.

existing-reactions

Fixes element-hq/element-web#9573
Fixes element-hq/element-web#9486

@jryans jryans requested a review from a team May 2, 2019 11:05
jryans added 3 commits May 2, 2019 12:05
This moves the check about whether an event is actionable (for the purpose of
replies, edits, reactions, etc.) to shared utils module.
This displays the existing reactions a message has from all users below the
message.

Since we don't currently have an API to actually get these events yet,
adds a temporary hook that looks for a specific message to inject some sample
data. This helps build out the UI for now and can be removed once it exists.

Fixes element-hq/element-web#9573
This allows you to increment an existing reaction below a message by clicking on
it.

At the moment, this is not linked to the action bar, so they each are using
local state. We'll likely want to add some mechanism so that we can local echo
to both of these UI areas at the same time, but that can be done separately.

Fixes element-hq/element-web#9486
@jryans jryans force-pushed the jryans/reactions-below-message branch from 3048a50 to 87f737b Compare May 2, 2019 11:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reactions: Display existing reactions below the message Reactions: Increment existing reaction type on click
2 participants