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

Custom logo positioning issues #2133

Open
adamcnaglich opened this issue Mar 1, 2019 · 1 comment
Open

Custom logo positioning issues #2133

adamcnaglich opened this issue Mar 1, 2019 · 1 comment
Labels

Comments

@adamcnaglich
Copy link

Describe the bug
The login button has a negative top-margin that can (depending on the logo size) obscure the custom logo. Additionally, (depending on the logo size) the logo may be left-aligned, when the expectation is for it to be centered.

To Reproduce
If you upload a logo that is 300 x 200, the bottom ~40px of the logo will be covered up by the login button. If you upload a logo that is smaller than 300px wide, the logo will align to the left of that 300px container.

Expected behavior
The expectation is for the custom logo to be centered and placed above the login button.

Screenshots
screen shot 2019-03-01 at 1 16 52 pm

screen shot 2019-03-01 at 1 14 36 pm

Applicable Versions:

  • netlify-cms 2.5.1
  • Git provider: GitHub
  • OS: macOS 10.14.3
  • Chrome Version 72.0.3626.119 (Official Build) (64-bit)
  • Node.JS version: 11.10.0

CMS configuration
No can do, sorry.

Additional context
https://github.com/netlify/netlify-cms/blob/636a141a4ab3c4c9a490aea6956b32fec2324e08/packages/netlify-cms-ui-default/src/AuthenticationPage.js

@stale
Copy link

stale bot commented Oct 29, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants