-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[HOLD #15310][$1000] Inconsistent default scrolling on the personal message section on Android that in turn hides the first line of texts, unlike mweb chrome #16930
Comments
Triggered auto assignment to @sonialiap ( |
Bug0 Triage Checklist (Main S/O)
|
@sonialiap Eep! 4 days overdue now. Issues have feelings too... |
1 similar comment
@sonialiap Eep! 4 days overdue now. Issues have feelings too... |
Reproducible |
Triggered auto assignment to @luacmartins ( |
Making it external |
Job added to Upwork: https://www.upwork.com/jobs/~010511c30c967553b5 |
Current assignee @sonialiap is eligible for the External assigner, not assigning anyone new. |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @fedirjh ( |
Current assignee @luacmartins is eligible for the External assigner, not assigning anyone new. |
ProposalPlease re-state the problem that we are trying to solve in this issue.On Android, the personal message textInput is scrolled to the bottom when opened. What is the root cause of that problem?This is the default behaviour of the textInput on native Android, if you remove the What changes do you think we should make in order to solve the problem?Implement the
and add the
What alternative solutions did you explore? (Optional)None |
Thanks @Ollyws for your proposal . Why native Android has a different behaviour ? is that expected ? Why adding |
We got one proposal from @Ollyws, but we have these outstanding questions. |
@sonialiap @luacmartins @fedirjh 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! |
Just having a dig in the RN source code, will answer those questions soon. |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
@fedirjh Seems like we already opened a PR for this but it has been sitting there for ages now without approval. |
@Ollyws Thanks for the update. The same PR Expensify/react-native#45 was merged on our RN fork and it’s being handled in #15310. @sonialiap I think we should hold this issue in favour of #15310 |
Issue not reproducible during KI retests. (First week) |
maybe we can close this one after the 2nd week? |
Waiting to see if we can still reproduce this during the next KI retest |
Still waiting on KI tests |
Waiting on KI |
Still waiting on KI |
I couldn't reproduce this. Both platforms were consistent and had the cursor set to the end of the text field. Closing. |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Action Performed:
Expected Result:
Android and mweb chrome personal message in invite members should have consistent behavior and the first lines of texts should also be shown on android properly
Actual Result:
The First line of text in personal message on invite members page on android scrolls to bottom automatically and the first line of text is not shown properly. But behaves different in mweb chrome
Workaround:
unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.2.94-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
Notes/Photos/Videos: Any additional supporting documentation
workspace.mp4
Expensify/Expensify Issue URL:
Issue reported by: @priya-zha
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1680599826934579
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: