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

Top toolbar: when toolbar is at top, move toolbar on Home screen to top as well #6820

Closed
Cheap-Skate opened this issue Nov 26, 2019 · 8 comments
Assignees
Labels
eng:qa:verified QA Verified eng:ready Ready for engineering Feature:HomeScreen feature request 🌟 New functionality and improvements Feature:Toolbar Address bar, see also Feature:Search P2 Upcoming release

Comments

@Cheap-Skate
Copy link

Cheap-Skate commented Nov 26, 2019

If I choose Top Toolbar, it seems odd that the toolbar on the Home/Tabs screen (with the Fenix logo) is at the bottom.

Would it make more sense to have ALL toolbars (Home/Tabs screen & Search screen & page content screen) at the top?

Thanks

┆Issue is synchronized with this Jira Task

@Cheap-Skate Cheap-Skate added the feature request 🌟 New functionality and improvements label Nov 26, 2019
@sblatz sblatz added the needs:UX-feedback Needs UX Feedback label Nov 26, 2019
@sblatz
Copy link
Contributor

sblatz commented Nov 26, 2019

Agreed, I think this makes a lot of sense :)

@brampitoyo
Copy link

Agreed. When the toolbar is set to top, all toolbars should be positioned on top, too – including toolbar on the start screen/tab manager.

@brampitoyo brampitoyo added eng:ready Ready for engineering and removed needs:UX-feedback Needs UX Feedback labels Nov 27, 2019
@Cheap-Skate
Copy link
Author

Agreed. When the toolbar is set to top, all toolbars should be positioned on top, too – including toolbar on the start screen/tab manager.

So, should it also be the case that if I choose a bottom toolbar the Search toolbar should also be on the bottom. At the moment it's at the top which I find odd.

i.e. either all toolbars at the top, or all toolbars at the bottom where all = Home & Search & content view screens

(this might be a separate issue)

@brampitoyo
Copy link

@Cheap-Skate Sorry, I should’ve clarified.

When toolbar Settings = “Bottom”, our positions are as follows:

Address bar Position, if toolbar Settings == “Bottom”
On start/home screen Bottom
On “Search with” Top
On webpages Bottom

If toolbar Settings == “Top”, these were the positions I proposed:

Address bar Position, if toolbar Settings == “Top”
On start/home screen Top
On “Search with” Top
On webpages Top

You correctly noticed an inconsistency. If toolbar Settings == “Bottom”, then why is the toolbar on “Search with” screen located on top?

It’s because there’s a conflict between your thumb (closer to the bottom of the screen) and the way you read search results (top to bottom).

In this particular scenario, we had to choose between ergonomics and readability. We prioritised the latter: show you search results from top to bottom.

This means that the top-most search results are the furthest away from your thumb, yes. But it also makes search results show in an orderly fashion. The topmost one is always the first, the bottom-most one always the last.

The address bar is where the search keyword originates from, right? It makes sense that the “source” is located at the apex of the hierarchy: the very top of the screen, always above the search results.

@boek boek added this to Prioritized Feature Backlog in Fenix Sprint Kanban Dec 3, 2019
@severinrudie severinrudie self-assigned this Dec 9, 2019
@severinrudie severinrudie moved this from Prioritized Feature Backlog to In Progress in Fenix Sprint Kanban Dec 9, 2019
@severinrudie severinrudie removed their assignment Dec 9, 2019
@bifleming bifleming removed this from In Progress in Fenix Sprint Kanban Dec 13, 2019
@boek boek added Feature:HomeScreen P2 Upcoming release labels Dec 19, 2019
@vesta0 vesta0 added the Feature:Toolbar Address bar, see also Feature:Search label Jan 14, 2020
@vesta0 vesta0 added this to Polish Bugs to Triage in Feature Polish via automation Jan 14, 2020
@vesta0 vesta0 moved this from Polish Bugs to Triage to Prioritized Backlog in Feature Polish Jan 14, 2020
@darkwing
Copy link
Contributor

@sblatz How difficult will this be. Any tips you can give me?

@sblatz
Copy link
Contributor

sblatz commented Jan 17, 2020

@darkwing
In theory not crazy difficult since we have a "fake" toolbar on the home screen that doesn't need to scroll away or have any movement on the screen (it should automatically animate when we transition to Search Fragment).

You'll just want to mess with bottom_bar in fragment_home.xml and base it off of shouldUseBottomToolbar (settings property)

Let me know how else I can help 😄

@darkwing darkwing self-assigned this Jan 21, 2020
darkwing added a commit to darkwing/fenix that referenced this issue Jan 22, 2020
darkwing added a commit to darkwing/fenix that referenced this issue Jan 22, 2020
darkwing added a commit to darkwing/fenix that referenced this issue Jan 22, 2020
darkwing added a commit to darkwing/fenix that referenced this issue Jan 23, 2020
darkwing added a commit to darkwing/fenix that referenced this issue Jan 31, 2020
sblatz added a commit to darkwing/fenix that referenced this issue Jan 31, 2020
boek pushed a commit that referenced this issue Feb 3, 2020
@darkwing darkwing added the eng:qa:needed QA Needed label Feb 4, 2020
@darkwing
Copy link
Contributor

darkwing commented Feb 4, 2020

Fixed in: 4b3effd

@sv-sdeiac
Copy link

Verified as fixed using the latest version of Firefox Preview Nightly 2/5 @20360606.

Devices:

  • Nexus 5 (Android 6.0.1);
  • Nokia 6 (Android 7.1.1);
  • Samsung Galaxy Note 8 (Android 9);
  • Google Pixel 3 (Android Q).

Feature Polish automation moved this from Prioritized Backlog to Done Feb 5, 2020
@sv-sdeiac sv-sdeiac added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Feb 5, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
eng:qa:verified QA Verified eng:ready Ready for engineering Feature:HomeScreen feature request 🌟 New functionality and improvements Feature:Toolbar Address bar, see also Feature:Search P2 Upcoming release
Projects
No open projects
Development

No branches or pull requests

8 participants