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

[Hold - Browser Resizing #13971] The report menu popup is not responsive when window resized #13912

Closed
1 task
kavimuru opened this issue Dec 30, 2022 · 25 comments
Closed
1 task
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Internal Requires API changes or must be handled by Expensify staff Monthly KSv2

Comments

@kavimuru
Copy link

kavimuru commented Dec 30, 2022

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Action Performed:

  1. Open the app, login with any account
  2. Try to open any report, click on plus button to open the action menu, after that resize the window
  3. Notice that the popup is not responsive when the window resized.

Expected Result:

The popup should be responsive when the window resized.

Actual Result :

The popup is not responsive when the window resized.

Workaround:

unknown

Platforms:

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

  • MacOS / Chrome / Safari

Version Number: 1.2.46-0
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:

Recording.1184.mp4
Screen.Recording.2022-12-30.at.17.35.22.1.mov

Expensify/Expensify Issue URL:
Issue reported by: @hungvu193
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1672396555740609

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01bbc9eb6361235e84
  • Upwork Job ID: 1609936247293853696
  • Last Price Increase: 2023-01-02
@kavimuru kavimuru added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Dec 30, 2022
@melvin-bot melvin-bot bot locked and limited conversation to collaborators Dec 30, 2022
@sketchydroide
Copy link
Contributor

it's not irresponsive, it's just focused on the menu.
Still a problem and a bug though

@melvin-bot melvin-bot bot added the Overdue label Jan 2, 2023
@sketchydroide sketchydroide self-assigned this Jan 2, 2023
@sketchydroide sketchydroide added the Internal Requires API changes or must be handled by Expensify staff label Jan 2, 2023
@sketchydroide
Copy link
Contributor

Was able to reproduce, so I think when we reduce the size is dropping the other view correctly, but needs to close the menu as well.
Screenshot 2023-01-02 at 15 34 56

@melvin-bot melvin-bot bot removed the Overdue label Jan 2, 2023
@sketchydroide
Copy link
Contributor

will work on probably tomorrow

@trjExpensify trjExpensify changed the title The report menu popup is not responsive when window resized [Hold - Browser Resizing #13971] The report menu popup is not responsive when window resized Jan 4, 2023
@trjExpensify
Copy link
Contributor

@sketchydroide putting this thread on your radar and I've created this parent issue. We should tackle all of these browser resizing issues holistically and need someone to own that plan.

@melvin-bot melvin-bot bot added the Overdue label Jan 5, 2023
@melvin-bot melvin-bot bot unlocked this conversation Jan 5, 2023
@sketchydroide
Copy link
Contributor

ok, anythink I need to do here, definetly interested in helping with it

@melvin-bot melvin-bot bot removed the Overdue label Jan 6, 2023
@trjExpensify
Copy link
Contributor

I don't think so on this issue in isolation, as it should be tackled with a wider browser resizing initiative.

@melvin-bot melvin-bot bot added the Overdue label Jan 9, 2023
@sketchydroide
Copy link
Contributor

should we hold this @trjExpensify?

@melvin-bot melvin-bot bot removed the Overdue label Jan 9, 2023
@trjExpensify
Copy link
Contributor

Yeah, I already put hold in the title and point to the master. I would drop it to Monthly for further movement on that wider initiative.

@sketchydroide sketchydroide added Monthly KSv2 and removed Daily KSv2 labels Jan 9, 2023
@melvin-bot melvin-bot bot added the Overdue label Feb 9, 2023
@sketchydroide
Copy link
Contributor

Still on hold I think? @trjExpensify?

@melvin-bot melvin-bot bot removed the Overdue label Feb 10, 2023
@trjExpensify
Copy link
Contributor

Yupp!

@melvin-bot melvin-bot bot added the Overdue label Mar 13, 2023
@mateocole mateocole added the Monthly KSv2 label Mar 31, 2023
@melvin-bot melvin-bot bot added the Overdue label May 2, 2023
@jliexpensify
Copy link
Contributor

is this still on hold @trjExpensify ? Thanks!

@melvin-bot melvin-bot bot removed the Overdue label May 17, 2023
@trjExpensify
Copy link
Contributor

Yup, the tracker is linked in the OP.

@jliexpensify
Copy link
Contributor

Might unassign Andre? He's gone now.

@jliexpensify
Copy link
Contributor

Still on hold

@melvin-bot melvin-bot bot removed the Overdue label Jun 24, 2023
@melvin-bot melvin-bot bot added the Overdue label Jul 25, 2023
@jliexpensify
Copy link
Contributor

On Hold

@melvin-bot melvin-bot bot removed the Overdue label Jul 26, 2023
@melvin-bot melvin-bot bot added the Overdue label Aug 28, 2023
@jliexpensify
Copy link
Contributor

On hold

@melvin-bot melvin-bot bot removed the Overdue label Aug 29, 2023
@melvin-bot melvin-bot bot added the Overdue label Sep 29, 2023
@jliexpensify
Copy link
Contributor

On hold

@melvin-bot melvin-bot bot removed the Overdue label Oct 3, 2023
@melvin-bot melvin-bot bot added the Overdue label Nov 3, 2023
@jliexpensify
Copy link
Contributor

Still on hold

@melvin-bot melvin-bot bot removed the Overdue label Nov 6, 2023
@melvin-bot melvin-bot bot added the Overdue label Dec 7, 2023
@jliexpensify
Copy link
Contributor

Still on hold

@melvin-bot melvin-bot bot removed the Overdue label Dec 8, 2023
@melvin-bot melvin-bot bot added the Overdue label Jan 8, 2024
@jliexpensify
Copy link
Contributor

Still on hold

@melvin-bot melvin-bot bot removed the Overdue label Jan 15, 2024
@JmillsExpensify
Copy link

We don't plan on prioritizing browser resizing anytime soon. We're more focused on migrating our existing customers from our old platform to our new platform, which will minimally take the remainder of this year. We can open up this issue in the future if/when this becomes a priority.

@JmillsExpensify JmillsExpensify closed this as not planned Won't fix, can't repro, duplicate, stale Jan 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Internal Requires API changes or must be handled by Expensify staff Monthly KSv2
Projects
None yet
Development

No branches or pull requests

7 participants