Blocked script info dialog should be under noScript icon

[srirambv]

Did you search for similar issues before submitting this one?

Describe the issue you encountered:
Blocked script info window is off location

Expected behavior:
Should be below the blocked script icon

• Platform (Win7, 8, 10? macOS? Linux distro?):
  Windows 10 x64
• Brave Version:
  0.12.0 RC2 (Clean install)
• Steps to reproduce:
  1. Enable Block scripts in Shields
  2. Visit any site and click on the blocked script icon next to URL bar.
  3. Info window is off position.
• Screenshot if needed:
  
• Any related issues:

[luixxiul]

I'm guessing it should be like the bookmark hanger, which appears with the arrowUp under the bookmark star icon.

The modal background color is going to be removed since 0.18.x (#9034).

#7989

[luixxiul]

@bradleyrichter could I know if you have a mockup of the dialog? I'm wondering if it would look nicer when commonForm is applied to make it like bookmark hanger:

With the arrowUp, it should appear under the noScript icon, which is 🚫 .

[luixxiul]

It would be solved like this way:

[bradleyrichter]

Good solution but we should retain the normal control styles and perhaps reduce the size of the panel itself We are looking at global style update as well but it's of course a separate project.

…

On Sep 15, 2017, at 8:05 PM, Suguru Hirahara ***@***.***> wrote: It would be solved like this way: — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

[luixxiul]

we should retain the normal control styles and perhaps reduce the size of the panel itself

Replacing the checkboxes with switches? it would look fine.

I'd reduce the panel size and adjust it through l10n QA.

[luixxiul]

How about this one?

the wording can be improved to close #7377

also Unselect will be changed into Select.

[bsclifton]

cc: @bradleyrichter @alexwykoff @jenn-rhim @jamesmudgett