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

"Widget Exceptions" are confusing #3007

Open
ghostwords opened this issue Aug 1, 2024 · 1 comment
Open

"Widget Exceptions" are confusing #3007

ghostwords opened this issue Aug 1, 2024 · 1 comment
Labels
bug enhancement ux User experience research needed widgets Click-to-activate placeholders for blocked but potentially useful social buttons/widgets

Comments

@ghostwords
Copy link
Member

ghostwords commented Aug 1, 2024

Users keep misunderstanding what adding a widget to Widget Exceptions does. It does not mean that widget will always be allowed to load everywhere. It means that widget will still be blocked, but no longer replaced. It's like a selective way of unchecking "Enable widget replacement".

Beyond this confusion, some users want a way to always allow a widget everywhere, and we don't have an intuitive way of doing this.

Is there a reason for someone to turn off widget replacement either globally or selectively? Where PB keeps blocking but no longer replaces with a placeholder? If this functionality can be removed, we could then resolve the above confusion and allow users to globally allow particular widgets by re-purposing Widget Exceptions. Basically, make Widget Exceptions work the way people tend to think it does. First implementation thoughts:

  • Remove the "Enable widget replacement" checkbox from the UI, and from widget replacement logic
  • Reword "Don't replace the following widgets:" to something like "Always allow the following widgets:". (May ease the process to use a new locale message for this.)
  • Have widget exceptions save to a new field in storage. When adding an exception, move all associated domain toggles to green. When removing an exception, restore the toggles to red. (Probably have to save the associated domains in the new field, so that when the associated domains change in the JSON, we can do the right thing (add new domains, remove deleted domains) on extension update.)
  • Remove the original widget exceptions field from widget replacement logic
  • Plan to remove "Enable widget replacement" and the original widget exceptions from storage (and from locales) in a separate followup update

After this, issues like #2539 that can now be worked around by disabling replacement for a particular widget could be worked around by always allowing the widget instead.

Previously: #2972, #2612

@ghostwords ghostwords added bug enhancement widgets Click-to-activate placeholders for blocked but potentially useful social buttons/widgets ux User experience research needed labels Aug 1, 2024
@ghostwords
Copy link
Member Author

@DJCrashdummy Any thoughts about this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug enhancement ux User experience research needed widgets Click-to-activate placeholders for blocked but potentially useful social buttons/widgets
Projects
None yet
Development

No branches or pull requests

1 participant