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

Update Reactions component #6244

Merged
merged 9 commits into from
Jun 15, 2022
Merged

Conversation

gileluard
Copy link
Contributor

resolves #5370

Expand / collapse reactions:
ezgif-1-6e89c05fed

Press Add reaction button:
ezgif-1-575298e334

Long press:
ezgif-1-b35fe5e6e4

@gileluard gileluard requested review from a team, SBiOSoftWhare and MaximeEvrard42 and removed request for a team June 3, 2022 13:53
@github-actions
Copy link

github-actions bot commented Jun 3, 2022

📱 Scan the QR code below to install the build for this PR.
🔒 This build is for internal testing purpose. Only devices listed in the ad-hoc provisioning profile can install Element Alpha.

QR code

If you can't scan the QR code you can install the build via this link: https://i.diawi.com/gBnGgN

- Update after review
@gileluard gileluard requested review from a team and pixlwave and removed request for MaximeEvrard42 and a team June 8, 2022 15:07
Copy link
Member

@pixlwave pixlwave left a comment

Choose a reason for hiding this comment

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

Ah this is is such a nice addition! 🤩

One minor thing I noticed in the screenshots is that reactionsMoreAction image doesn't appear to be centred in the pill/button.


// MARK: Reactions

"room_event_action_reaction_more" = "%@ more";
Copy link
Member

Choose a reason for hiding this comment

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

Just to check, strings added to Untranslated.strings will never be added to Weblate, so can't be translated by the community. Is this the intention here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not at all! I thought we should put all new untranslated strings in this file. Thanks for the heads up :)

- fixed reaction misalignment on iOS15.5
- moved translatable strings to right Vector file
@gileluard gileluard requested a review from pixlwave June 10, 2022 12:38
- updated podfile with online MatrixSDK
- Update after review
# Conflicts:
#	Riot/Assets/en.lproj/Untranslated.strings
#	Riot/Generated/UntranslatedStrings.swift
Copy link
Member

@pixlwave pixlwave left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@gileluard gileluard merged commit 4f18e40 into develop Jun 15, 2022
@gileluard gileluard deleted the gil/5370-Update_Reactions_component branch June 15, 2022 14:08
stefanceriu pushed a commit that referenced this pull request Jun 28, 2022
* Update Reactions component

- Done
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.

Update Reactions component
3 participants