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

New tax rate empty state has incorrect label sizes #40813

Open
6 tasks
luacmartins opened this issue Apr 23, 2024 · 21 comments
Open
6 tasks

New tax rate empty state has incorrect label sizes #40813

luacmartins opened this issue Apr 23, 2024 · 21 comments
Assignees
Labels
Daily KSv2 Internal Requires API changes or must be handled by Expensify staff

Comments

@luacmartins
Copy link
Contributor

luacmartins commented Apr 23, 2024

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:
Reproducible in staging?:
Reproducible in production?:
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:
Slack conversation:

Action Performed:

  1. Navigate to Workspace > Taxes > Add rate
  2. Verify that the input labels have incorrect font size of 13px

Expected Result:

Input labels should have font size of 15px

Actual Result:

Input labels should have font size of 13px

Workaround:

N/A

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

IMG_9563

Add any screenshot/video evidence

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~0165be538517a8973a
  • Upwork Job ID: 1782835092067209216
  • Last Price Increase: 2024-04-23
Issue OwnerCurrent Issue Owner: @shawnborton
@MahmudjonToraqulov

This comment was marked as off-topic.

@luacmartins luacmartins added the Internal Requires API changes or must be handled by Expensify staff label Apr 23, 2024
Copy link

melvin-bot bot commented Apr 23, 2024

Job added to Upwork: https://www.upwork.com/jobs/~0165be538517a8973a

Copy link

melvin-bot bot commented Apr 23, 2024

Triggered auto assignment to Contributor Plus for review of internal employee PR - @rushatgabhane (Internal)

@luacmartins
Copy link
Contributor Author

Thanks for the proposal @MahmudjonToraqulov, but we're handling this issue internally!

@MahmudjonToraqulov
Copy link
Contributor

Ok, sorry.

@melvin-bot melvin-bot bot added the Overdue label Apr 25, 2024
@luacmartins
Copy link
Contributor Author

luacmartins commented Apr 25, 2024

@shawnborton did you get a chance to look into this one yet?

@melvin-bot melvin-bot bot removed the Overdue label Apr 25, 2024
@shawnborton
Copy link
Contributor

Not yet!

@melvin-bot melvin-bot bot added the Overdue label Apr 29, 2024
@shawnborton
Copy link
Contributor

Not overdue, hoping to look at it this week!

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Apr 29, 2024
@luacmartins
Copy link
Contributor Author

Shawn will be working on this one!

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels May 1, 2024
@shawnborton
Copy link
Contributor

Sorry! Still on my plate, just haven't looked into it yet

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels May 6, 2024
Copy link

melvin-bot bot commented May 9, 2024

@shawnborton, @luacmartins, @rushatgabhane Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@shawnborton
Copy link
Contributor

Still on my plate

@melvin-bot melvin-bot bot removed the Overdue label May 13, 2024
@shawnborton
Copy link
Contributor

@luacmartins I think I'm going to need some help with this one!

It looks like we have a component called MenuItemWithTopDescription which seems to have the correct label behavior. But for tax rates, it's using InputWrapper which has some different styles. Anyways, let's plan to look at this during our next mentor sync.

@luacmartins
Copy link
Contributor Author

@shawnborton you're close! If you look into InputWrapper, we pass an InputComponent prop to it, which is ultimately the input component we render. For the Name field, we are passing TextPicker. Perhaps we can take a look at the styles TextPicker is using?

@melvin-bot melvin-bot bot added the Overdue label May 16, 2024
Copy link

melvin-bot bot commented May 17, 2024

@shawnborton, @luacmartins, @rushatgabhane Whoops! This issue is 2 days overdue. Let's get this updated quick!

@shawnborton
Copy link
Contributor

Not overdue, still on my plate :)

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels May 20, 2024
Copy link

melvin-bot bot commented May 23, 2024

@shawnborton, @luacmartins, @rushatgabhane Whoops! This issue is 2 days overdue. Let's get this updated quick!

@shawnborton
Copy link
Contributor

No update

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels May 27, 2024
@shawnborton
Copy link
Contributor

Maybe today is the day

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels May 30, 2024
Copy link

melvin-bot bot commented Jun 4, 2024

@shawnborton, @luacmartins, @rushatgabhane Huh... This is 4 days overdue. Who can take care of this?

@shawnborton
Copy link
Contributor

Not overdue, still on my plate

@melvin-bot melvin-bot bot removed the Overdue label Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Daily KSv2 Internal Requires API changes or must be handled by Expensify staff
Projects
Status: Polish
Development

No branches or pull requests

4 participants