Open lanitochka17 opened 2 weeks ago
Triggered auto assignment to @garrettmknight (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.
@garrettmknight 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
We think that this bug might be related to #vip-vsp
Job added to Upwork: https://www.upwork.com/jobs/~0106a5ae69cbfb2cf2
Triggered auto assignment to Contributor-plus team member for initial proposal review - @getusha (External
)
Workspace - Workspace name is truncated inconsistently in mWeb and android
The title doesn't take the full width because flex1
isn't applied to workspaceTitleStyle
.
https://github.com/Expensify/App/blob/1abad69fa3e3144c0a0959be8e57fbba5cc639a5/src/styles/index.ts#L4851-L4854
Apply flex:1/styles.flex1
to workspaceTitleStyle
style object and also check for similar issues in other fields.
Add flex1
to menu item title and description if needed.
https://github.com/Expensify/App/assets/85894871/7b71f4d4-c191-4154-b0e6-1b598c3e6cfe
https://github.com/Expensify/App/assets/85894871/675e4db8-8f33-4a63-b99f-4658b4c5d090
@tienifr didn't we fix this before?
@getusha Hmm I'm not sure, I've been working on many PRs relating to truncation. Could you link the PR you mentioned?
@tienifr here, this one https://github.com/Expensify/App/pull/40824 I copied your PR to fix some conflicts.
I think we missed that case on Android, the PR was merged too fast. But this issue is external and the solution from @Krishna2323 will fix the issue. Should we fix that problem here? @getusha
Actually after a closer look, looks like it's on a different place. Workspace profile and Workspace list.
@Krishna2323 why was this happening only on android native?
@getusha, this is a known issue in android native, the text component doesn't fill up the space automatically. We have already applied flex1
to fix this issue in many places, e.g., https://github.com/Expensify/App/issues/30222.
Triggered auto assignment to @hayata-suenaga, see https://stackoverflow.com/c/expensify/questions/7972 for more details.
I agree that @Krishna2323's proposal looks good.
π£ @getusha π An offer has been automatically sent to your Upwork account for the Reviewer role π Thanks for contributing to the Expensify app!
π£ @Krishna2323 π An offer has been automatically sent to your Upwork account for the Contributor role π Thanks for contributing to the Expensify app!
Offer link Upwork job Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review π§βπ» Keep in mind: Code of Conduct | Contributing π
@getusha, PR ready for review.
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.4.68 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 Issue reported by: Applause - Internal Team
Action Performed:
Expected Result:
Workspace name truncated must be consistent in mweb and android
Actual Result:
Workspace name is truncated inconsistently in mweb and android
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @getusha