Browser-laptop: implement new bravery panel spec

Created on 27 May 2016  路  16Comments  路  Source: brave/browser-laptop

from @bradleyrichter:
pasted image at 2016_05_26 01_05 pm

since we don't yet support flash, the "Block Flash Objects" switch should be un-switchable

featurshields

Most helpful comment

so, it should look like this: (big favicon to come later)

image

All 16 comments

@diracdeltas Let's leave the Flash switch out rather than disabled since we are including an un-switchable Block Popups switch which does block but can't be changed yet.

@bradleyrichter did you intend to set Block Scripts to be unswitchable in the screenshot above?

"Block Popups" and "Block Flash" are in the same boat right now (enabled but unswitchable). I'm fine with only showing "Block Popups" and not "Block Flash" though.

Unintended confusion. We will just remove the "unswitchable" switches and the style for OFF will be dark grey.

++

so, it should look like this: (big favicon to come later)

image

This is basically done now except for the 'Edit Bravery Global Settings...' link which is in progress

I'm pushing a PR to make this

into

align-items: center could be flex-start.

@luixxiul thanks. looks better, though i would prefer the question-mark to be closer to "Fingerprinting Protection". on my screen, it looks fine because everything fits in one line:
screen shot 2016-05-28 at 4 35 29 pm

is it possible for you to make the panel wider so that the text fits in one line without making it wider than the minimum browser window size?

This is a great looking screen :smile:

@diracdeltas I noticed that once too. I think maybe it's not enough to solve the issue completely by changing the width of the panel due to the nature of display: flex. You'll have to rewrite the divs around the question mark. I'll take a look at it again.

I noticed that the previous commit has broken the margin between the shields switch and "Up". I'll push another commit to fix it.

@diracdeltas The reason for the original order of the number display was to put the least-used at the far left. (assuming blocking scripts is least-used)

Is there a reason you preferred or needed the order that was implemented?

Also - Let's add left-margin in front of the ? mark button, and make it dark grey to reduce it's visual importance. (when time permits)

@bradleyrichter I think fingerprinting protection will be less used than script blocking actually. I ordered them so that the one most likely to be '0' is at the end.

I think fingerprinting protection will be used more than script blocking but I'm fine with whatever. Just because the most web breaking thing is script blocking.

Do we need the disabled vertical scrollbar here?

cf. https://twitter.com/bcrypt/status/737171844092956672

This needs to turn into a popup that hides on any click outside of the popup region. I'm worried it will otherwise be confusing when users open them accidentally.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

luixxiul picture luixxiul  路  3Comments

luixxiul picture luixxiul  路  3Comments

jonathansampson picture jonathansampson  路  3Comments

shortstuffsushi picture shortstuffsushi  路  3Comments

bsclifton picture bsclifton  路  3Comments