App icon indicating copy to clipboard operation
App copied to clipboard

[$250] BUG: Mac Safari - Sidebar animation is different reported by @mdneyazahmad

Open kavimuru opened this issue 2 years ago • 21 comments

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Action Performed:

  1. Goto staging https://staging.new.expensify.com/
  2. Click on fab
  3. Click request money
  4. Enter the amount
  5. Click next, and observe the animation
  6. Select the contact, and observe the animation

Expected Result:

Animation should be same across all sidebars

Actual Result:

Animation is not same, the sidebar does not come into view from the end of the screen like other sections

Workaround:

unknown

Platform:

Where is this issue occurring?

  • Web

Version Number: 1.2.13-3 Reproducible in staging?: y Reproducible in production?: y Email or phone of affected tester (no customers): Logs: https://stackoverflow.com/c/expensify/questions/4856 Notes/Photos/Videos:

https://user-images.githubusercontent.com/43996225/195650307-6f05428f-904f-4a0f-be8b-f85072bb6c2d.mp4

https://user-images.githubusercontent.com/43996225/195650316-a5bd57e3-6a5d-4dc3-887f-9eefae88e27e.mov

Expensify/Expensify Issue URL: Issue reported by: @mdneyazahmad Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1665552011863769

View all open jobs on GitHub

kavimuru avatar Oct 13 '22 16:10 kavimuru

Triggered auto assignment to @zanyrenney (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

melvin-bot[bot] avatar Oct 13 '22 16:10 melvin-bot[bot]

Hey @kavimuru sorry I am not clear on what the bug is here. Could you please add more detail so that I can try to reproduce. The video is not clear to me what the expected vs actual result is so if you can explain it in text so I can follow the guide to reproduce, that would be really helpful - thank you!

zanyrenney avatar Oct 13 '22 17:10 zanyrenney

@zanyrenney I am attaching a new video. The sidebar animation appears from right end of the screen in profile page, preference page. But when initiating the IOU after entering the money, hitting Next no animation and next page appears from left.

https://user-images.githubusercontent.com/43996225/196058321-c3b00b94-a113-493d-b798-da554e6ced37.mp4

kavimuru avatar Oct 16 '22 21:10 kavimuru

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

melvin-bot[bot] avatar Oct 17 '22 07:10 melvin-bot[bot]

Been OOO.

zanyrenney avatar Oct 18 '22 13:10 zanyrenney

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

melvin-bot[bot] avatar Oct 19 '22 11:10 melvin-bot[bot]

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

melvin-bot[bot] avatar Oct 19 '22 11:10 melvin-bot[bot]

Current assignee @thienlnam is eligible for the External assigner, not assigning anyone new.

melvin-bot[bot] avatar Oct 19 '22 11:10 melvin-bot[bot]

Going OOO next week, going to re-assign this so we can keep moving forward with proposals

thienlnam avatar Oct 21 '22 01:10 thienlnam

Current assignee @JmillsExpensify is eligible for the External assigner, not assigning anyone new.

melvin-bot[bot] avatar Oct 21 '22 01:10 melvin-bot[bot]

Current assignee @rushatgabhane is eligible for the External assigner, not assigning anyone new.

melvin-bot[bot] avatar Oct 21 '22 01:10 melvin-bot[bot]

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

melvin-bot[bot] avatar Oct 21 '22 01:10 melvin-bot[bot]

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

melvin-bot[bot] avatar Oct 24 '22 07:10 melvin-bot[bot]

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

melvin-bot[bot] avatar Oct 24 '22 07:10 melvin-bot[bot]

Looks like we're still waiting on proposals, right?

Luke9389 avatar Oct 25 '22 21:10 Luke9389

yepp, let's make it a weekly?

rushatgabhane avatar Oct 26 '22 03:10 rushatgabhane

I'm not sure if we should fix this. We're going to refactor this entire flow very shortly, so I would be more inclined to put this bug report on hold. At the point we refactor the flow we can see if this is still an issue and then either close (and pay for reporting), or fix. Thoughts guys?

JmillsExpensify avatar Oct 26 '22 17:10 JmillsExpensify

I'm fine with that. I also don't necessarily see this as a problem. It doesn't bother me at all that the animation goes two different ways.

Luke9389 avatar Oct 27 '22 19:10 Luke9389

This is just about consistency. Maybe, the design team can confirm?

mdneyazahmad avatar Oct 28 '22 05:10 mdneyazahmad

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

melvin-bot[bot] avatar Oct 31 '22 07:10 melvin-bot[bot]

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

melvin-bot[bot] avatar Oct 31 '22 08:10 melvin-bot[bot]

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

melvin-bot[bot] avatar Oct 31 '22 20:10 melvin-bot[bot]

hey @shawnborton,

What do you think about the fact that the animation goes two different directions here? Problem, not problem?

Luke9389 avatar Oct 31 '22 20:10 Luke9389

My take is that in the Settings pages, each modal page is animated. But in the IOU flow, just the content of the modal (everything below the title) is being animated. Is that the difference that we're talking about? It's a bit unclear from the comments.

shawnborton avatar Oct 31 '22 21:10 shawnborton

@shawnborton Great point of clarification. It's basically this.

The sidebar animation appears from right end of the screen in profile page, preference page. But when initiating the IOU after entering the money, hitting Next, there no animation and next page appears from left.

This is all super subtle.

JmillsExpensify avatar Nov 01 '22 19:11 JmillsExpensify

Got it. I mean yeah, ideally this would all be consistent and feel silky smooth (IOU animations feel janky on desktop at least).

shawnborton avatar Nov 01 '22 19:11 shawnborton

100%. Where I struggle is whether we do anything now or re-evaluate once N7 hits. Maybe we just bite the bullet and tackle this now along with all the other bugs. Hmm, yeah that sounds the most logical. Agreed?

JmillsExpensify avatar Nov 01 '22 19:11 JmillsExpensify

I can get down with that.

shawnborton avatar Nov 01 '22 20:11 shawnborton

Ok cool. Upwork job is here: https://www.upwork.com/jobs/~010cf9eb9f9b6cb3b2. We're open for proposals.

JmillsExpensify avatar Nov 01 '22 23:11 JmillsExpensify

waiting for proposals.

Luke9389 avatar Nov 04 '22 20:11 Luke9389