brave-browser icon indicating copy to clipboard operation
brave-browser copied to clipboard

[Android] Include option for users to hide privacy stats from the NTP

Open Brave-Matt opened this issue 3 years ago • 25 comments
trafficstars

Description

Users should have a way to hide the privacy stats widget from their NTP if desired. I've seen several users make this request and think that it is reasonable and valid, especially considering we allow this option on the desktop version of the browser. image

Possible solutions/suggestions:

  • Add overflow/three dots menu to stats widget that offers option to hide them
  • Include a toggle option in Settings --> New Tab Page that allows user to toggle the widget on/off
  • Include option in Settings or Main menu to display privacy report (whether widget is toggled on/off or not)

@bradleyrichter @jenn-rhim @SergeyZhukovsky

Additional information

Community thread: https://community.brave.com/t/how-to-hide-brave-stats-in-android/384365 Internal Slack convo: https://bravesoftware.slack.com/archives/C0816QH3R/p1651262346660889

Brave-Matt avatar May 18 '22 17:05 Brave-Matt

The first desire is to add back the settings toggles as you suggested:

"Include a toggle option in Settings --> New Tab Page that allows user to toggle the widget on/off"

No other UI work is needed until we have privacy stats complete including the label and click area in the stats widget.

bradleyrichter avatar May 18 '22 17:05 bradleyrichter

We could go with the following copy to keep Settings --> New Tab Page consistent:

Show stats widget On/Off

image

@rmcfadden3 does that copy sound good to you?

timchilds avatar May 18 '22 20:05 timchilds

@timchilds — yeah, copy looks good! ship it!

rmcfadden3 avatar May 18 '22 20:05 rmcfadden3

I have the same issue and opened a similar ticket (#22853).

Some year ago I removed Brave from all systems because of this.. then I saw that this was in some manner fixed and I reinstalled it. I find the reintroduction of this issue very irritating. I do not understand the usefulness for the user to know that 32k ads are blocked or that 63 minutes are saved etc.. It is only garbage on the device screen. Why should a programmer waste time in implementing that?

angelog0 avatar May 18 '22 20:05 angelog0

BTW, I find that collecting that data is not exactly in line with the policy of the Secure, Fast & Private Web Browser ...

angelog0 avatar May 18 '22 20:05 angelog0

@timchilds @bradleyrichter wouldn't it make sense to keep the settings text similar to desktop version ? we have Show Brave stats and Show top sites in the settings for desktop with switch to toggle.

deeppandya avatar May 18 '22 23:05 deeppandya

Keeping it consistent with desktop makes sense to me. Unless @bradleyrichter / @rmcfadden3 think otherwise.

timchilds avatar May 18 '22 23:05 timchilds

@deeppandya & @timchilds — sorry, I think I'm missing something. I thought this task was for Settings toggles on whether to "Show background images" and "Show sponsored images" … But @deeppandya looks like you're commenting on a different set of settings RE: showing stats & top sites?

Correct me if I'm totally garbling things here.

rmcfadden3 avatar May 19 '22 01:05 rmcfadden3

@rmcfadden3 this ticket is to include a setting for users to hide the privacy stats from the NTP.

The two options discussed for the copy:

Option 1 Show stats widget On/Off

Option 2 Show Brave stats On/Off

Option 2 is what we currently have on desktop. Let us know which option you think we should use? Option 2 makes sense to me to keep things consistent.

timchilds avatar May 19 '22 01:05 timchilds

@timchilds — oh wow. Apologies. I completely missed that.

Yes, keeping consistent with desktop LGTM.

rmcfadden3 avatar May 19 '22 01:05 rmcfadden3

yep. Following desktop will be fine.

image

bradleyrichter avatar May 19 '22 01:05 bradleyrichter

Closing the issue as this should be fixed by https://github.com/brave/brave-core/pull/13519. https://github.com/brave/brave-browser/issues/23118 was the issue linked to the PR but was closed as dupe of this issue. Adding it to 1.41.x milestone to match PR and requires update if uplifted

srirambv avatar Jun 08 '22 10:06 srirambv

@srirambv https://github.com/brave/brave-core/pull/13519 only updates code for top sites. It doesnt update anything for brave stats as we need more information to remove it.

deeppandya avatar Jun 08 '22 12:06 deeppandya

@srirambv brave/brave-core#13519 only updates code for top sites. It doesnt update anything for brave stats as we need more information to remove it.

Removing the 1.41.x milestone as this wasn't fixed/addressed as per the above. https://github.com/brave/brave-core/pull/13519 only gives the user the ability to disable Top Sites and not the Stats.

kjozwiak avatar Jun 14 '22 19:06 kjozwiak

Hello, I would like to know why this issue hasn't been addressed yet? And why users privacy is not being taken seriously?

There is no reason why your browser needs to keep track of any of my following browser habits:

1.) Trackers & Ads Blocked

2.) Estimated Data Saved

3.) Estimated Time Saved

All that tells me is that even though your ad blocker saves me time, and data, you are still learning my browser habits and behaviors by logging that information.

