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

Summary of requested UI changes #601

Closed
nodiscc opened this issue Feb 15, 2015 · 9 comments
Closed

Summary of requested UI changes #601

nodiscc opened this issue Feb 15, 2015 · 9 comments

Comments

@nodiscc
Copy link
Contributor

nodiscc commented Feb 15, 2015

As discussed in #517 (comment) here is a quick draft of what added UI elements could look like, if added as requested in:

rp-ui

This could help getting a clearer view of what the menu UI could look like. I omitted "bundle" rules #338 as there is already a screenshot there. I'll upload a screenshot without annotations if needed, feedback welcome.

I forgot the visual indicator for private browsing mode (#261 - temporary rules only). Having FF's "private browsing" icon on bottom right with text Private browsing. Only temporary rules allowed should do it.

@Cerberus-tm
Copy link

This is all very good, well done!

One more thing: I'd like to be able to individually make temporary rules permanent too, perhaps by right-clicking a temporary rule?

I'd also like to e.g. allow all subdomains on .cloudfront, instead of xsdkfljd.cloudfront or uyxwiuet.cloudfront, etc. (sometimes the same site uses several cloudfront subdomains). But that is probably already solved by your edit button next to the rule? I wouldn't mind clicking that button, then being sent to the rule-editing screen, then removing "xsdkfljd" to get that.

@nodiscc
Copy link
Contributor Author

nodiscc commented Apr 9, 2015

I'd like to be able to individually make temporary rules permanent too, perhaps by right-clicking a temporary rule?

Please discuss in #294. The Make permanent button would make all current temporary rules permanent. Making just one of these rules permanent is not planned.

'd also like to e.g. allow all subdomains on .cloudfront, instead of xsdkfljd.cloudfront or uyxwiuet.cloudfront, etc.

Please discuss in #470. The edit button would show a dialog with a pre-filled rule you can edit:
[Allow] requests from [*] to [*.gravatar.com] [OK] [Cancel]

Thanks for your feedback

@Cerberus-tm
Copy link

Ad 1: I've added my comment to issue number 294. Rarely would I want to make all temporary permissions permanent: nearly always I only want to make one or two permanent. Perhaps this depends on the kinds of sites you visit, but especially with complex sites this would save me a lot of time. As it is, I have to reload the site twice in order to make a single permission permanent. The workflow in the OP's second post on issue number 294 is exactly what I do all the time.

Ad 2: That is great! That should work for all possible scenarios.

Thanks for all the work you're doing, that mock-up looks good.

@myrdd
Copy link
Member

myrdd commented Apr 23, 2015

Thanks a lot @nodiscc for putting it all together, it will help me a lot. Thumbs up!
I've added the wanted label to this issue, because those changes will improve the menu a lot!

@Cerberus-tm
Copy link

Yay, this will be great! Especially being able to make temporary permissions permanent one by one will help me.

@netjeff
Copy link

netjeff commented Feb 7, 2016

I'd really appreciate options to disable or switch default policies only for the current tab. Like "Disable blocking on this tab" or "Switch default policy on this tab". This would be very similar to #422, but the timeliness would be for the life of the current tab rather than the options discussed on #422.

Here's my use case: I run with "default deny". Many times I'll have a tab open on a site that I'm unlikely to come back to (or else rarely), but I need to buy something or maybe fill out a support form. Even with the "temporarily allow all from ThisSite.com", due to 3rd party request (especially for credit cards) even that doesn't work. Usually I'm willing to take a bit of risk for convenience, and use the "disable blocking" option. But that puts me at risk in all my other tabs, plus I have to remember to undisable after I'm done. So having a "disable blocking for this tab" (or maybe switch default policy for this tab) would limit the risk only to that one tab.

I'm not sure how to fit this onto an already crowded menu... maybe a checkbox next to the switch, labelled something like "this tab only" ?

@Cerberus-tm
Copy link

@netjeff Excellent idea, I would use this! I often experience the same scenario.

@timacs
Copy link

timacs commented Feb 7, 2016

👍
Also, we would want some warning from RPC about that carte blanche tab. Maybe, a crossed flag?

@myrdd
Copy link
Member

myrdd commented Feb 9, 2016

I agree, I also would like that feature ;) There's already an issue about it. I've posted my comment there: #495

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants