Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: misaligned page title when logo is not displayed #578

Merged
merged 1 commit into from
Feb 17, 2025

Conversation

connorlanigan
Copy link
Contributor

When the navbar is configured not to show a logo, the page title still has a small margin on the left side. When the page content is configured to have the same width as the navbar (e.g. wide), this leads to a misalignment between the left edge of the page title and the left edge of the side navigation. This PR fixes that by moving the margin onto the logo element, so that the margin is not there when the logo is not there.

Screenshots

Before

screenshot before the change, the page title is not aligned with the side navigation

After

screenshot after the change, the page title is aligned with the side navigation

Copy link

netlify bot commented Feb 15, 2025

Deploy Preview for hugo-hextra ready!

Name Link
🔨 Latest commit c1cdac0
🔍 Latest deploy log https://app.netlify.com/sites/hugo-hextra/deploys/67b0a0f3c1f5b2000855feb1
😎 Deploy Preview https://deploy-preview-578--hugo-hextra.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link
Owner

@imfing imfing left a comment

Choose a reason for hiding this comment

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

@connorlanigan thanks!

@imfing imfing changed the title fix: Misaligned page title when logo is not displayed fix: misaligned page title when logo is not displayed Feb 17, 2025
@imfing imfing merged commit ab56c66 into imfing:main Feb 17, 2025
4 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants