Closed Bug 1107590 Opened 5 years ago Closed 5 years ago

Add entry point to Site Settings from Site Identity popup

Categories

(Firefox for Android :: General, defect)

35 Branch
All
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 40
Tracking Status
firefox40 --- fixed

People

(Reporter: Margaret, Assigned: liuche)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

Now that we're showing the site identity popup for all sites, I think we should look at how we can add site settings/permissions to this popup, which would allow us to get rid of the "Site Settings" menu item.

It would then be cool to build on this in the future with even more help site identity info, such as how much data the page used to load.
(In reply to :Margaret Leibovic from comment #0)
> Now that we're showing the site identity popup for all sites

I think this is only on new tablet.
Summary: Show site settings in site identity popup → Show site settings in site identity popup on new tablet
But actually, I just filed bug 1107591 to always show the site identity popup when hitting the favicon on phones, making this relevant to all devices.
No longer depends on: 1107591
Summary: Show site settings in site identity popup on new tablet → Show site settings in site identity popup
(In reply to Michael Comella (:mcomella) from comment #2)
> But actually, I just filed bug 1107591 to always show the site identity
> popup when hitting the favicon on phones, making this relevant to all
> devices.

Great, thanks!
antlam, in the funnel review today we decided to make the site identity popup effort a P1, with a focus on improving our site settings interaction.

If you have time, it would be great to start thinking about what we could do here to increase the visibility of site settings, including giving users an easier way to change these settings (replacing the "Edit site settings" menu item).
Flags: needinfo?(alam)
Definitely! From what I understand, something like this could potentially be a good starting point as well. 

Will keep noodling on this.
Flags: needinfo?(alam)
(In reply to Anthony Lam (:antlam) from comment #5)
> Created attachment 8562249 [details]
> prev_siteID_update_mock2.png
> 
> Definitely! From what I understand, something like this could potentially be
> a good starting point as well. 
> 
> Will keep noodling on this.

Ooh, love that simple launch point. This wouldn't be too hard to add, and it would be a nice improvement.
Assignee: nobody → liuche
Let's get a few more specs on this:

- Title should be favicon + url
- "Secure Connection" instead of "Encrypted"?
- Any other copy changes?

Also:
- What does the non-encrypted version look like?

Can you attach the lock icon resources (and also whatever icon we need for non-encrypted)?
Flags: needinfo?(alam)
Actually, let's simplify this bug so that it's just to add the link to "Edit site settings". We can style the rest of the site settings doorhanger in a different bug - that's scope creeping this bug a lot!
Flags: needinfo?(alam)
Summary: Show site settings in site identity popup → Add entry point to Site Settings from Site Identity popup
Attached file MozReview Request: bz://1107590/liuche (obsolete) —
/r/7409 - Bug 1107590 - Add entry point to Site Settings from Site Identity popup. r=ally

Pull down this commit:

hg pull -r f4e328b35a8dcf0dacb6a6d383c82d66b0c0affd https://reviewboard-hg.mozilla.org/gecko/
Attachment #8595588 - Flags: review?(ally)
Comment on attachment 8595588 [details]
MozReview Request: bz://1107590/liuche

https://reviewboard.mozilla.org/r/7407/#review6237

Ship It!
Attachment #8595588 - Flags: review?(ally) → review+
https://hg.mozilla.org/mozilla-central/rev/c9ddb9235baf
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 40
Comment on attachment 8595588 [details]
MozReview Request: bz://1107590/liuche

/r/7409 - Bug 1107590 - Make Site Identity text styling consistent with other doorhangers. r=ally
/r/7443 - Bug 1126608 - Add Doorhanger:Logins message from Gecko. r=ally
/r/8071 - Bug 1126608 - Add "Select" LoginDoorhanger to SiteIdentityPopup. r=ally
/r/8297 - Bug 1126608 - Add buttons. r=ally

Pull down these commits:

hg pull -r 07b2cd024fdbbcd13f991808fce9b27a5723f546 https://reviewboard-hg.mozilla.org/gecko/
Attachment #8595588 - Flags: review+
Comment on attachment 8595588 [details]
MozReview Request: bz://1107590/liuche

I apparently copy-pasted the wrong bug number for the first commit, and now it's forever linked to this bug. I'm going to obsolete this review request, so that it causes less confusion.
Attachment #8595588 - Attachment is obsolete: true
Blocks: 1174366
You need to log in before you can comment on or make changes to this bug.