-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Description
Copied from #7249 (comment)
OK, here's my review opinion. I'm adding it as a comment since it's not breaking issues, but I would like to see these fixed:
-
Logo is too high up; actually, its distance from top is OK, but now it's too far from the
formitself:

I would remove 16px top+bottom margins fromform {}in guest.css since they don't seem needed -
Is there a reason why
footerhasheight: 70px? We can easily remove it (I think) and change thep.info {margin: 0 auto; padding-top: 20px;to simplerp.info {margin: 20px auto;}Added advantage to these changes would mean that the whole page would fit nicely on 320*480 screen:

-
"Wrong password " warning is above the
#submit# button, but the#lost-password` warning is bellow - makes no sense at all:

-
Additionally, "Lost Password" warning also has
opacity: .7;added to it for some reason.