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

In the "My Filters" tab, no information is shown when the list is disabled #3161

Closed
9 tasks done
ruihildt opened this issue Mar 8, 2024 · 19 comments
Closed
9 tasks done
Labels
enhancement New feature or request fixed issue has been addressed

Comments

@ruihildt
Copy link

ruihildt commented Mar 8, 2024

Prerequisites

  • I verified that this is not a filter list issue. Report any issues with filter lists or broken website functionality in the uAssets issue tracker.
  • This is NOT a YouTube, Facebook or Twitch report. These sites MUST be reported by clicking their respective links.
  • This is not a support issue or a question. For support, questions, or help, visit /r/uBlockOrigin.
  • I performed a cursory search of the issue tracker to avoid opening a duplicate issue.
  • The issue is not present after disabling uBO in the browser.
  • I checked the documentation to understand that the issue I am reporting is not normal behavior.

I tried to reproduce the issue when...

  • uBO is the only extension.
  • uBO uses default lists and settings.
  • using a new, unmodified browser profile.

Description

When "My filters" list is deactivated in the "Filters list", users are confused that adding custom filters in the "My filters" tab doesn't work, because they don't notice the whole thing is disabled.

I would like to suggest showing a message on the "My filters" tab page when the "My filters" list isn't enabled.

This is especially an issue for Mullvad Browser users, since the "My filters" list is disabled by default, for browser fingerprinting concerns (custom filters could make you more uniquely identifiable).

(FYI, I'm working for Mullvad on the Mullvad Browser)

A specific URL where the issue occurs.

https://mullvad.net

Steps to Reproduce

  • Start Firefox with uBlock Origin
  • Click on the
  • Disable the "My filters" list in the "Filters list" tab
  • Go to the "My filters" tab

Expected behavior

When the "My filters" list is disabled, an information message is displayed in the "My filters" tab.

Actual behavior

When the "My filters" list is disabled, there is no way to know in the "My filters" tab that the list is currently disabled.

uBO version

1.56.0

Browser name and version

Firefox 123

Operating System and version

Fedora KDE Spin 39

@garry-ut99
Copy link

garry-ut99 commented Mar 8, 2024

Also a related issue/comment wherein someone already asked same:

UnitedMarsupials : #3076 (comment) : But shouldn't there have been a warning of some sort?

@gwarser gwarser added the enhancement New feature or request label Mar 8, 2024
@gorhill
Copy link
Member

gorhill commented Mar 9, 2024

How about a checkbox to enable/disable "My filters" directly in the pane?

image

@ruihildt
Copy link
Author

ruihildt commented Mar 9, 2024

@gorhill That would be even clearer.

gorhill added a commit to gorhill/uBlock that referenced this issue Mar 11, 2024
Related issues:
- uBlockOrigin/uBlock-issues#3161
- uBlockOrigin/uBlock-issues#2895 (reply in thread)

Two checkboxes have been added to the "My filters "pane:

1. A checkbox to wholly disable/enable "My filters". This is equivalent
   to the checkbox for "My filters" in "Filter lists" pane.

2. A checkbox to enable/disable the trustworthiness of the content
   of "My filters". Default to untrusted.

Since toggling these checkboxes requires reloading all filter lists,
their new state must be committed through the "Apply changes" button.

Additionally: a "book" icon has been added to the top-right of the
dashboard, which is a link to the wiki according to whichever pane is
currently active.
@u-RraaLL
Copy link
Contributor

Could you add a button (e.g. next to the export buttons) for mobile to collapse the checkbox section?

They take a significant amount of space, which mobile doesn't have a lot of when typing.

@gwarser gwarser added the fixed issue has been addressed label Mar 12, 2024
@garry-ut99
Copy link

garry-ut99 commented Mar 12, 2024

I definately agree to the above comment, these tickboxes take precious space in My filters tab on Mobile, as a result it's now more hard/cumbersome to edit/type and maintentain user filters.

(...) add a button to collapse the checkbox section?

Maybe also to collapse "Do not add filters from untrusted sources." message as well
(unless declined for security reasons for majority/novice users).

gorhill added a commit to gorhill/uBlock that referenced this issue Mar 12, 2024
@u-RraaLL
Copy link
Contributor

Nice changes. The search input placeholder which is meant to replace the outside magnifying glass icon kinda looks like a pin/location icon to me though. Why not go the same route as in the Filter lists pane where the svg is simply inside the input?

If you want to keep the placeholder method instead, probably best to change it in that pane too. For unity.

But I think a better choice for a placeholder might be the telephone recorder which looks like the right-facing magnifying glass: ऋ.

@gorhill
Copy link
Member

gorhill commented Mar 12, 2024

better choice for a placeholder might be the telephone recorder

That's how it looks on my side, in both Chromium and Firefox:

image

I had forgotten about what I did for "Filter lists", but the idea was that I wanted to keep it simple by just using a Unicode character in the placeholder attribute. There is an emoji for magnifier -- 🔍 -- but support is not guaranteed.

gorhill added a commit to gorhill/uBlock that referenced this issue Mar 13, 2024
@krystian3w

This comment was marked as outdated.

@garry-ut99
Copy link

garry-ut99 commented Mar 16, 2024

There is an issue related to the new Support maximizing editor to viewport size functionality on Mobile, sometimes while in Maximized editor, after changing/editing, and then pressing minimize editor icon-button to return to normal view, another button - Apply button is accidentaly/randomly being pressed altogether because Apply button is perfectly behind the minimize editor icon-button, the issue happens when the press/tap is a long enough (but still visually short), it happens only on Mobile, it doesn't happen on Desktop when clicking using mouse, so it seems that on Mobile longer pressings/taps are considered as double/multi "clicks", hence if there is a second button hidden behind the first button, they sometimes are pressed/triggered both at the same time, which is not an expected behavior, at least not when triggered randomly/accidentaly, it should be made consistent: either fixed so that Apply button is never being accidentaly pressed or to make the both buttons always being pressed altogether (means working as autosave feature when leaving out maximized editor state - but this is just a quick idea, I didn't think about it long enought)

