Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Home overflow menus don't inflate up when towards bottom of the screen #986

Closed
sv-ohorvath opened this issue Mar 13, 2019 · 12 comments
Closed
Assignees
Labels
b:landscape Landscape bug 🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified Feature:HomeScreen needs:ac Needs Android Component Work Release Blocker Blocks a Release

Comments

@sv-ohorvath
Copy link
Contributor

sv-ohorvath commented Mar 13, 2019

Steps to reproduce

  1. Turn the device in landscape mode.
  2. Open a session's overflow menu.

Expected behavior

The menu is displayed above the fold.

Actual behavior

The menu is displayed under the fold, you have to pull it up to see it. Scrolling the page down doesn't work.
Turning the device to portrait mode, the session title will disappear.
20190313_112142

Device information

  • Android device: Pixel (Android 9), Nexus 5(Android 6)
  • Fenix version: master build 3/13.

┆Issue is synchronized with this Jira Task

@sv-ohorvath sv-ohorvath added the 🐞 bug Crashes, Something isn't working, .. label Mar 13, 2019
@jatinkumarg
Copy link
Contributor

Hey, just want to add more info about this bug. For me, on my device, it also removes the Archive button whenever I switch from portrait to landscape or vice versa

Potrait
potrait

Landscape
landscape

boek added a commit to boek/fenix that referenced this issue Mar 19, 2019
boek added a commit to boek/fenix that referenced this issue Mar 19, 2019
boek added a commit to boek/fenix that referenced this issue Mar 22, 2019
boek added a commit to boek/fenix that referenced this issue Mar 22, 2019
@vesta0 vesta0 added Feature:Sessions P1 Current sprint labels Apr 1, 2019
@vesta0 vesta0 added this to the MVP Bugs milestone Apr 1, 2019
@vesta0 vesta0 added the b:landscape Landscape bug label Apr 30, 2019
@topotropic
Copy link

We got rid of this menu view with introduction of the new Collections features but if you tap on the overflow menu now it still renders below the fold.

@vesta0 vesta0 modified the milestones: Bugs, Sprint 5-2 (Milestone 5) May 8, 2019
@vesta0 vesta0 added this to High priority backlog (not in the current sprint) in Fenix Sprint Kanban May 8, 2019
@sblatz
Copy link
Contributor

sblatz commented May 15, 2019

@topotropic can you clarify what you mean?

I'm not able to reproduce this bug.

@sblatz sblatz self-assigned this May 15, 2019
@topotropic
Copy link

See screenshot but I guess that's a rare case
Screenshot (May 16, 2019 10:35:18 AM)

@bifleming bifleming added the Release Blocker Blocks a Release label May 17, 2019
@vesta0 vesta0 modified the milestones: Sprint 5-2 (Milestone 5), Bugs May 22, 2019
@bifleming bifleming moved this from High priority backlog (not in the current sprint) to Ready for Dev in Fenix Sprint Kanban May 28, 2019
@sblatz sblatz removed their assignment May 28, 2019
@bifleming bifleming removed the P1 Current sprint label May 30, 2019
@sblatz sblatz changed the title Sessions overflow menu is not visible in landscape Home overflow menus don't inflate up when towards bottom of the screen May 31, 2019
@sblatz
Copy link
Contributor

sblatz commented May 31, 2019

This is actually blocked on mozilla-mobile/android-components#2916.

@vesta0, @bifleming can we get that a-c ticket properly prioritized?

@sblatz sblatz added the needs:ac Needs Android Component Work label May 31, 2019
@Amejia481
Copy link
Contributor

@sblatz We could detect when the phone is in landscape and use if so pass an up orientation. Could this be an alternative?
orientation = BrowserMenu.Orientation.UP

Screenshot_20190531-170302. Could this be an alternative?

@ekager ekager self-assigned this Jun 5, 2019
ekager added a commit to ekager/fenix that referenced this issue Jun 5, 2019
@bifleming bifleming added this to Blocked by A/C in Blocking Fenix Jun 7, 2019
@bifleming bifleming removed the 🙅 waiting Issues that are blocked or has dependencies that are not ready label Jun 10, 2019
@project-bot project-bot bot moved this from Blocked to In Progress in Fenix Sprint Kanban Jun 10, 2019
@sblatz sblatz added eng:qa:needed QA Needed and removed in progress labels Jun 11, 2019
@project-bot project-bot bot moved this from In Progress to Ready for QA in Fenix Sprint Kanban Jun 11, 2019
@lobontiumira
Copy link

This is not fixed yet. The menu is not entirely visible, it is scrollable though - please see the attached screenshot.
Tested with the latest build 1,0,1924 (#11630619), with OnePlus 5T (Android 9), Motorola Nexus 6 (Android 7,1,1).

2019_06_12_15_49_37

@lobontiumira lobontiumira added in progress and removed eng:qa:needed QA Needed labels Jun 12, 2019
@project-bot project-bot bot moved this from Ready for QA to In Progress in Fenix Sprint Kanban Jun 12, 2019
@Amejia481
Copy link
Contributor

I think this is a different issue from the original one. This only happens after the menu is opened on portrait mode, and while is open the phone is switched to landscape mode.

@lobontiumira
Copy link

Arturo, the same behavior is when opening that menu directly in landscape mode.

@sblatz
Copy link
Contributor

sblatz commented Jun 12, 2019

I agree with @Amejia481 this seems like a separate issue. When I open the menu in landscape, it properly inflates upwards:
demo

@softvision-miralobontiu are you not seeing the menu inflate upwards in landscape?

I can file the rotation issue as a separate ticket :)

@sblatz
Copy link
Contributor

sblatz commented Jun 12, 2019

Follow up issue filed at #3390

@lobontiumira
Copy link

@sblatz in landscape the menu inflates, as you've mentioned.

I will close this as fixed.

@lobontiumira lobontiumira added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Jun 13, 2019
@lobontiumira lobontiumira moved this from Ready for QA to Done in Fenix Sprint Kanban Jun 13, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
b:landscape Landscape bug 🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified Feature:HomeScreen needs:ac Needs Android Component Work Release Blocker Blocks a Release
Projects
No open projects
Blocking Fenix
Blocked by A/C
Development

No branches or pull requests

10 participants