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

[Bug] Menu not reversed if the toolbar position is changed to top (opened from Fenix homescreen) #9365

Closed
sflorean opened this issue Mar 24, 2020 · 5 comments
Assignees
Labels
🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified eng:ready Ready for engineering Feature:Toolbar Address bar, see also Feature:Search S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist

Comments

@sflorean
Copy link
Contributor

sflorean commented Mar 24, 2020

Steps to reproduce

  1. Open Fenix and go to Settings-> Customize - Toolbar.
  2. Set to "Top".
  3. Go to the home screen.
  4. Tap on the three-dot menu.

Expected behavior

The menu is reversed.

Actual behavior

The menu is displayed the same as for toolbar position to bottom.

Device information

  • Android device: Samsung Galaxy Note10 (Android 9.0), Xiaomi Mi Pad 2 (Android 5.1).
  • Fenix version: Beta v4.2.0-beta.1, Nightly 3/24.

top

┆Issue is synchronized with this Jira Task

@sflorean sflorean added 🐞 bug Crashes, Something isn't working, .. S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist labels Mar 24, 2020
@github-actions github-actions bot added the needs:triage Issue needs triage label Mar 24, 2020
@sflorean sflorean added the Feature:Toolbar Address bar, see also Feature:Search label Mar 24, 2020
@mcarare mcarare added the needs:UX-feedback Needs UX Feedback label Mar 26, 2020
@mcarare
Copy link
Contributor

mcarare commented Mar 26, 2020

For UX: How should the reverse menu look, taking into account the "Library" section?

@brampitoyo brampitoyo self-assigned this Apr 1, 2020
@brampitoyo
Copy link

@mcarare When toolbar is set to top, the main menu items should be ordered in reverse, like this:

Main menu on homescreen (top to bottom)

Help
Settings

--

Library
Bookmarks
History

--

What’s new

Main menu on sites (top to bottom)

Bookmark | Share | Forward | Refresh

--

Reader view
Desktop site
Save to collection
Add to home screen
Add to top sites
Find in page
Report site issue

--

Settings
Add-ons
Library

--

(Any add-on you have installed goes here)

@brampitoyo brampitoyo added eng:ready Ready for engineering and removed needs:UX-feedback Needs UX Feedback labels Apr 1, 2020
@mcarare mcarare self-assigned this Apr 1, 2020
mcarare added a commit to mcarare/fenix that referenced this issue Apr 1, 2020
@mcarare
Copy link
Contributor

mcarare commented Apr 1, 2020

@brampitoyo , I suppose also, "Quit", if existing, will be on top.

mcarare added a commit to mcarare/fenix that referenced this issue Apr 1, 2020
mcarare added a commit to mcarare/fenix that referenced this issue Apr 1, 2020
mcarare added a commit to mcarare/fenix that referenced this issue Apr 1, 2020
mcarare added a commit to mcarare/fenix that referenced this issue Apr 1, 2020
mcarare added a commit to mcarare/fenix that referenced this issue Apr 1, 2020
@brampitoyo
Copy link

@mcarare I could be wrong, but I thought there’s no longer a “Quit” list item in the main menu?

If exists, “Quit” should be the last item on the list. Meaning: if the toolbar is on bottom, then “Quit” should sit on the very top of the list. If the toolbar is on top, then “Quit” should sit on the very bottom.

@mcarare mcarare added the eng:qa:needed QA Needed label Apr 3, 2020
@AndiAJ
Copy link
Collaborator

AndiAJ commented Apr 6, 2020

Hi, verified as fixed on the latest Nightly Build #20970607 from 4/6 using a Google Pixel 3a (Android 10)

► Screenshot Home screen
Screenshot_20200406-101655

► Screenshot Page view
Screenshot_20200406-101546

@AndiAJ AndiAJ closed this as completed Apr 6, 2020
@AndiAJ AndiAJ added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Apr 6, 2020
@liuche liuche mentioned this issue Apr 13, 2020
32 tasks
@ekager ekager removed the needs:triage Issue needs triage label Apr 17, 2020
@liuche liuche mentioned this issue Apr 28, 2020
32 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🐞 bug Crashes, Something isn't working, .. eng:qa:verified QA Verified eng:ready Ready for engineering Feature:Toolbar Address bar, see also Feature:Search S2 Major Functionality/product severely impaired and a satisfactory workaround doesn't exist
Projects
None yet
Development

No branches or pull requests

5 participants