-
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
[$500] Web - Emojis flickers when scrolling a list of emojis and sometimes part of them disappear #30948
Comments
Job added to Upwork: https://www.upwork.com/jobs/~013d1a1c15ee78f5b3 |
Triggered auto assignment to @adelekennedy ( |
Bug0 Triage Checklist (Main S/O)
|
Triggered auto assignment to Contributor-plus team member for initial proposal review - @alitoshmatov ( |
ProposalPlease re-state the problem that we are trying to solve in this issue.Emojis flickers when scrolling a list of emojis and sometimes part of them disappear What is the root cause of that problem?The main problem is What changes do you think we should make in order to solve the problem?We have a huge number of properties to optimize FlatList But the most optimal as for me is the addition of new properties of Which will increase small memory consumption but will positively affect the display of elements So we can add App/src/components/EmojiPicker/EmojiPickerMenu/index.js Lines 504 to 524 in ece8312
Before Screen.Recording.2023-11-07.at.14.28.25.movAfter Screen.Recording.2023-11-07.at.14.27.25.movSince this problem is specific to browsers What alternative solutions did you explore? (Optional)As an alternative, we can use other properties |
ProposalPlease re-state the problem that we are trying to solve in this issue.In chrome on mac web, the emoji's flicker, particularly when scrolling fast or jumping through emoji categories What is the root cause of that problem?I identified that this is only happening in chrome and is fine in safari/firefox, and can be "fixed" if hardware acceleration is turned off in chrome system settings. So the root cause is a conflict with the chrome hardware acceleration and the emoji rendering. This could be because the What changes do you think we should make in order to solve the problem?Removing the styling of the As a proof of concept, if I remove all styling on Note in the solution the styling will match the current styling, this is a POC. What alternative solutions did you explore? (Optional)I looked into the |
📣 @daviddigital! 📣
|
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
not overdue - proposals to review above! |
Reviewing |
@ZhenjaHorbach I can still easily reproduce this issue after your solution: Screen.Recording.2023-11-13.at.21.59.02.mov |
@daviddigital I don't think removing styles is an option, even removing them didn't help. I can easily reproduce this issue |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Still open to better proposals |
@alitoshmatov, @adelekennedy Huh... This is 4 days overdue. Who can take care of this? |
Waiting for proposals here |
@alitoshmatov @adelekennedy 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! |
Swapping out the FlatList to FlashList in the linked issue (#30911) may fix this so I haven't made another proposal.
|
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
Thank you @daviddigital, I think there is a very little possibility of #30911 fixing this issue. I did a quick implementation of |
still open to better proposals |
@alitoshmatov @adelekennedy this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ and will need to go internal. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks! |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
We don't have any proposals yet that solve this issue. |
still pending better proposals (small issue so not increasing price) |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
@alitoshmatov @adelekennedy this issue is now 4 weeks old and preventing us from maintaining WAQ, can you:
Thanks! |
Current assignee @alitoshmatov is eligible for the Internal assigner, not assigning anyone new. |
I'm going to close this one - it's a small bug that I even find challenging to reproduce and we aren't getting any traction |
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: 1.3.96.0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Issue found when executing PR #30201
Action Performed:
Expected Result:
All emoji should be visible when list of emoji scrolling.
Actual Result:
Emojis flickers when scrolling a list of emojis and sometimes part of them disappear
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Bug6266710_1699318980930.30201_Mac_Chrome.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: