588689 - Add access to site preferences from the site identity block

Status: RESOLVED WONTFIX

(Firefox :: General, defect)

Description

[Alex Faaborg [:faaborg] (Firefox UX)]

Once site preferences lands, we should hook up access from the site identity block.

Mockup: https://bugzilla.mozilla.org/attachment.cgi?id=467310

Comment 1

[Mehdi Mulani [:mmm] (I don't check this)]

Attachment: Initial patch.

Small patch that replaces the "More information..." -> More Info panel call to cause a click on "More information..." to open up Site Preferences with the site selected.

This does not handle the UI change yet.

Comment 2

[:Gavin Sharp [email: [e-mail]]]

Comment on attachment 467430 [details] [diff] [review]
Initial patch.

>diff --git a/browser/base/content/utilityOverlay.js b/browser/base/content/utilityOverlay.js

>+function openWebSitePreferences() {
>+  let host = makeURI(window.content.document.location, null, null).host;

This will fail for non-nsIStandardURL nsIURIs (e.g. about: URIs). Just use content.document.location.hostname (it's safe since http://hg.mozilla.org/mozilla-central/rev/fc754baf4c59 ).

Comment 3

[Mehdi Mulani [:mmm] (I don't check this)]

(In reply to comment #2)
> This will fail for non-nsIStandardURL nsIURIs (e.g. about: URIs). Just use
> content.document.location.hostname (it's safe since
> http://hg.mozilla.org/mozilla-central/rev/fc754baf4c59 ).

Oh sweet, I was looking for a way to do this nicely.
Thanks.

Comment 4

[:Margaret Leibovic]

I'll take over this bug, since I'm working on bug 573176.

Comment 5

[:Margaret Leibovic]

I'm not actively working on this, so un-assigning myself. Also, this needs some UX direction before it can go forward.

Comment 6

[Florian Bender]

Chrome added something like this in the meantime: You can see and change *any* site-specific pref when clicking the identity icon. 

The mockup in comment 0 is pretty cool too. 

Can we have a combination of both? I. e. one can easily access and change the site specific prefs from the Site Identity doorhanger panel (e. g. "hidden" behind a tab like Chrome does it or an accordion) AND highlight specific prefs (as proposed by the mockup, plus possibly in the complete list) the website requested (e. g. the last 2 requested prefs).

Comment 7

[Florian Bender]

Sorry, I missunderstood the scope of this bug. 

Do you think a chrome-style site specific pref pane is worth considering?

Comment 8

[Dão Gottwald [:dao]]

about:permissions has been removed in bug 933917.