Skip to content
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

Implement Shield Panel design with deep link to global settings on Android (core) #6831

Open
anthonypkeane opened this issue Nov 7, 2019 · 15 comments
Assignees
Labels
feature-request OS/Android Fixes related to Android browser functionality priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes

Comments

@anthonypkeane
Copy link

anthonypkeane commented Nov 7, 2019

Description

Implement the simpler shield panel design

Designs

shields 1

Tapping the help icon will show more information about Brave Shields:

shields 2

Advanced site-specific controls are available in an expand/collapse section, which persists across sites. So if the user expanded Advanced controls on one site, it will still be expanded when they open Shields on a different site.

shields 3

Label changes:

  • "Block Ads & Tracking" becomes "Block cross-site trackers"
  • "HTTPS Everywhere" becomes "Upgrade connections to HTTPS"
  • Same options to block cookies and fingerprinting as on desktop

Tapping on "Block cross-site cookies" opens options in a subpanel:
shields 4

Tapping on "Block cross-site fingerprinting" opens options in a subpanel:
shields 5

Tapping "Change global Shields defaults" under Advanced Controls deeplinks the user to the app settings page:

image

Tap on the cookies line item to see more options in a sub-page:

Settings 3

Tap on the fingerprinting line item to see more options in a sub-page:

Settings 4

When Brave Shields is down (whether Advanced controls is expanded or not), users can report a broken site by tapping "Report a broken site"

shields webcompat 1
shields webcompat 2

The confirmation message should dismiss on its own after 1s if the user doesn't tap out first:

shields webcompat 3

Dark theme

Dark theme is supported. Text, toggles, buttons, links, and icons have lighter colors:

dark theme shields 1
dark theme shields 2
dark theme shields 3
dark theme shields 4
dark theme shields 5
dark theme shields 6

Landscape mode

This works the same as currently.

shields landscape

Settings

Since Brave Shields global defaults now has its own page in app settings, make some minor organizational changes to main app settings page as well:

image

image

Interaction finesse

The expand/collapse Advanced controls should animate content in (this is an iOS example but also applies to Android):

shields expand collapse

Breakdown: Grow white panel 0.2s, then content fades in Y+10 0.3s

Also, use existing Android functionality to slide content in from left for each new view.

To add delight to the experience as well as a fun touch complimenting the oversized toggle, implement new color gradient when the toggle is ON:

fabtoggle-android

it's a loop with 2ms between each color state + easy ease:
linear-gradient(120deg, rgba(255,167,59,1) 0%, rgba(255,118,84,1) 100%);
linear-gradient(120deg, rgba(255,118,84,1) 0%, rgba(251,84,43,1) 100%);
linear-gradient(120deg, rgba(251,84,43,1) 0%, rgba(247,36,28,1) 100%);
linear-gradient(120deg, rgba(247,36,28,1) 0%, rgba(252,79,130,1) 100%);
linear-gradient(120deg, rgba(252,79,130,1) 0%, rgba(255,167,59,1) 100%);
linear-gradient(120deg, rgba(255,167,59,1) 0%, rgba(255,118,84,1) 100%);
#FFA73B ➡️ #FF7654 ➡️ #FB542B ➡️ #F7241C ➡️ #FC4F82 ➡️ #FFA73B

fabtoggle.mov.zip
As reference, it can be done in CSS+JS https://www.joshwcomeau.com/posts/rainbow-button/

User Flow

android prototype

Zipped .mov:
android prototype.mov.zip

Notes

Shields will still show the "Report a broken site" button when Shields is down even if they have already reported that site.

Assets

Figma: https://www.figma.com/file/c5Sq98FXH6oz1z3FL8K3eN/Android-Brave-Shields

icons.zip
settings icons.zip

@anthonypkeane anthonypkeane added design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. feature-request QA/Yes labels Nov 7, 2019
@bsclifton bsclifton added the OS/Android Fixes related to Android browser functionality label Nov 7, 2019
@karenkliu
Copy link

working on this design currently, will update the issue here when done

@anthonypkeane anthonypkeane added OS/Android Fixes related to Android browser functionality and removed OS/Android Fixes related to Android browser functionality labels Dec 19, 2019
@karenkliu
Copy link

@anthonypkeane Designs added!

@deeppandya FYI we still need to finalize the "Google Safe Browsing" item in Settings so there are still some pending changes. But all the other screens aside from Settings are ready. :)

@karenkliu
Copy link

@deeppandya Settings designs finalized and assets uploaded. lmk if you need anything else!

@anthonypkeane anthonypkeane removed the design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. label Feb 12, 2020
@deeppandya deeppandya self-assigned this Apr 24, 2020
@karenkliu
Copy link

@deeppandya Added gradient info!

@deeppandya
Copy link
Contributor

Sub issue for shields panel design. #9888

@srirambv
Copy link
Contributor

Moving out of 1.11.x as @anthonypkeane mentioned this is for settings and wont go into 1.11.x

More info: https://bravesoftware.slack.com/archives/C0816QH3R/p1595532140182800?thread_ts=1595492138.179500&cid=C0816QH3R

@anthonypkeane please update the title of the issue appropriately so that it avoids confusion with other issues

@karenkliu
Copy link

I updated this to include the Privacy Report section

@anthonypkeane anthonypkeane added the priority/P3 The next thing for us to work on. It'll ride the trains. label Sep 9, 2020
@Hazbelll
Copy link

My apologies for asking that question, but is there any ETA on when this will begin to roll out? It appears excellent progress has been made, but unsure what else might be pending.

It would greatly enhance the mobile Brave experience, including squashing some unintended bugs (See #11246 for example).

@anthonypkeane
Copy link
Author

Hi @Hazbelll
These feature is actually live now. Please update to the latest version on the play store.

@deeppandya can we close this ticket now?

@deeppandya
Copy link
Contributor

@anthonypkeane it includes changes in settings screen as well. If we can create a separate issue for settings changes then we can definitely close it.

@Hazbelll
Copy link

@anthonypkeane Thank you so much! That's a nice surprise, although it's not visible in the Play Store for me yet after just now checking for updates.

I'll keep my eyes peeled over the next few days for when it pops up.

@Hazbelll
Copy link

Hazbelll commented Oct 21, 2020

Hello @anthonypkeane, I today received an update for Brave on the Play Store , however this has not included the changes outlined here.

It seems this Brave update (1.15.76) merely updates the Chromium version, according to Brave's support team on Reddit.

Could you please clarify which version of Brave the following changes are slated for? Thank you.

@srirambv srirambv changed the title Implement simpler Shield Panel design on Android (core) Implement Shield Panel design with deep link to global settings on Android (core) Oct 22, 2020
@srirambv
Copy link
Contributor

Updated the issue title to make it more clear. @deeppandya @anthonypkeane please update if required or if you want to log a new one instead of this. Or better to update graphics to show what is not implemented yet

@Hazbelll simpler shields is live in 1.15.76. This issue is for having global settings options in shields and settings reorder which is still not done so won't be available in the builds from playstore

@Hazbelll
Copy link

@simonhong Thank you for clarifying and my apologies as I thought initially everything outlined here had been pushed to release.

I hope the settings menu changes are merged soon. They would squash the bug in issue #11246 (As previously mentioned). Thanks.

@Hazbelll
Copy link

Has this been abandoned? Why has everything fallen silent on something that looks feature-complete?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request OS/Android Fixes related to Android browser functionality priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes
Projects
None yet
Development

No branches or pull requests

7 participants