@gorhill
Copy link
Member

gorhill commented Mar 16, 2024

The event handler for the search bar is hooked onto mousedown, I can't remember why I implemented that way. Using mousedown instead of click is probably what is causing this issue.

@u-RraaLL
Copy link
Contributor

Weird, I've been able to reproduce this on my 3rd and 4th try, but not any of the latter dozens of tries I've made.

@gorhill
Copy link
Member

gorhill commented Mar 19, 2024

I will use click instead of mousedown, this should fix the issue. As to why I used mousedown, I believe it was because using click causes the CodeMirror editor to lose focus, but I found that just calling cm.focus() in the event handler gives back focus to the editor.

gorhill added a commit to gorhill/uBlock that referenced this issue Mar 19, 2024
Simplify code to force syntax highlighter and linter to reprocess
the content when toggling trust-related checkbox.

Fix issue with using `mousedown` on searchbar widgets, related
feedback:
uBlockOrigin/uBlock-issues#3161 (comment)

Fix issues reported by eslint.
@garry-ut99
Copy link

garry-ut99 commented Mar 21, 2024

Another issue is that Undo / Redo doesn't work in search field in "My filters" and "Trusted sites" and also in "resource viewer", for a comparision - it works fine in "Filter lists" and "My rules".

@gorhill
Copy link
Member

gorhill commented Mar 21, 2024

it works on my side when the search field has focus, i.e. when the caret is visible in the search field.

@garry-ut99
Copy link

I forgot to check in Chromium - yes indeed it does work in Chromium, but not in Firefox.

@gorhill
Copy link
Member

gorhill commented Mar 21, 2024

Works the same in Firefox on my side.

@garry-ut99
Copy link

garry-ut99 commented Mar 21, 2024

I forgot, yes it works fine in FF 124, the issue occurs in Firefox 115 ESR only (perhaps in FF 116-123 too - I didn't check), I won't bother you with a repair on the FF 115 ESR, unless you decide to do it yourself.

@gorhill
Copy link
Member

gorhill commented Mar 21, 2024

When you reproduce the issue, is the undo applied to the content of the CodeMirror editor?

@garry-ut99
Copy link

garry-ut99 commented Mar 21, 2024

No, Undo/Redo is broken in CodeMirror editor as well.
It's also broken in "My rules" editor as well, but not broken in the search field of "My rules".

gorhill added a commit to gorhill/uBlock that referenced this issue Mar 26, 2024
Simplify code to force syntax highlighter and linter to reprocess
the content when toggling trust-related checkbox.

Fix issue with using `mousedown` on searchbar widgets, related
feedback:
uBlockOrigin/uBlock-issues#3161 (comment)

Fix issues reported by eslint.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request fixed issue has been addressed
Projects
None yet
Development

No branches or pull requests

6 participants