Skip to content
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 for payment on Wednesday, September 8th] IOU - "What's it for?" placeholder is capitalized #4673

Closed
isagoico opened this issue Aug 16, 2021 · 18 comments
Assignees
Labels
Engineering External Added to denote the issue can be worked on by a contributor Weekly KSv2

Comments

@isagoico
Copy link

isagoico commented Aug 16, 2021

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel! https://www.upwork.com/jobs/~01eb24ca989c0c7fb6


Action Performed:

  1. Click on the global create button
  2. Click on request money
  3. Set an amount
  4. Select another user as an attendee

Expected Result:

What's it for? placeholder is not capitalized.

Actual Result:

"What's it for?" placeholder is capitalized. Looks strange since all placeholders are not in all caps.

Workaround:

None needed, visual issue.

Platform:

Where is this issue occurring?

  • Web
  • iOS
  • Android
  • Desktop App
  • Mobile Web

Version Number: 1.0.85-6

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

image

Expensify/Expensify Issue URL:

View all open jobs on Upwork


From @Julesssss https://expensify.slack.com/archives/C01GTK53T8Q/p1629109910376800

The ‘WHATS’S IT FOR? ’ message looks quite jarring. I think is is supposed to be lowercase?

@MelvinBot
Copy link

Triggered auto assignment to @Julesssss (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

@thesahindia
Copy link
Member

Hi there, I think that this can be fixed by changing the translations in en.js and es.js files.
If it's available then I would like to work on this issue.

@Julesssss Julesssss added the External Added to denote the issue can be worked on by a contributor label Aug 16, 2021
@MelvinBot
Copy link

Triggered auto assignment to @laurenreidexpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@mananjadhav
Copy link
Collaborator

mananjadhav commented Aug 16, 2021

Proposal

Updated the strings in en.js

iOUConfirmationList: {
        whoPaid: 'WHO PAID?', --> 'Who paid?'
        whoWasThere: 'WHO WAS THERE?', ---> 'Who was there?'
        whatsItFor: 'WHAT\'S IT FOR?',  --> 'What\'s it for?'
    }

and es.js

iOUConfirmationList: {
        whoPaid: '¿QUIÉN PAGO?', ---> '¿Quién pago?'
        whoWasThere: '¿QUIÉN ASISTIÓ?', --> '¿Quién asistió?'
        whatsItFor: '¿PARA QUÉ ES?', --> '¿Para Qué es?'
    },

If you notice all the three strings are in caps but the other two are headers in the split money tab.

image

We could just update the given string or all of them?

@laurenreidexpensify
Copy link
Contributor

@MelvinBot MelvinBot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Aug 17, 2021
@Julesssss
Copy link
Contributor

This is a simple one, so let's go with the first submission.

@thesahindia, would you like to make a start on a PR?

@shawnborton
Copy link
Contributor

While we're at it, can we also add some margin to the top of the "What's it for?" field so there is some more visual separation between the user and the text input? Perhaps @thesahindia you can tag me on your PR and I can help guide you on the spacing from there.

@Julesssss
Copy link
Contributor

Ah, thanks Shawn -- I forgot to add that note to the issue. I don't think we'll need a change to the proposal, as long as @thesahindia is happy

@thesahindia
Copy link
Member

@thesahindia, would you like to make a start on a PR?

Sure.
Also let me know if we also want the margin. Adding the comparison below.

  1. with marginTop:8
  2. without any margin

image

@shawnborton
Copy link
Contributor

how about mt4 so there is a bit more spacing than what you have on the left?

@thesahindia
Copy link
Member

With mt4 :
image

@shawnborton
Copy link
Contributor

Nice, I think mt4 looks good.

@MelvinBot MelvinBot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Aug 17, 2021
@laurenreidexpensify
Copy link
Contributor

@thesahindia can you pls apply for the job in upwork so I can hire you? Thanks!

@thesahindia
Copy link
Member

Thanks @laurenreidexpensify, I have just applied to the job.

@laurenreidexpensify
Copy link
Contributor

QA still on going - I am OOO for payment, so reassigning. When it's deployed pls start clock for payment (which I think is now automated anyways).

@MelvinBot MelvinBot removed the Overdue label Aug 20, 2021
@laurenreidexpensify laurenreidexpensify removed their assignment Aug 20, 2021
@laurenreidexpensify laurenreidexpensify added External Added to denote the issue can be worked on by a contributor and removed External Added to denote the issue can be worked on by a contributor labels Aug 20, 2021
@MelvinBot
Copy link

Triggered auto assignment to @adelekennedy (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@Julesssss Julesssss changed the title IOU - "What's it for?" placeholder is capitalized [HOLD for payment on Thursday 26th Aug] IOU - "What's it for?" placeholder is capitalized Aug 20, 2021
@Julesssss Julesssss added Weekly KSv2 and removed Daily KSv2 labels Aug 20, 2021
@adelekennedy
Copy link

We're not overdue Melvin - just waiting for this PR to fully merge (following this SO)

Screen Shot 2021-08-30 at 8 30 18 AM

@MelvinBot MelvinBot removed the Overdue label Aug 30, 2021
@adelekennedy adelekennedy changed the title [HOLD for payment on Thursday 26th Aug] IOU - "What's it for?" placeholder is capitalized [HOLD for payment on Wednesday, September 8th] IOU - "What's it for?" placeholder is capitalized Sep 1, 2021
@adelekennedy
Copy link

@thesahindiahas been paid via Upwork, the contract has been completed and the job is closed!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Engineering External Added to denote the issue can be worked on by a contributor Weekly KSv2
Projects
None yet
Development

No branches or pull requests

8 participants