-
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
Workspace - In Workspace, while inviting applausemail user, full id is not shown #29772
Comments
Triggered auto assignment to @MitchExpensify ( |
Bug0 Triage Checklist (Main S/O)
|
ProposalPlease re-state the problem that we are trying to solve in this issue.While inviting applausemail user, full id is not shown in workspace invite page, What is the root cause of that problem?The root cause of this problem is found here in this Text component: App/src/components/SelectionList/UserListItem.js Lines 25 to 30 in 9e1f190
The problem is in the way how react native uses text break strategy for Android. What changes do you think we should make in order to solve the problem?We couldn't add styles.flex1 because of the direction of the flex for the container is set to column, therefore we can add <Text
style={[styles.optionDisplayName, isFocused ? styles.sidebarLinkActiveText : styles.sidebarLinkText, styles.sidebarLinkTextBold, styles.alignSelfStretch]}
numberOfLines={1}
>
{item.text}
</Text> The result would look like this: What alternative solutions did you explore? (Optional)NA |
I don't really understand this bug - The whole bug appears to show does it not? Or at least as much space as there is to show it @lanitochka17 ? |
Hey @MitchExpensify , The problem we're facing is that when inviting a member with a long email, the search result doesn't display the full email address (i.e the display name defaulting to emails for users with an unrenamed display name), cutting it off prematurely as displayed here on the screen shot: This issue has occurred in the past (referencing #23769 and #22453). The root of the issue lies in how Android native handles line breaks when the number of lines is set to just one. It's more prominent on physical Android devices, I have tested this on Samsung physical devices, as opposed to Android simulators (pixel device simulator) where it's not as noticeable. |
Ok got it, thanks for confirming! I do not think this is worth solving at this point tbh |
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: 1.3.85-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
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:
Action Performed:
Expected Result:
n Workspace, while inviting applausemail user, full id must be shown
Actual Result:
In Workspace, while inviting applausemail user, full id is not shown
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Android: Native
Bug6240272_1697538100332.ener.mp4
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari
MacOS: Desktop
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: