Skip to content

site: improve header display on small devices #21260

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Oct 29, 2024
Merged

site: improve header display on small devices #21260

merged 2 commits into from
Oct 29, 2024

Conversation

@dvdksn
Copy link
Contributor

@dvdksn dvdksn commented Oct 28, 2024 •
edited
Loading

  • site: move theme switch to footer
  • site: use condensed header logo on small screens

@dvdksn dvdksn requested a review from a team October 28, 2024 15:05
@netlify
Copy link

netlify bot commented Oct 28, 2024 •
edited
Loading

✅ Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 93f816e
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/671fa8247033650008d4a885
😎 Deploy Preview https://deploy-preview-21260--docsdocker.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.

@github-actions github-actions bot added the hugo Updates related to hugo label Oct 28, 2024
@dvdksn dvdksn changed the title site: fix header on small devices site: improve header display on small devices Oct 28, 2024
sarahsanders-docker
sarahsanders-docker approved these changes Oct 28, 2024
View reviewed changes
Copy link
Contributor

@sarahsanders-docker sarahsanders-docker left a comment

Choose a reason for hiding this comment

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

Verified on my iPhone, LGTM

@dvdksn dvdksn merged commit 2e0d0fd into docker:main Oct 29, 2024
14 checks passed
@dvdksn dvdksn deleted the fix-site-header-sm-screens branch October 29, 2024 09:08
@dvdksn dvdksn mentioned this pull request Oct 29, 2024
Closed
1 task
@thaJeztah
Copy link
Member

thaJeztah commented Oct 29, 2024

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@sarahsanders-docker sarahsanders-docker sarahsanders-docker approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

hugo Updates related to hugo status/review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dvdksn @thaJeztah @sarahsanders-docker