Implement Add filter list popup

[saroyanm]

Background

After several design iterations we finally have a final version of the new options page (See #5158).

What to change

Implement "add filter list popup" as specified in ​Page Specification.

[saroyanm]

As discussed: we don't need this for the initial launch as this filter lists yet possible to add from the general tab.

[greiner]

We do need it for the initial launch but it doesn't need to contain non-language filter lists yet.

[saroyanm]

Replying to greiner:

We do need it for the initial launch but it doesn't need to contain non-language filter lists yet.

The way I remember we decided to open "Add filter list via URL" dialog meanwhile.
Anyway I'll increase priority until it's not confirmed.

[greiner]

Replying to saroyanm:

The way I remember we decided to open "Add filter list via URL" dialog meanwhile.

Gotcha, I thought you wanted to get rid of the dialog in general but yeah, we don't need it in the Advanced tab for the initial release. Sorry about the confusion.

[saroyanm]

As discussed: we don't need it for the initial release.

[agiammarchi]

This looks like a very interesting task for someone new, but I'd like to ask few questions about specifications.

Focused filter list

Is that just a hover ? Would a click or enter set that row like the last one ?
Should it move down there too ?

What actually is the last Malware Domains greyed out row ? Is it selectable too or just there disabled ?

Language filter lists

Are these shown independently of the installed filters and, if that's the case, are these shown in that table exact order ?

If that's not the case, is there any specific way to relate languages to filters beside the name ?

Add custom subscription popup

If I understand correctly the current extension popup is already the one described here as custom, beside some UI differences.

UI a part, is there anything else that needs to change in the current subscription popup ?

Close VS Cancel

Just a quess, really: is there any reason to have these two around ? The Cancel button, as described, does exactly what the closing button does.

[greiner]

Hope you don't mind if I'll only address the ones that don't require clarification from Product.

Replying to agiammarchi:

Focused filter list

What actually is the last Malware Domains greyed out row ? Is it selectable too or just there disabled ?

It's disabled because it has already been installed.

Language filter lists

Are these shown independently of the installed filters and, if that's the case, are these shown in that table exact order ?

If that's not the case, is there any specific way to relate languages to filters beside the name ?

This dialog already exists in the General tab when you click on "+ Add a language". The only differences are that non-language filter lists need to be shown when opening it from here and that there's a button at the bottom.

Also note that the existing implementation still differs from the specced version (e.g. in regards to styling). But also that we still need to replace the dialog in the General tab with the expected UI element (see ​Adding / Changing a language).

Add custom subscription popup

If I understand correctly the current extension popup is already the one described here as custom, beside some UI differences.

That's correct.

[agiammarchi]

Hope you don't mind ...

not at all, thanks a lot for these info !

we still need to replace the dialog in the General tab with the expected UI element

wouldn't be better to have a separate ticket for that?

This once involves already quite some change but I don't see a ticket specific for the language bit in the other tab.

Glad to see I can reuse most of the current language dialog anyway

[greiner]

Replying to agiammarchi:

wouldn't be better to have a separate ticket for that?

This once involves already quite some change but I don't see a ticket specific for the language bit in the other tab.

Absolutely. I've created #6397 for tackling that.

[saroyanm]

Replaced with the Gitlab link.

Note: Seems like the Specs in the Bitbucket and currently updated reference are different, I can't see currently Non Language Blocking Filter Lists listed.

[greiner]

@saroyanm It looks like this has already been implemented by now or is there anything left to do here? If not please add a reference to the commit in which this change has landed and close this ticket.

[saroyanm]

Replying to greiner:

@saroyanm It looks like this has already been implemented by now or is there anything left to do here? If not please add a reference to the commit in which this change has landed and close this ticket.

Right, This ticket is definitely irrelevant anymore.