Person description appears under hackergotchi in Gecko

Bug #93765 reported by Nafallo Bjälevik
4
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Low
Elliot Murphy

Bug Description

In Gecko browsers (but not KHTML or Presto), <https://beta.launchpad.net/~nafallo> has the person description _under_ the image instead of _beside_ it.

It may make sense to fix bug 103114 at the same time as this bug.

Revision history for this message
James Henstridge (jamesh) wrote :

Looking at the page source, we have:

  <div style="float: right; margin: 0 0 0.5em 0.5em"><img alt="" class="mugshot" src="https://librarian.launchpad.net/981881/minigotchi.png" /></div>

The intent here is to float the hackergotchi to the left as you say. It seems that the wrapping div is actually the full width of the content area though, so the text starts below.

For people who have not set a hackergotchi, the HTML is:

    <img alt=""
         style="float: right; margin: 0 0 0.5em 0.5em"
         src="/@@/person-mugshot" />

This does get rendered as expected. So it'd be good to get rid of the <div> in the first case.

Changed in launchpad:
importance: Undecided → Low
status: Unconfirmed → Confirmed
description: updated
Revision history for this message
James Henstridge (jamesh) wrote :

Perhaps do the styling based on the "mugshot" class, and add class="mugshot" to the default mugshot. This would give consistent results for the two cases, and remove the need for inline CSS here.

Revision history for this message
Elliot Murphy (statik) wrote :

trying to fix this after discussion with mpt

Changed in launchpad:
assignee: nobody → emurphy
description: updated
Revision history for this message
Diogo Matsubara (matsubara) wrote :

Seems fixed. The description appears beside the image now.

Changed in launchpad:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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