Bug 788589. Use information icon for logged out users and don't show generic buddy icon when logged in. r=felipe ui-r=Boriss

This commit is contained in:
Shane Caraveo 2012-09-25 11:39:09 -07:00
parent 47adc85527
commit 2423a06338
5 changed files with 5 additions and 7 deletions

View File

@ -332,7 +332,7 @@ let SocialShareButton = {
if (profile && profile.displayName) {
profileRow.hidden = false;
let portrait = document.getElementById("socialUserPortrait");
portrait.setAttribute("src", profile.portrait || "chrome://browser/skin/social/social.png");
portrait.setAttribute("src", profile.portrait || "chrome://global/skin/icons/information-32.png");
let displayName = document.getElementById("socialUserDisplayName");
displayName.setAttribute("label", profile.displayName);
} else {
@ -540,7 +540,7 @@ var SocialToolbar = {
// response. In that case we'll be called again when it's available, via
// social:profile-changed
let profile = Social.provider.profile || {};
let userPortrait = profile.portrait || "chrome://browser/skin/social/social.png";
let userPortrait = profile.portrait || "chrome://global/skin/icons/information-32.png";
document.getElementById("social-statusarea-user-portrait").setAttribute("src", userPortrait);
let notLoggedInLabel = document.getElementById("social-statusarea-notloggedin");

View File

@ -56,8 +56,6 @@ var tests = {
testProfileUnset: function(next) {
Social.provider.updateUserProfile({});
// check dom values
let portrait = document.getElementById("social-statusarea-user-portrait").getAttribute("src");
is(portrait, "chrome://browser/skin/social/social.png", "portrait is generic");
let userButton = document.getElementById("social-statusarea-username");
ok(userButton.hidden, "username is not visible");
let ambience = document.getElementById("social-status-iconbox").firstChild;

View File

@ -2702,7 +2702,6 @@ html|*#gcli-output-frame {
height: 32px;
border-radius: 2px;
margin: 10px;
list-style-image: url("chrome://browser/skin/social/social.png");
}
#social-statusarea-username {
@ -2713,6 +2712,7 @@ html|*#gcli-output-frame {
cursor: pointer;
min-width: 0;
margin: 0 6px;
list-style-image: none;
}
#social-statusarea-username:hover {
text-decoration: underline;

View File

@ -3382,7 +3382,6 @@ html|*#gcli-output-frame {
width: 32px;
height: 32px;
margin: 10px;
list-style-image: url("chrome://browser/skin/social/social.png");
}
#social-statusarea-username {
@ -3391,6 +3390,7 @@ html|*#gcli-output-frame {
cursor: pointer;
min-width: 0;
margin: 0 6px;
list-style-image: none;
}
#social-statusarea-username:hover {

View File

@ -3401,7 +3401,6 @@ html|*#gcli-output-frame {
height: 32px;
border-radius: 2px;
margin: 10px;
list-style-image: url("chrome://browser/skin/social/social.png");
}
#social-statusarea-username {
@ -3412,6 +3411,7 @@ html|*#gcli-output-frame {
cursor: pointer;
min-width: 0;
margin: 0 6px;
list-style-image: none;
}
#social-statusarea-username:hover {
text-decoration: underline;