Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Warning user when disabling MV2 uBlock Origin #42608

Closed
3 of 6 tasks
ryanbr opened this issue Dec 2, 2024 · 3 comments · Fixed by brave/brave-core#26903
Closed
3 of 6 tasks

Warning user when disabling MV2 uBlock Origin #42608

ryanbr opened this issue Dec 2, 2024 · 3 comments · Fixed by brave/brave-core#26903
Assignees
Labels

Comments

@ryanbr
Copy link

ryanbr commented Dec 2, 2024

Description

If a user unselects uBO in brave://settings/extensions/v2 All filters and custom settings are removed.

Prompt the user with a warning will clear uBO data and settings.

Steps to reproduce

  1. Unselect mv2 uBO (and probably other extensions will clear settings)

Actual result

Currently deletes all settings automatically.

Expected result

Warn user that unselecting will clear extension data

Reproduces how often

Easily reproduced

Brave version (brave://version info)

Current Brave ver

Channel information

  • release (stable)
  • beta
  • nightly

Reproducibility

  • with Brave Shields disabled
  • with Brave Rewards disabled
  • in the latest version of Chrome

Miscellaneous information

ref: https://old.reddit.com/r/uBlockOrigin/comments/1h41myj/end_of_support_for_ubo_on_chrome_chromium/lzv1cmt/?context=3

Why is there no warning that disabling an extension there will REMOVE IT (wiping your config) instead of just disabling it so you can restore it later?

You should either add a warning or offer the user a choice to remove or disable it temporarily. A lot of people doesn't backup their extension settings regularly so having an option like that with no warnings seems like a terrible idea to me.
@ryanbr ryanbr added OS/Desktop priority/P4 Planned work. We expect to get to it "soon". labels Dec 2, 2024
@ShivanKaul
Copy link
Collaborator

Instead of showing a warning, we can simply split the current "toggled off" state to be different between "disabled" and "uninstalled". When the user toggles the button on an extension in brave://settings/extensions/v2, we should "disable" first, and then show a button called Remove next to the toggle. Then we don't need a warning.

@u-RraaLL
Copy link

u-RraaLL commented Dec 3, 2024

Yup, that's very close to what I meant with:

or offer the user a choice to remove or disable it temporarily

@MadhaviSeelam
Copy link

MadhaviSeelam commented Jan 10, 2025

Verification PASSED using

Brave | 1.75.157 Chromium: 132.0.6834.83 (Official Build) beta (64-bit)
-- | --
Revision | d8524431e89200eff892315294bd590b314f47d2
OS | Windows 11 Version 24H2 (Build 26100.2605)
  1. Installed 1.75.157
  2. launched Brave
  3. closed and relaunched Brave to pull in griffin
  4. opened brave://settings/extensions/v2
  5. Toggled Enable uBlock Origin extension
  6. clicked Add extension
  7. verified the extension is added via toolbar
  8. toggled off the extension,
  9. confirmed Remove button is shown
  10. opened brave://extensions
  11. confirmed the extension is disabled
  12. clicked Remove
  13. confirmed Remove button disappeared
  14. confirmed the extension is uninstalled via brave://extensions
step 5 step 7 step 9 step 8 step 11 step 13 step 14
Image Image Image Image Image Image Image

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
7 participants