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

[$100] Increase size of checkbox to default size of 20px #48839

Closed
blimpich opened this issue Sep 9, 2024 · 17 comments
Closed

[$100] Increase size of checkbox to default size of 20px #48839

blimpich opened this issue Sep 9, 2024 · 17 comments
Assignees
Labels
Engineering External Added to denote the issue can be worked on by a contributor NewFeature Something to build that is a new item. Reviewing Has a PR in review Weekly KSv2

Comments

@blimpich
Copy link
Contributor

blimpich commented Sep 9, 2024

Problem

Coming from this thread, we currently have checkboxes that are at a reduced size (16px) for tasks inside DMs. We recently added avatars to our checkboxes and realized that we would actually prefer for those checkboxes to be our standard size.

Solution

Lets increase the size of the checkboxes to our standard size, which is 20px.

cc: @Expensify/design

Example (16px on left, 20px on the right)

comparison

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021833257022819149664
  • Upwork Job ID: 1833257022819149664
  • Last Price Increase: 2024-09-09
  • Automatic offers:
    • DylanDylann | Reviewer | 103898533
    • ChavdaSachin | Contributor | 103898534
Issue OwnerCurrent Issue Owner: @DylanDylann
@blimpich blimpich added the External Added to denote the issue can be worked on by a contributor label Sep 9, 2024
Copy link

melvin-bot bot commented Sep 9, 2024

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

@melvin-bot melvin-bot bot changed the title Increase size of checkbox to default size of 20px [$250] Increase size of checkbox to default size of 20px Sep 9, 2024
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Sep 9, 2024
Copy link

melvin-bot bot commented Sep 9, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @DylanDylann (External)

@blimpich blimpich changed the title [$250] Increase size of checkbox to default size of 20px [$100] Increase size of checkbox to default size of 20px Sep 9, 2024
Copy link

melvin-bot bot commented Sep 9, 2024

Upwork job price has been updated to $100

@blimpich
Copy link
Contributor Author

blimpich commented Sep 9, 2024

Decreased job price since this is probably a simple and small css change. Will change if that ends up not being the case.

@blimpich blimpich added the NewFeature Something to build that is a new item. label Sep 9, 2024
Copy link

melvin-bot bot commented Sep 9, 2024

Triggered auto assignment to @puneetlath (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Sep 9, 2024
Copy link

melvin-bot bot commented Sep 9, 2024

⚠️ It looks like this issue is labelled as a New Feature but not tied to any GitHub Project. Keep in mind that all new features should be tied to GitHub Projects in order to properly track external CAP software time ⚠️

Copy link

melvin-bot bot commented Sep 9, 2024

Current assignee @dannymcclain is eligible for the NewFeature assigner, not assigning anyone new.

@blimpich blimpich added Daily KSv2 and removed Weekly KSv2 labels Sep 9, 2024
@Nodebrute
Copy link
Contributor

Nodebrute commented Sep 9, 2024

Proposal

Please re-state the problem that we are trying to solve in this issue.

Increase size of checkbox to default size of 20px

What is the root cause of that problem?

We are using 16 here

App/src/styles/index.ts

Lines 4113 to 4114 in 4d250ac

height: 16,
width: 16,

What changes do you think we should make in order to solve the problem?

We should change both value to 20

App/src/styles/index.ts

Lines 4113 to 4114 in 4d250ac

height: 16,
width: 16,

    taskCheckbox: {
            height: 20,
            width: 20,
        },

What alternative solutions did you explore? (Optional)

@ChavdaSachin
Copy link
Contributor

Proposal

Please re-state the problem that we are trying to solve in this issue.

  • Change Task checkbox size to default.

What is the root cause of that problem?

  • A custom style for task checkbox size is included here

What changes do you think we should make in order to solve the problem?

Now checkbox has default size which is 20px

Result

Screenshot 2024-09-10 at 3 37 29 AM
### What alternative solutions did you explore? (Optional)

@DylanDylann
Copy link
Contributor

@ChavdaSachin's proposal looks good to me

🎀 👀 🎀 C+ reviewed

Copy link

melvin-bot bot commented Sep 10, 2024

Current assignees @puneetlath and @blimpich are eligible for the choreEngineerContributorManagement assigner, not assigning anyone new.

@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Sep 10, 2024
Copy link

melvin-bot bot commented Sep 10, 2024

📣 @DylanDylann 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job

Copy link

melvin-bot bot commented Sep 10, 2024

📣 @ChavdaSachin 🎉 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 📖

@ChavdaSachin
Copy link
Contributor

Thanks for accepting my Proposal.
I will raise PR within an Hour.

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Sep 10, 2024
@ChavdaSachin
Copy link
Contributor

PR is ready for review.

cc. @DylanDylann

@ChavdaSachin
Copy link
Contributor

Payment is due today.
cc.@puneetlath @blimpich

@puneetlath
Copy link
Contributor

Right you are, thanks for the bump. All paid. Thanks everyone!

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 NewFeature Something to build that is a new item. Reviewing Has a PR in review Weekly KSv2
Projects
None yet
Development

No branches or pull requests

6 participants