-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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 - #11768] [$500] Unable to open chat through search when the browser/app size is decreased - reported by @adeel0202 #9734
Comments
Triggered auto assignment to @strepanier03 ( |
Using Chrome browser and trying multiple different sizes I'm unable to recreate this behavior. Is this specific to a certain browser type @mvtglobally? |
@strepanier03, it is reproducible in chrome and safari both (I couldn't check on any other browser). It is also reproducible in the desktop app. |
Hmm, I'll send it on but I can't repro myself in Chrome or Firefox. |
Triggered auto assignment to @Gonals ( |
Triggered auto assignment to @trjExpensify ( |
Good candidate for external! |
Ah yeah, I can repro this. I misread it initially, but yes.. accessing a chat from the |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @Santhosh-Sellavel ( |
I caught up with Marc earlier and I'm going to pop the issue on |
This is held, Melv. |
on hold |
^^ 👋 @Luke9389, has this come off hold or something? |
OH! huh. Weeeird. I coulda sworn someone commented that this was off hold. My bad! |
Ha, all good! |
Updated the title to be more specific that this is held on the |
snagging this as I'm on the tracking issue. |
See the tracking issue for the latest. |
Still holding. |
Still held as this initiative goes through the design process. |
Still holding on the larger react navigation reboot. |
Is this really a case that we should be supporting? I think we only care about "normally" sized browser windows and mobile web clients. My guess is that at these widths, there is quite a bit more broken than just this navigation issue, and so supporting this browser platform and window size should be regarded as a new feature and not a bug. |
That's a really great point. I think you're right – to the extent that this also fails the "supported platform" stipulation for bugs as well. I'm actually going to close as a result but anyone from Expensify on this issue can re-open if they feel strongly. |
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:
Expected Result:
When the screen width on Web/desktop has been decreased to the size of mobile/tablet, it should work in exactly the same way as those devices.
<
in the top left of the chat to return to the LHNActual Result:
The LHN is in view and not the chat
Workaround:
unknown
Platform:
Where is this issue occurring?
Version Number: 1.1.79-11
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: Any additional supporting documentation
Screen.Recording.2022-06-16.at.9.41.42.PM.mov
Upwork URL: https://www.upwork.com/jobs/~010e7681ab13edf3b1
Issue reported by: @adeel0202
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1655398548680369
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: