-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[$125] Android- Emojis -Some emojis are blacked out and seen with a square with skin tone in composer #50929
Comments
Triggered auto assignment to @greg-schroeder ( |
We think that this bug might be related to #wave-collect - Release 1 |
@greg-schroeder FYI 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 |
Job added to Upwork: https://www.upwork.com/jobs/~021846719033693258327 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @akinwale ( |
Upwork job price has been updated to $125 |
Hey! I'm Eto Olei from Callstack. I will like to take a look at this issue. Please assign it to me. Thanks! |
ProposalPlease re-state the problem that we are trying to solve in this issue.Some emojis are blacked out and seen with a square with skin tone in composer What is the root cause of that problem?We are missing some emoji unicodes in the source code. What changes do you think we should make in order to solve the problem?We should put the correct unicode in here. What alternative solutions did you explore? (Optional)If the browser(like Chrome, Safari) can not render the emoji unicode, we can add the svg file instead. Contributor detailsExpensify account email: [email protected] |
📣 @Anaslancer! 📣
|
anas-1995 Your proposal will be dismissed because you did not follow the proposal template. |
Updated, it works? |
@akinwale, @greg-schroeder Eep! 4 days overdue now. Issues have feelings too... |
📣 @akinwale 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app! |
Assigning to @teneeto from CS |
@akinwale , @greg-schroeder Did you have a chance to review my proposal? |
ProposalPlease re-state the problem that we are trying to solve in this issue.Some emojis are blacked out and seen with a square with skin tone in composer What is the root cause of that problem?We are missing some emoji unicodes in the source code. What changes do you think we should make in order to solve the problem?We should put the correct unicode in here. What alternative solutions did you explore? (Optional)If the browser(like Chrome, Safari) can not render the emoji unicode, we can add the svg file instead. Contributor detailsYour Expensify account email: [email protected] |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
Submit again here, because bot says my proposal will be dismissed. |
Working on this now. will provide more details as I continue. |
@akinwale, @teneeto, @greg-schroeder Huh... This is 4 days overdue. Who can take care of this? |
Thanks @teneeto |
@akinwale @teneeto @greg-schroeder this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks! |
@akinwale, @teneeto, @greg-schroeder 6 days overdue. This is scarier than being forced to listen to Vogon poetry! |
Hi @akinwale, I paused a bit to finalise on another issue. I'll continue with this later today or first thing by tomorrow morning. |
Thanks @teneeto |
@akinwale, @teneeto, @greg-schroeder 8 days overdue is a lot. Should this be a Weekly issue? If so, feel free to change it! |
Hey Melvin, I still have an eye on this. |
Sorry for the delays, but I just got freed up with some capacity to investigate this now. I'm currently investigating this and will share any outcome. |
Screen.Recording.2024-11-07.at.09.30.47.movCan we re-test to confirm this issue still happens? Although I haven't tested on real device (can't get my hands on my android device at the moment), but I did try on simulator and it seems to work perfectly. |
Pinging QA for a retest |
I also couldn't reproduce, so I'm going to close this as fixed elsewhere. Please reopen if you're able to reproduce. |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: 9.0.49-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5088056&group_by=cases:section_id&group_order=asc&group_id=229070
Issue reported by: Applause Internal Team
Action Performed:
Expected Result:
All the emojis in compose box should be displayed without any reduced opacity and should be correctly added to compose box when selected.
Actual Result:
Some emojis are displayed blacked out in compose box and are displayed with a code and a square with the skin tone in compose box, when selected.
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6636511_1729084254740.Blacked_Out.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: