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

[Workspace feeds] Card art on card details page looks a bit smaller than the mockups #51910

Open
1 of 8 tasks
m-natarajan opened this issue Nov 1, 2024 · 16 comments
Open
1 of 8 tasks
Assignees
Labels
AutoAssignerNewDotQuality Used to assign quality issues to engineers Bug Something is broken. Auto assigns a BugZero manager. Design Needs Reproduction Reproducible steps needed Reviewing Has a PR in review Weekly KSv2

Comments

@m-natarajan
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!


Version Number:
Reproducible in staging?: needs reproduction
Reproducible in production?: needs reproduction
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
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: @joekaufmanexpensify
Slack conversation (hyperlinked to channel name): quality

Action Performed:

Prerequisite:
Members and company cards added in workspace , 8+ export options for card

  1. Go to staging.new.expensify.com
  2. Choose the workspace from settings
  3. Click company cards >Card Details

Expected Result:

Card art on the card details should match the mocks

Actual Result:

Card art on card details page looks a good bit smaller than the mockups, and compared to analogous pages for Expensify Card.

Workaround:

unknown

Platforms:

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

  • Android: Standalone
  • Android: HybridApp
  • Android: mWeb Chrome
  • iOS: Standalone
  • iOS: HybridApp
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence ![image - 2024-10-31T152943 856](https://github.com/user-attachments/assets/8ed3e674-fddd-47dc-b4c2-3fe297242e22) ![image - 2024-10-31T152932 168](https://github.com/user-attachments/assets/32220c5a-aac5-41dd-89e9-f7f32d8c18d9)

View all open jobs on GitHub

@m-natarajan m-natarajan added Needs Reproduction Reproducible steps needed Bug Something is broken. Auto assigns a BugZero manager. AutoAssignerNewDotQuality Used to assign quality issues to engineers labels Nov 1, 2024
Copy link

melvin-bot bot commented Nov 1, 2024

Triggered auto assignment to @Julesssss (AutoAssignerNewDotQuality)

Copy link

melvin-bot bot commented Nov 1, 2024

Triggered auto assignment to @puneetlath (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@melvin-bot melvin-bot bot added Weekly KSv2 Daily KSv2 and removed Weekly KSv2 labels Nov 1, 2024
@m-natarajan
Copy link
Author

BZ team member, Callstack will work on this issue, please wait for them to comment before moving the issue forward.

@m-natarajan m-natarajan added Weekly KSv2 and removed Daily KSv2 labels Nov 1, 2024
@MelvinBot
Copy link

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

@mountiny mountiny changed the title Card art on card details page looks a bit smaller than the mockups [Workspace feeds] Card art on card details page looks a bit smaller than the mockups Nov 1, 2024
@mountiny
Copy link
Contributor

mountiny commented Nov 1, 2024

Handling this as part of the project I have worked on

@waterim
Copy link
Contributor

waterim commented Nov 4, 2024

Hello, Im Artem from Callstack and would like to help with this issue!

@puneetlath puneetlath removed their assignment Nov 4, 2024
@mountiny
Copy link
Contributor

mountiny commented Nov 4, 2024

assigned, thanks

@waterim
Copy link
Contributor

waterim commented Nov 5, 2024

@mountiny Here is an issue with exported SVGs we have
It has sizes defined in svg itself for icons(for selections list, list of cards etc.)
and all of the cards have size: viewBox="0 0 40 40"
While expensify card has viewBox="0 0 235 148"

And I cant just change the size with ignoring aspect ratio because svg is not optimized for this scale and looks like this:
image

I think we need to devide icons company cards and full big card images with another aspect ratio

Some designer is needed here for exporting all correct card images

@waterim
Copy link
Contributor

waterim commented Nov 5, 2024

I need: Amex, BOA, CapitalOne, Chase, Citibank, WellsFargo, BrexCompany, Stripe, Other

@mountiny mountiny added the Design label Nov 6, 2024
Copy link

melvin-bot bot commented Nov 6, 2024

Triggered auto assignment to @dannymcclain (Design), see these Stack Overflow questions for more details.

@mountiny
Copy link
Contributor

mountiny commented Nov 6, 2024

👋 any chance you could help with the request above? @dannymcclain thanks!

@dannymcclain
Copy link
Contributor

Absolutely. I'll upload the updated card SVGs here once I get a chance to make them! We can't be showing squish squashed cards in the app 😂

@mountiny
Copy link
Contributor

mountiny commented Nov 8, 2024

@dannymcclain friendly bump

@dannymcclain
Copy link
Contributor

Getting to this today!

@dannymcclain
Copy link
Contributor

Alright let's try these:

Card Images - Large.zip

@allgandalf
Copy link
Contributor

@waterim do you think you can get the PR up for review today ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AutoAssignerNewDotQuality Used to assign quality issues to engineers Bug Something is broken. Auto assigns a BugZero manager. Design Needs Reproduction Reproducible steps needed Reviewing Has a PR in review Weekly KSv2
Projects
Status: No status
Development

No branches or pull requests

8 participants