Not possible to change a person's logo anymore

Bug #262739 reported by Guilherme Salgado
22
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Low
Unassigned

Bug Description

It used to be possible to change one's logo but that logo wasn't actually displayed anywhere. The new maps introduced as part of the fix for bug 1261 are now using the logo, but now one can't change it anymore.

We need to re-add the ability to change one's logo to the +edit page.

Tags: lp-registry
Changed in launchpad:
assignee: nobody → salgado
importance: Undecided → Medium
milestone: none → 2.1.9
status: New → Triaged
Revision history for this message
Matthew Paul Thomas (mpt) wrote :

I suggest making the map use a resized version of the mugshot (like Facebook does, for example), instead of a separate image. I think it's unreasonable to require people to upload their photo in two different sizes.

Revision history for this message
Guilherme Salgado (salgado) wrote : Re: [Bug 262739] Re: Not possible to change a person's logo anymore

I like your suggestion, Matthew. I wonder, though, if it'd be okay to
generate the logo on the fly from the mugshot or if we must do that
before (probably at the time the user changes their mugshot).

Revision history for this message
Matthew Paul Thomas (mpt) wrote :

If the map was the only place the small version was going to be used, it wouldn't really matter if you did it on the fly, because the map shows only one person's picture at a time, so a little bit of wait for the image then wouldn't matter much.

However, a team's gallery page <https://edge.launchpad.net/~bugsquad/+mugshots> probably should use a small version too, to make the page faster and more compact for large teams. In that case, the small images should be pre-generated, otherwise that page would be extremely slow.

So, I suggest generating the small version (ideally with Lanczos resampling) whenever the large version is changed.

Revision history for this message
Nizar Kerkeni (nizarus) wrote :

I agree to Matthew suggestion, and we can give the choice to the user, for example :
1) use a resized version of the Mugshot
2) upload a new logo.
In the two cases the logo in the map must be updated after Mugchot or logo update which is actually not done.

Curtis Hovey (sinzui)
Changed in launchpad-foundations:
milestone: 2.1.9 → 2.1.10
Curtis Hovey (sinzui)
Changed in launchpad-foundations:
milestone: 2.1.10 → none
Changed in launchpad-registry:
milestone: none → 2.1.10
Revision history for this message
Curtis Hovey (sinzui) wrote :

I think we are discussing a new feature here. There is no simple fix for this. I'm resizing the images to keep them synced.

Changed in launchpad-registry:
milestone: 2.1.10 → none
Changed in launchpad-registry:
assignee: salgado → nobody
Curtis Hovey (sinzui)
Changed in launchpad-registry:
importance: Medium → Low
Curtis Hovey (sinzui)
Changed in launchpad-registry:
milestone: none → 3.0
Curtis Hovey (sinzui)
Changed in launchpad-registry:
milestone: 3.0 → 3.1.10
Revision history for this message
Nizar Kerkeni (nizarus) wrote :

With Launchpad 2.2.6 (r9503) beta site it's possible to change the logo.

Revision history for this message
Nizar Kerkeni (nizarus) wrote :

Sorry it's Launchpad 2.2.7 (r9503) version

Revision history for this message
Curtis Hovey (sinzui) wrote :

The icon is still missing. The icon is used on maps.

Revision history for this message
Nizar Kerkeni (nizarus) wrote :

in https://edge.launchpad.net/~username/+branding page there is Logo and Mugshot images. I updated my logo image and it's used in the map.

Revision history for this message
Curtis Hovey (sinzui) wrote :

I see I am mistaken.

Curtis Hovey (sinzui)
Changed in launchpad-registry:
milestone: 3.1.10 → 3.0
milestone: 3.0 → 2.2.8
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Related questions

Remote bug watches

Bug watches keep track of this bug in other bug trackers.