Admin Footer overlaps other content in some browsers

[caesarsgrunt]

Caused by negative top margin. Patch attached. Only tested in Safari and Firefox.

[caesarsgrunt]

Remove negative -75px top margin on #footer

[caesarsgrunt]

Remove negative -75px top margin on #footer

[azaozz]

That would make short pages to always have a vertical scrollbar, in all browsers. There are other glitches with short pages too, the base layout needs fixing in general.

[caesarsgrunt]

Yes, you're right, sorry. I'll have another look and see if I can come up with another fix.

Actually, I always have had a vertical scrollbar on short pages, which scrolls just a few pixels. It irritates me, though it's true that it means the page doesn't jump horizontally. Setting overflow-x: scroll; overflow-y: hidden; is a nicer solution in those browsers which support it, IMO, since the scrollbar is then disabled on short pages.

[caesarsgrunt]

BTW - not relevant to this ticket, but...
This is the second path I've had which has appeared a second time a few minutes after I uploaded it, even though after the successful first upload I look at the page and only one is there, and then I go away, losing the tab altogether. I had a trac ticket created twice yesterday, too.

[caesarsgrunt]

Seems it applies to all other long dashboard pages, too.

[DD32]

Got the same issue in Opera 9.52 here. Applying that patch fixes it for Opera too.

[caesarsgrunt]

This second patch may be better - doesn't cause scrollbars on short screens.

Tested in :

• Firefox,
  
• Safari,
  
• Opera (in which there are other bugs in the footer too, BTW),
  
• IE 7 (Yuck. Has a horizontal scrollbar...)

Didn't cause any problems in any of the above. Caused a slight gap below the content on long pages in IE, and a 75px gap in Fx - but I don't think these really matter.

Try it out, people...

---

Note :
The whole admin CSS needs redoing really, IMO. (Help, ​Dean J Robinson!)
It would be nice to cut out the JS which lengthens the page on short pages, somehow. I'll look at how Fluency does it.

[caesarsgrunt]

Possibly better patch, still hopefully temporary...

[azaozz]

The js that stretches the page was a temp fix for Crazyhorse (like some other places). You're right, the base css needs redoing, the IE css too. It has to be fluid both horizontally and vertically.

[azaozz]

(In [8785]) Base css layout, fixes #7620