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 August 11th] iPad - "Manage cards" text is not centered in the button #4338

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

Comments

@isagoico
Copy link

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:

  1. Log in to staging
  2. Navigate to a workspace

Expected Result:

All texts that are inside buttons should be centered

Actual Result:

The text "Manage cards" is not centered in the button

Workaround:

Can the user still use Expensify without this being fixed? Have you informed them of the workaround?

Platform:

Where is this issue occurring?

  • iOS iPad

Version Number: 1.0.81-3

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

Notes/Photos/Videos:

image

Expensify/Expensify Issue URL:

View all open jobs on Upwork


From @jasperhuangg https://expensify.slack.com/archives/C01GTK53T8Q/p1627608730003300

“Manage Cards” button text isn’t centered on iPad

@MelvinBot
Copy link

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

@stitesExpensify stitesExpensify removed their assignment Jul 30, 2021
@stitesExpensify stitesExpensify added the External Added to denote the issue can be worked on by a contributor label Jul 30, 2021
@MelvinBot
Copy link

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

@rdjuric
Copy link
Contributor

rdjuric commented Jul 31, 2021

This is caused by the padding we're adding to textStyle here

textStyles={[
!isSmallScreenWidth ? styles.p5 : {},
]}

We add this padding because our page was designed to only use width: auto on the button when the viewport passes our isSmallScreenWidth check. When it doesn't, we manually add padding so that the text has some breathing room inside the button.

The problem is that styles.p5 also adds vertical padding while we only want horizontal padding. Changing it to [styles.pr5, styles.pl5] gives the correct behavior on all platforms.

Simulator Screen Shot - iPad Pro (12 9-inch) (4th generation) - 2021-07-30 at 21 00 55

@MelvinBot MelvinBot added Overdue Help Wanted Apply this label when an issue is open to proposals by contributors labels Aug 2, 2021
@MelvinBot
Copy link

Triggered auto assignment to @iwiznia (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@MelvinBot MelvinBot removed the Overdue label Aug 2, 2021
@Christinadobrzyn
Copy link
Contributor

Christinadobrzyn commented Aug 2, 2021

Created job in Upwork

Internal: https://www.upwork.com/ab/applicants/1422005847785369600/job-details
External: https://www.upwork.com/jobs/~01030e3a8abdb33839

Invited @rdjuric to the job in upwork!

@iwiznia
Copy link
Contributor

iwiznia commented Aug 2, 2021

Proposal from @rdjuric looks good @Christinadobrzyn

@Christinadobrzyn
Copy link
Contributor

Hired @rdjuric! 🎉

@Christinadobrzyn Christinadobrzyn removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Aug 5, 2021
@Christinadobrzyn Christinadobrzyn added Weekly KSv2 and removed Daily KSv2 labels Aug 5, 2021
@Christinadobrzyn Christinadobrzyn changed the title iPad - "Manage cards" text is not centered in the button [Hold for payment August 10th] iPad - "Manage cards" text is not centered in the button Aug 6, 2021
@Christinadobrzyn Christinadobrzyn changed the title [Hold for payment August 10th] iPad - "Manage cards" text is not centered in the button [Hold for payment August 11th] iPad - "Manage cards" text is not centered in the button Aug 11, 2021
@Christinadobrzyn
Copy link
Contributor

Paid @rdjuric in Upwork and closed job!

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

6 participants