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

Logo on mobile is off center #1436

Closed
jpeacock29 opened this issue Nov 24, 2019 · 3 comments
Closed

Logo on mobile is off center #1436

jpeacock29 opened this issue Nov 24, 2019 · 3 comments

Comments

@jpeacock29
Copy link
Contributor

Describe the bug

When the logo parameter is set (in params.toml), the logo does not appear centered on mobile. See attached screenshot.

To Reproduce

Set the logo parameter to some image. Build and deploy the site and view on mobile. The logo does not appear centered.

Expected behavior

The logo is centered.

Technical details:

  • Academic Version: 4.6.1
  • Hugo Version: 0.59.1
  • Browser/OS: Checked on Android in Chrome and Firefox.

signal-attachment-2019-11-23-220802

jpeacock29 added a commit to jpeacock29/hugo-academic that referenced this issue Nov 24, 2019
@jpeacock29
Copy link
Contributor Author

Hmm, I pulled these changes but the logo now appears on the left, covering the hamburger menu (I also had to specify a smaller logo file since it wasn't scaled any more, but I assume that may be as intended):

Screen Shot 2019-11-24 at 11 51 13

Is this as intended? (Also, glad to open a new issue if preferable.)

@gcushen
Copy link
Collaborator

gcushen commented Nov 24, 2019

@jpeacock29 which browser and browser version(s) are you using where this behaviour appears? Have you tried the latest Chrome, Firefox, or Safari?

@jpeacock29
Copy link
Contributor Author

The above is from Firefox 70.0.1 (on Desktop but with a narrow window, I can't currently check on mobile proper). Chrome 78.0.3904.108 seems to have different issue at many widths, distorting the logo:

Screen Shot 2019-11-24 at 10 06 43 PM

SJamieson pushed a commit to SJamieson/hugo-academic that referenced this issue Jan 22, 2020
YoungWilliamZ pushed a commit to YoungWilliamZ/hugo-academic that referenced this issue Jul 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants