Closed
Bug 788598
Opened 12 years ago
Closed 12 years ago
logged out state has no login
Categories
(Firefox Graveyard :: SocialAPI, defect)
Firefox Graveyard
SocialAPI
Tracking
(firefox17+ fixed)
RESOLVED
FIXED
Firefox 18
People
(Reporter: mixedpuppy, Assigned: mixedpuppy)
References
Details
(Whiteboard: [Fx17])
Attachments
(1 file, 1 obsolete file)
5.99 KB,
patch
|
akeybl
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
if logged out of the provider, the profile section of the provider menu has a generic face and says "not logged in". This should be clickable and take the user to the top level of the provider website.
Assignee | ||
Updated•12 years ago
|
Whiteboard: [Fx17?] → [Fx17]
Comment 1•12 years ago
|
||
As going to the top-level origin may not always be the most appropriate place to land, this implies some API might be necessary to indicate where that click should go.
OS: Mac OS X → All
Hardware: x86 → All
Comment 2•12 years ago
|
||
This is an extremely confusing state and I think we need to do something about it for 17. Changing the text from "not logged in" to a link that says "log in" and takes you to the social providers front page would probably be sufficient. I think we're OK going to the top level origin for now though we may want a better solution in the future.
Assignee | ||
Comment 3•12 years ago
|
||
I created bug 789585 for future enhancement, and for now we'll just do the top-level origin for the "login" link.
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → mixedpuppy
Assignee | ||
Comment 5•12 years ago
|
||
This patch also makes the entire profile area in the menu clickable
Attachment #665553 -
Flags: review?(jaws)
Comment 6•12 years ago
|
||
Comment on attachment 665553 [details] [diff] [review] v1 Review of attachment 665553 [details] [diff] [review]: ----------------------------------------------------------------- ::: browser/base/content/browser-social.js @@ +32,4 @@ > openUILinkIn(Social.provider.profile.profileURL, "tab"); > + else { > + // bug 789585 open the top level of the provider domain, in the future > + // we will modify the api to allow a provider to define a login url This comment is a little confusing. Please replace it with, // XXX Bug 789585 will implement an API for provider-specified login pages. ::: browser/base/content/browser.xul @@ +668,5 @@ > hidden="true"> > <hbox id="social-toolbar-button-box" class="social-statusarea-container"> > <button id="social-provider-image" type="menu"> > <menupopup id="social-statusarea-popup"> > + <hbox id="social-statusarea-user" pack="left" align="center" pack="start" @@ +674,4 @@ > <image id="social-statusarea-user-portrait"/> > <vbox> > + <button id="social-statusarea-notloggedin" > + class="link">&social.notLoggedIn.label;</button> label="&social.notLoggedIn.label;"
Attachment #665553 -
Flags: review?(jaws) → review+
Assignee | ||
Comment 8•12 years ago
|
||
feedback implemented, rebased on top of another bug modifying the same area of xul, carrying forward r+
Attachment #665553 -
Attachment is obsolete: true
Assignee | ||
Updated•12 years ago
|
tracking-firefox17:
--- → ?
Keywords: checkin-needed
Assignee | ||
Comment 9•12 years ago
|
||
Comment on attachment 666115 [details] [diff] [review] V2 [Approval Request Comment] User impact if declined: poor ux in social provider menu String or UUID changes made by this patch: none
Attachment #666115 -
Flags: approval-mozilla-aurora?
Comment 10•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/09a1c6254392
Flags: in-testsuite-
Keywords: checkin-needed
Comment 11•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/09a1c6254392
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 18
Updated•12 years ago
|
Comment 12•12 years ago
|
||
Comment on attachment 666115 [details] [diff] [review] V2 Isolated changes to Social API without l10n impact, approving for Aurora 17.
Attachment #666115 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 13•12 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/53a6cfba7382
status-firefox17:
--- → fixed
Updated•5 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•