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

Sync Recent Emojis #166

Open
TomasKostadinov opened this issue Mar 13, 2017 · 4 comments
Open

Sync Recent Emojis #166

TomasKostadinov opened this issue Mar 13, 2017 · 4 comments
Labels
app-update Requires an app update feature New feature!

Comments

@TomasKostadinov
Copy link

Expected Behavior

When opening the emoji panel the icons should display a "recent used" tab like in the app itself.

Current Behavior

The recent emoji tab is missing the webclient.

Possible Solution

Maybe sync emojis on startup or whenever a new emoji gets used in-app.

Your Environment

  • Threema Web version: 1.0.5
  • Threema for Android version: 3.1
  • Threema for Android source: Play Store
  • Operating system and version: Android 7.1.2 Developer Preview
@dbrgn dbrgn added feature New feature! app-update Requires an app update labels Mar 13, 2017
@sillych
Copy link
Contributor

sillych commented Sep 6, 2017

What if a emoji used in the web client? should they also be synchronized to the app?

@lgrahl
Copy link
Contributor

lgrahl commented Sep 6, 2017

@sillych Yeah, my opinion is they should be synced bidirectional.

Another thing: Syncing Emojis may be slightly problematic due to different supported Unicode versions, see #317.

@rugk
Copy link
Contributor

rugk commented Sep 6, 2017

Syncing Emojis may be slightly problematic due to different supported Unicode versions,

Well… the web client and app could just skip emojis they don't know, when creating the "recently used" list.

@lgrahl
Copy link
Contributor

lgrahl commented Sep 6, 2017

Nah, it's not that easy if you sync bidirectionally. Without taking precautions you'd erase unknown emojis from each other's lists.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app-update Requires an app update feature New feature!
Development

No branches or pull requests

5 participants