-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
Emoji - Specific "♤"Emoji shows blank in popup and doesn't appear in "Frequently Used" #46496
Comments
Triggered auto assignment to @VictoriaExpensify ( |
@VictoriaExpensify I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors |
We think that this bug might be related to #vip-vsp |
ProposalPlease re-state the problem that we are trying to solve in this issue.Some emojis shows blank in popup and don't appear in "Frequently Used". What is the root cause of that problem?We don't have a mapping for ♤ and some other emojis as well. Line 39 in 5968b22
What changes do you think we should make in order to solve the problem?Update the mapping for missing emojis. We can get a list from the design team.
|
@VictoriaExpensify Can you check if this can have the external label? |
So this does look buggy but seems like a pretty low priority if it's only happening with the one emoji. I wonder if this needs to a reduced price if we decide to fix it. I'm going to look into that |
@VictoriaExpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Ok I've looked into this more. This issue actually isn't related to emoji's, it's related to ascii characters. But it's actually different to the issue being described because this actually effects all ascii characters: @ShridharGoel would you interested in giving an updated proposal based on the updated problem? If not, we might just close this one out and have a new issue reopened. |
I'm going to close this one out because the issue described is not the correct issue. Since this is very minor, closing it out seems like the right course of action. |
I don't think this is about ASCII, ASCII even includes 0-9, A-Z and a-z.
This is happening for emojis which don't have a mapping in our codebase.
…On Thu, 8 Aug 2024 at 12:09, VictoriaExpensify ***@***.***> wrote:
I'm going to close this one out because the issue described is not the
correct issue. Since this is very minor, closing it out seems like the
right course of action.
—
Reply to this email directly, view it on GitHub
<#46496 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIPLJHDNT3M7AW6SV2OWNULZQMHALAVCNFSM6AAAAABLWKEGY6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZVGA3DCNBWHA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 9.0.14-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): applausetester+en@applause.expensifail.com
Issue reported by: Applause - Internal Team
Issue found when executing PR #45860
Action Performed:
Expected Result:
User expects that in the popup, the "♤" is displayed but enlarged and also shows in "Frequently used" emoji section
Actual Result:
♤ does not appear in the popup and also does not appear in frequently used section
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6556951_1722310591662.Specific_emoji_does_not_appear.mp4
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: