Skip to content

[$500] WS avatar does not have any indent from the bottom edge of the header #36835

Closed

Description

@kavimuru

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: 1.4.43-0
Reproducible in staging?: y
Reproducible in production?: n
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: applause internal team
Slack conversation:

Action Performed:

  1. Launch App
  2. Navigate to any WS
  3. Tap settings > Profile

Expected Result:

WS avatar should have an indent from the bottom edge of the header

Actual Result:

WS avatar does not have any indent from the bottom edge of the header.

Workaround:

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

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

Add any screenshot/video evidence
Bug6385261_1708398822677!ws

Bug6385261_1708398822692.Screen_Recording_2024-02-19_at_9.12.41_PM.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01c86444f0c4b200d4
  • Upwork Job ID: 1759797554514292736
  • Last Price Increase: 2024-02-20

Activity

  1. added
    DeployBlockerCashThis issue or pull request should block deployment
    ExternalAdded to denote the issue can be worked on by a contributor
    BugSomething is broken. Auto assigns a BugZero manager.
    on Feb 20, 2024
  2. changed the title [-]WS avatar does not have any indent from the bottom edge of the header[/-] [+][$500] WS avatar does not have any indent from the bottom edge of the header[/+] on Feb 20, 2024
  3. melvin-bot commented on Feb 20, 2024

    @melvin-bot
  4. added
    Help WantedApply this label when an issue is open to proposals by contributors
    on Feb 20, 2024
  5. melvin-bot commented on Feb 20, 2024

    @melvin-bot

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

  6. melvin-bot commented on Feb 20, 2024

    @melvin-bot

    Triggered auto assignment to @puneetlath (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

  7. github-actions commented on Feb 20, 2024

    @github-actions
    Contributor

    👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

    1. Identify the pull request that introduced this issue and revert it.
    2. Find someone who can quickly fix the issue.
    3. Fix the issue yourself.
  8. 4 remaining items

  9. situchan commented on Feb 20, 2024

    @situchan
    Contributor
  10. narefyev91 commented on Feb 20, 2024

    @narefyev91
    Contributor

    i will take care of it

  11. narefyev91 commented on Feb 20, 2024

    @narefyev91
    Contributor
  12. removed their assignment
    on Feb 20, 2024
  13. tienifr commented on Feb 21, 2024

    @tienifr
    Contributor
  14. puneetlath commented on Feb 21, 2024

    @puneetlath
    Contributor

    Interesting, fair question, but this doesn't feel the same as the linked case. In this case it doesn't feel like your exact proposal was used. It was a simple fix that the original contributor raised on their own. And you didn't find the offending PR, so I don't think any compensation is due here.

  15. puneetlath commented on Feb 21, 2024

    @puneetlath
    Contributor

    Closing since the PR was raised by the original C/C+ and CP'd to staging. Thanks for the quick fix!

  16. shawnborton commented on Feb 26, 2024

    @shawnborton
    Contributor

    So we actually don't want any space below the avatar and the background image:
    CleanShot 2024-02-26 at 16 27 29@2x

    The problem is that in light mode, we aren't using the right background image.

  17. situchan commented on Feb 26, 2024

    @situchan
    Contributor

    There's bug in dark mode as well. Background image is cut, not stretched gradiently to the bottom

    Screenshot 2024-02-27 at 3 37 07 AM
  18. shawnborton commented on Feb 26, 2024

    @shawnborton
    Contributor

    Yeah, I think we need a follow up here with new images for the header area for both light and dark mode. And we need to revert this change. I also outlined a few more issues:

    • The workspace name is not using the right font
    • The currency row can be tapped even in read-only mode
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

BugSomething is broken. Auto assigns a BugZero manager.EngineeringExternalAdded to denote the issue can be worked on by a contributorHelp WantedApply this label when an issue is open to proposals by contributorsReviewingHas a PR in reviewWeeklyKSv2

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions