Fenix: [Bug] Remove line between URL bar and Android gesture bar

Created on 18 Dec 2019  Â·  21Comments  Â·  Source: mozilla-mobile/fenix

Steps to reproduce

  1. Open Firefox Preview on Android 10 with Settings > Display > Navigation bar > Full screen gestures enabled.
  2. There is no line between “Search on enter address” and Android gesture bar on the bottom. It looks clean. ✅

Screenshot_20191217-231541

  1. Open any website
  2. There is a line between the URL bar and Android gesture bar on the bottom. It looks not clear. ❎

Screenshot_20191217-231240

Expected behavior

I think we do not need this extra line between the URL bar and the Android gesture bar. It will be nice if on step 4 we do no have the line as well as we do not have it on step 2.

Actual behavior

There is a line between the URL bar and Android gesture bar

Device information

  • Android device: Pixel 4 XL
  • Fenix version: 3.0.0 (Build 13431956)
Toolbar P2 S2 engverified ready implementation review 🐞 bug

Most helpful comment

Another way is remove a line between bottom navbar and gestures bar. We already have it in Firefox Preview homepage

Screenshot_20200318-192653

And many Google’s apps integrated gestures bar to their UI

Screenshot_20200318-192827
Screenshot_20200318-192802
Screenshot_20200318-192737

All 21 comments

@AmyYLee @topotropic Is this a bug?

I checked other popular Android applications with the bottom bar, and most of them don’t have a line between the bottom bar and the Android gesture bar. Most of them use a black background for a gesture bar. But some of them use the same color for bottom bar and system gesture bar without line between them.

  • Yandex Browser: black gesture bar.
  • WhatsApp: black gesture bar.
  • WeChat: gesture bar has the same color, no line between the text field and gesture bar.
  • Telegram, VK chat: black gesture bar.

@AmyYLee @topotropic Is this a bug?

I'm okay with removing the hairline between the URL bar and the gesture bar. We are doing this in PBM https://github.com/mozilla-mobile/fenix/issues/4107. @topotropic What do you think?

I spoke with @topotropic about this. Can you please set the the android gesture nav bar to black on Light/Dark themes. Thanks

What is the next step for this issue?

I spoke with @topotropic about this. Can you please set the the android gesture nav bar to black on Light/Dark themes. Thanks

@ai Please see last comment. Can you set the android navigation bar to black for Light/Dark themes. Please leave Private browsing as is since it's already a dark purple. Thanks.

Sure. Here is the latest stable Firefox Preview on Pixel 4 XL

Screenshot_20200317-133644
Screenshot_20200317-133654
Screenshot_20200317-133726

@ai I mean the bottom navigation bar. See below. Thanks

Dark/Light theme

Screen Shot 2020-03-17 at 15 57 42

@AmyYLee Google Pixel 4 doesn’t have this navigation bar by default. By default, it uses gesture navigation (similar to iOS) instead of 3-button navigation.

Gesture navigation draws only small navbar with a single dash in the middle.

Do you want me to switch to 3 buttons navigation to show the current Firefox Preview with this mode? (Firefox looks good with 3 buttons mode, the issue only about UX in gesture mode)

@AmyYLee Google Pixel 4 doesn’t have this navigation bar by default. By default, it uses gesture navigation (similar to iOS) instead of 3-button navigation.

Gesture navigation draws only small navbar with a single dash in the middle.

Do you want me to switch to 3 buttons navigation to show the current Firefox Preview with this mode? (Firefox looks good with 3 buttons mode, the issue only about UX in gesture mode)

@ai Hi, yeah I think we should still have the navigation bar coloured black (even in gesture mode it will still show a narrower version as shown in the screenshot).

@AmyYLee yeap, it is a nice idea. Most apps with bottom panel do the same black gesture bar (WhatsApp, WeChat).

@ai Hi, yeah I think we should still have the navigation bar coloured black (even in gesture mode it will still show a narrower version as shown in the screenshot).

It will be weird to have the status bar translucent (ie, showing app theme color) and the navigation bar black.
Also having the navigation translucent makes the app more integrated and web browsing more immersive.

Another way is remove a line between bottom navbar and gestures bar. We already have it in Firefox Preview homepage

Screenshot_20200318-192653

And many Google’s apps integrated gestures bar to their UI

Screenshot_20200318-192827
Screenshot_20200318-192802
Screenshot_20200318-192737

Another way is remove a line between bottom navbar and gestures bar. We already have it in Firefox Preview homepage

@ai Works for me. Thanks

@AmyYLee what is the next step? I’d like to help, but not sure that I know the righr programming language

So reading this thread it sounds like the action item here is to remove the bottom hairline from the homescreen toolbar? Is that correct @AmyYLee ?

So reading this thread it sounds like the action item here is to remove the bottom hairline from the homescreen toolbar? Is that correct @AmyYLee ?

Yes. I'm looking at Nightly right now and it seems it's already removed.

@AmyYLee it looks like on the browser screen it is still there in light and dark theme. Should we also remove it there?

@AmyYLee it looks like on the browser screen it is still there in light and dark theme. Should we also remove it there?

Oh I see it now. Yes please!

Our intern will be working on this bug, please don't pick this up

Verified as fixed on Nightly 5/18 with Google Pixel 4 XL (10).
Note that the bottom hairline is no longer displayed in the home screen or browser screen for light and dark themes in Normal and Private browsing.

Screenshot_20200518-135149

Was this page helpful?
0 / 5 - 0 ratings

Related issues

vesta0 picture vesta0  Â·  3Comments

clitetailor picture clitetailor  Â·  3Comments

thelazyoxymoron picture thelazyoxymoron  Â·  3Comments

softvision-miralobontiu picture softvision-miralobontiu  Â·  3Comments

abodea picture abodea  Â·  3Comments