Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed the centering of the "Log in" button on admin login page. #12847

Merged
merged 1 commit into from Jun 26, 2020
Merged

Fixed the centering of the "Log in" button on admin login page. #12847

merged 1 commit into from Jun 26, 2020

Conversation

jdufresne
Copy link
Member

Previously, the button's container had a padding of 9.4em which was
approximately center but not precisely. Now, the container uses 'align:
text-center' to let the browser center the button.

Previously, the button's container had a padding of 9.4em which was
approximately center but not precisely. Now, the container uses 'align:
text-center' to let the browser center the button.
@carltongibson
Copy link
Member

buildbot, test on selenium.

@django django deleted a comment from jdufresne Jun 25, 2020
@carltongibson
Copy link
Member

Hey @jdufresne. I rebased this after #12846. Can you confirm it's how you'd expect it? Ta.

Copy link
Member

@carltongibson carltongibson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks! Just wait for Selenium and @jdufresne... 👍

@jdufresne
Copy link
Member Author

👍 Rebase looks good to me. Thanks for handling that.

@felixxm felixxm merged commit feb6581 into django:master Jun 26, 2020
@jdufresne jdufresne deleted the center-login branch July 1, 2020 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants