Closed dhowe closed 2 years ago
@mushon Any idea how are we going to deal with other languages (and likely longer texts) the space is a bit tight so I imagine it will overflow
In the toggle?
I prefer not but we could go for a smaller font size if need be.
@dhowe what are the main languages that are used with adnauseam ? The toggle buttons indeed might become an issue since the space is already quite tight with english...
not sure -- here's the full list: https://crowdin.com/project/adnauseam
not sure -- here's the full list: https://crowdin.com/project/adnauseam
I mean the most often used. English, Russian, Chinese, German... Another one that we should definetelly test? Just to make sure that the UI doesnt get very broken
@dhowe I implemented new footer menu with mostly working interactions (not implemented the "On this page" / "domain" differences):
Perhaps remove the "DISABLE - EVERYWHERE" functionality, it does not exist in uBlock and it would require some other not-implemented mechanism.
agreed - the browser already provides this function for all addons
Implement some sort of notification or status when Strict is enabled (and maybe linking to the strictBlockList page)
also probably need a new color for the adn icon (we already use purple=normal, red=disabled, green=dnt)
@mushon @dhowe I think for the strict state this type (no ads collected) of menu alert should be implemented? Perhaps better than the usual "warning"
that is an alert for when there are no ads FOUND on the page (either there are none, or we couldn't parse any) we could change the text to read 'in strict mode', or we could do something similar to below, also with updated text also need a new color for the icon
@dhowe @mushon I wonder if it should be this alert instead, since some ads can still be collected even if Strict Blocked mode is enabled.
Maybe with a different type of alert image.
That works for me (though the css should match the other alerts) I think we also need a link (perhaps to the "strict-blocked" text) to the FAQ entry And strict-blocked should probably be lower-case...
That works for me (though the css should match the other alerts) I think we also need a link (perhaps to the "strict-blocked" text) to the FAQ entry And strict-blocked should probably be lower-case...
Will do
The strict block list serves to help the users to strict block specific domains or pages more easily. link
@dhowe any final comments?
again, strict block is only for domains, correct?
Yes. But the user can still make a page-based strict-block directly in the "Strict Block List".
yes, but it doesn't make it 'more easy' to do...
@dhowe any final comments?
Let's add a dash between 'strict' and 'blocked', but otherwise looks good to me, @mushon?
@dhowe I suggest a super minor change, that is kind of trickying me up a bit:
Keep the uBlock button on the same side as before. My muscle memory is making me click the wrong button all the time, and I guess for other long time users this change of position might be a little irritation.
Super minor, but perhaps valid.
ok for me, @mushon ?
Sure thing..
Mushon Zer-Aviv
Recent work: Normalizi.ng Mushon.com | Shual.com | @Mushon
On 29 April 2022 at 12:03:52, Daniel Howe @.***) wrote:
ok for me, @mushon ?
— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.
Great work! As we already have implemented i'll be closing this issue
Need to support the following in the bottom section of the menu (*indicates a new item):
There is also the potential concern that the current 'disable' button suggests disabling ADN altogether, rather than only for the current page/site