It takes up a lot of space on my new tab page and is very Intrusive and would like to opt out of the collection of these statistics and remove this widget from the browser.

Can anyone aware of this certain issue provide an update on when this issue will get fixed or why it willn't be or a reason for why there's been no recent discussion in the past 3 weeks about it? @bradleyrichter @timchilds @deeppandya @srirambv @kjozwiak @ShivanKaul @bradleyrichter @jamesmudgett @kylehickinson

Thank you.

Jpersh27 avatar Jul 04 '22 23:07 Jpersh27

Hello again,

It has been 3 days now and I haven't been acknowledged in anyway or given a reason for why this issue is being ignored or still unresolved.

It would be much appreciated if someone would respond back with some form of details. At this point I feel as though that any issues being raised by the community are being ignored and put on the “back burner " till further notice for whenever you deem it necessary to address them.

Please persuade me otherwise @bradleyrichter @timchilds @deeppandya @srirambv @kjozwiak @ShivanKaul @bradleyrichter @jamesmudgett @kylehickinson point any information is better than being ignored and shunned by the development team.

Thank you.

Jpersh27 avatar Jul 07 '22 00:07 Jpersh27

Is there a chance that this will be fixed soon? It's really annoying to me. @bradleyrichter @timchilds @deeppandya @srirambv @kjozwiak @ShivanKaul @bradleyrichter @jamesmudgett @kylehickinson

der-p avatar Jul 09 '22 03:07 der-p

@Jpersh27 @der-p thank you so much for reaching out to us. The data we show in brave stats is aggregated only on the client device and not passed in any way to Brave. So we are keeping our brand promise regarding user data protection and privacy. We are looking into providing a proper way to resolve the concern of hiding the stats. Privacy stats is very important for Brave shields so we want to address the issue in a proper way.

deeppandya avatar Jul 12 '22 15:07 deeppandya

@deeppandya Thank you for your answer. I understand that brave shields is an important feature for you. In general, features provided by Brave should be as optional as possible, since not every user needs and wants all of them. and I think it's not just about where the data is stored. it's about a clean look and feel too. since the desktop version already has the option, the way to implement it properly is already obvious imo. anyway, i'll keep an eye on this and use another chromium fork until then.

der-p avatar Jul 23 '22 07:07 der-p

I just upgraded to 1.42.83: same situation described in this issue. Nothing changed since April and, frankly, I am annoyed each time to revert to 1.37 which is the last that works as expected. I do not understand because the Brave developers do not revert their changes regarding this issue to the 1.37 version. ALL THIS IS VERY ANNOIING. I will wait another TWO weeks then I will remove Brave from ALL my device (Win, GNU/Linux, macOS, Android) because I use the same browser on all my systems. I already did so a few year ago. Now I will do that definitively. I am afraid...

angelog0 avatar Aug 01 '22 00:08 angelog0

@anthonypkeane Here is my suggestion for the setting page to toggle off Privacy Report. @rmcfadden3 Please check this, i am calling them "widgets" to group it with Top sites. Also i have removed the "Show" verb from the item as it is unnecessary as it has a toggle and the state of it directly below.

Hide Privacy Stats

Sam-Sibley avatar Aug 18 '22 19:08 Sam-Sibley

@Sam-Sibley — yeah, this lgtm. I think "widgets" is widely understood as a term. Ship it!

rmcfadden3 avatar Aug 18 '22 19:08 rmcfadden3

So @anthonypkeane and @Sam-Sibley and @rmcfadden3 why has this Option not been released yet?

It has been 4 months since this issue was opened. Also been 25 days since you said “ship it".

An explanation would be great, Yes or No?? But I know I'm asking for to much at this point....

As of Beta version 1.44.82 there is still no option to disable it from the homepage. That is very disappointing and doesn't impress me nor convinces me to come back and use the brave web browser.

Jpersh27 avatar Sep 12 '22 22:09 Jpersh27

@deeppandya here is how iOS implemented this: https://github.com/brave/brave-ios/issues/6003

The NTP design (3 dots on privacy stats widget) is in the iOS ticket comments - can we do the same on Android?

The menu design is covered by @Sam-Sibley previous comment.

timchilds avatar Sep 21 '22 17:09 timchilds

@timchilds we can definitely do that.

deeppandya avatar Sep 21 '22 17:09 deeppandya

Any update on when this fix will be added ?

eataiwo avatar Nov 22 '22 06:11 eataiwo

Please please allow us to remove the stat in NTP. The mockups proposed here all look good.

snhv avatar Nov 28 '22 21:11 snhv

Updated design Screenshot_20221202-125626_Brave - Nightly.jpg Screenshot_20221202-125615_Brave - Nightly.jpg

deeppandya avatar Dec 02 '22 17:12 deeppandya

Ok @deeppandya, this has been already described in previous posts in some manner.. but WHEN this will be released???

angelog0 avatar Dec 02 '22 21:12 angelog0

Ok @deeppandya, this has been already described in previous posts in some manner.. but WHEN this will be released???

We have the changes ready to merge. So It will be available on nightly soon.

deeppandya avatar Dec 02 '22 21:12 deeppandya