Footer is wider than the rest of the page

RESOLVED FIXED in Bugzilla 2.18

Status

()

defect
P2
normal
RESOLVED FIXED
16 years ago
7 years ago

People

(Reporter: henrik, Assigned: gerv)

Tracking

unspecified
Bugzilla 2.18
Dependency tree / graph
Bug Flags:
approval +

Details

Attachments

(3 attachments, 1 obsolete attachment)

After the recent "upgrade" of b.m.o I have now got 70 queries in the footer.
However the lines  breaks are such that the footer is wider than the rest of the
page.

I believe this to be caused by either the number of queries or the way the
"logout: <email addr>" formatting. I will attach a screenshot
This shows the probem, as seen only half the footer is shown.
Problem is slightly bettered if I close the sidebar.
Henrik: could you do a Save Page As on a sample page with your footer, and
attach the resulting HTML?

Gerv
Assignee: myk → gerv
Priority: -- → P2
Target Milestone: --- → Bugzilla 2.18
The main problem is that the second column of the first table row in the footer
(everything from "New" to the email address) has HTML attribute nowrap="nowrap".

If you want to see this problem, just make your window ~730 pixels wide (as in
the screenshot).  But then, a lot of the web looks broken at 730 pixels...
Posted patch Patch v.1Splinter Review
This patch rearranges the footer a bit so that it resizes more gracefully, and
is organised a bit more logically.

Gerv
Posted image Screenshot of new footer (obsolete) —
And here's what it looks like on my local test installation.

Gerv
Comment on attachment 140443 [details] [diff] [review]
Patch v.1

Myk: what do you think?

Gerv
Attachment #140443 - Flags: review?(myk)
linkinfooter is coming back per bug 232176, and nowrap has been around since
before b.m.o last upgraded, so removing this from the list of regressions and
making it dependent on 232176.
No longer blocks: bmo-regressions-old
Depends on: 232176
Comment on attachment 140443 [details] [diff] [review]
Patch v.1

(UI review only)

Looks pretty good, but "prefs" should move to the first line so the average
user doesn't have to see the second line of "edit" links at all.
Attachment #140443 - Flags: review?(myk)
Myk: I actually think the extra line serves a useful purpose as a buffer between
the action links and the saved searches. Otherwise, the thing gets to be a bit
of a forest where it's hard to see what's wanted.

(To fix that for admins, actually, maybe we should style saved searches
differently...)

Gerv
Attachment #140443 - Flags: review?(justdave)
Dave: this review request is still valid :-)

Gerv
Comment on attachment 140443 [details] [diff] [review]
Patch v.1

I don't think SanityCheck belongs on the Edit line.

I also think all of the admin stuff should disappear out of the footer
entirely, and be replaced with a link to a page with an overview of the
administrative tools.  But that's a task for another bug. :)
Posted patch Patch v.2Splinter Review
Moves Sanity Check to the top line.

Gerv
Attachment #140443 - Attachment is obsolete: true
Attachment #140444 - Attachment is obsolete: true
Attachment #140443 - Attachment is obsolete: false
Attachment #140443 - Flags: review?(justdave)
Attachment #144213 - Flags: review?(justdave)
Comment on attachment 144213 [details] [diff] [review]
Patch v.2

This works for me :)

re-wraps nicely when you resize the window, too.  My only nit is that "My
Votes" gets split onto both lines if you resize the window just right.	I don't
think it's a big deal though and you're welcome to not fix it :)
Attachment #144213 - Flags: review?(justdave) → review+
Comment on attachment 144213 [details] [diff] [review]
Patch v.2

>Index: template/en/default/global/useful-links.html.tmpl
>         [% IF user && Param('usevotes') %]
>           | <a href="votes.cgi?action=show_user">My Votes</a>

Yeah, missed an &nbsp; here.
Fixed.

Checking in template/en/default/global/useful-links.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/global/useful-links.html.tmpl,v
 <--  useful-links.html.tmpl
new revision: 1.24; previous revision: 1.23
done

Gerv
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.