-
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
[$250] Bank account - SSN translated placeholder overlaps with data entered in the field #49807
Comments
Triggered auto assignment to @twisterdotcom ( |
Edited by proposal-police: This proposal was edited at 2024-09-26 19:28:01 UTC. ProposalPlease re-state the problem that we are trying to solve in this issue.SSN translated placeholder overlaps with data entered in the field What is the root cause of that problem?The Spanish translation is too long which is why it overlaps. This issue will occur in all the places where the placeholder goes to the second line. We don’t see this elsewhere because we don’t have long translations as placeholders. Line 2107 in 965bcf9
What changes do you think we should make in order to solve the problem?We can use a small translation here
Instead of using the full form 'social security number' in the Spanish translation, we can use 'SSN.' We should make sure that we are using the correct Spanish translation. The Spanish translation can be confirmed in PR. We should also check for places where we are using long texts as placeholders. What alternative solutions did you explore? (Optional)Or we can use
|
Looks like some padding issue with placeholder. I can handle it with padding or placeholder font size. |
📣 @jais-nikhil! 📣
|
Edited by proposal-police: This proposal was edited at 2024-09-27 08:43:52 UTC. ProposalPlease re-state the problem that we are trying to solve in this issue.
What is the root cause of that problem?
What changes do you think we should make in order to solve the problem?
What alternative solutions did you explore? (Optional) |
Job added to Upwork: https://www.upwork.com/jobs/~021839606882609058645 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @Ollyws ( |
@twisterdotcom Can you confirm the expected behavior here? In case the input label is too long, should we:
|
We should use the elipses |
@twisterdotcom The Spanish translation here is not the same as the English translation. We don't have long placeholders anywhere else. I think we should use the same translation as we have in English here. Also, such long placeholders look weird. My suggestion is to use the same copy as we do in English. |
@twisterdotcom Also, we use 'SSN' in other places too, where we have the correct and shorter translation. We can use the same translation here to fix this Line 289 in 53a11d7
|
Ah, good point. Yes, absolutely. I think the acronym is fine to keep using. |
If we disable overflow it it will read |
Triggered auto assignment to @deetergp, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
How to join myself as a contributor for expensify? |
📣 @Nomanahmeda789! 📣
|
📣 @Nodebrute 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
@twisterdotcom The pr was deployed to production 2 weeks ago #49863 (comment). It is ready for payment. |
Payment Summary:
|
Requested in ND. |
$250 approved for @Ollyws |
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: 9.0.40-1
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
Email or phone of affected tester (no customers): applausetester+vd_web092624@applause.expensifail.com
Issue reported by: Applause - Internal Team
Action Performed:
Pre-requisite: user must have created a workspace and have enabled Workflows. Also, user must have set the language to Spanish
Expected Result:
SSN placeholder should not overlap with the data entered in the field
Actual Result:
SSN placeholder overlaps with the data entered in the field
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6616465_1727374059409.bandicam_2024-09-26_13-52-07-941.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @OllywsThe text was updated successfully, but these errors were encountered: