URL where problem occurs:
The logo on the login page is positioned a bit awkwardly. Flash messages can overlap it and be hard to read (particularly the activation message, which sometimes appears despite ), which won't be 100% fixed by this (text can always wrap onto multiple lines), but it's a good reason to tweak it anyway... as is the fact the CSS background property isn't set quite right.
What should happen:
The logo should move down and right a bit – 2.5em should be good, since the right and left margins on the page are 2.5em.
Check the layout on big, midsize (e.g. tablet in portrait mode), and small (e.g. phone) screens. (Small screens shouldn't display an image.)
My pull request sets it to 0.5em to the right and 3em down; I found that any further to the right made it overlap the login text on mobile screens, and it needed to be at least 3em down to avoid the banner. I also used the background-position property as the dimensions weren’t recognised as part of the background property. The url is $devinstance/users/login (with the Vagrant dev setup, this defaults to: http://127.0.0.1:7000/users/login ).
“I also used the background-position property as the dimensions weren’t recognised as part of the background property.” – actually this was just me failing at css; it works fine if you remember to delete the existing ‘top left’ dimensions first…
It’s definitely lower, and a tiny bit farther right, but yeah, we can’t really do much about the horizontal positioning without running into the text.
It barely avoids a 1-line error message. It overlaps if the message is multi-line / the screen is narrower. I don't think the extra space up top improves it much, but I like the extra left space.