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

Single emojis should be large #97

Closed
simmac opened this issue Feb 26, 2017 · 4 comments · Fixed by #475
Closed

Single emojis should be large #97

simmac opened this issue Feb 26, 2017 · 4 comments · Fixed by #475
Labels
app-update Requires an app update feature New feature! ui User interface related

Comments

@simmac
Copy link

simmac commented Feb 26, 2017

Expected Behavior

When sending only one single emoji, the emoji should be displayed largely to be consistent with the App.

Current Behavior

The emoji is displayed in the normal font size

Possible Solution

Check if a message only contains a single emoji (watch out for emoji modifiers like skin tones and ZWJs which could count as multiple symbols, but represent a single emoji) and set the font size to a higher amount.

Steps to Reproduce (for bugs)

  1. Send a single emoji as a message
  2. Compare the displayed message on Threema Web and the Android app

Your Environment

  • Threema Web version: 1.0.3
  • Threema for Android version: 3.02
  • Threema for Android source: Play Store
  • Browser name and version: Chrome 56
  • Operating system and version: macOS 10.12
@dbrgn dbrgn added app-update Requires an app update feature New feature! ui User interface related labels Feb 27, 2017
@simmac
Copy link
Author

simmac commented Feb 27, 2017

@dbrgn Just curious: Why would this need an app update?

@dbrgn
Copy link
Contributor

dbrgn commented Feb 27, 2017

@dbrgn Just curious: Why would this need an app update?

Because of this:

the emoji should be displayed largely to be consistent with the App

This is configurable in the app, so it would need to be synced with Threema Web. I know a few people that disabled it. (I'm not a big fan either 🙂)

@FloThinksPi
Copy link
Contributor

@dbrgn could this be also also included in local storage settings as first step like issue #8 or #9 ?

@dbrgn
Copy link
Contributor

dbrgn commented Mar 8, 2017

It could, but in contrast to the notification settings I don't see why people would want large emojis in the app but not in the browser, or vice versa.

dbrgn pushed a commit that referenced this issue May 24, 2018
@dbrgn dbrgn closed this as completed in #475 Aug 2, 2018
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! ui User interface related
Development

Successfully merging a pull request may close this issue.

3 participants