Closed Bug 232338 Opened 17 years ago Closed 17 years ago
Footer is wider than the rest of the page
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...
This patch rearranges the footer a bit so that it resizes more gracefully, and is organised a bit more logically. Gerv
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
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.
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.
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
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. :)
Moves Sanity Check to the top line. Gerv
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+
17 years ago
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 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: 17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.