Skip to content

fix: Improve spacing of mobile navigation menu #262

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
therobrob merged 3 commits into from
Jul 28, 2025
Merged

fix: Improve spacing of mobile navigation menu #262

therobrob merged 3 commits into from
Jul 28, 2025

Conversation

@MoritzWeber0
Copy link
Member

@MoritzWeber0 MoritzWeber0 commented Jul 27, 2025 •
edited
Loading

  • Make sure that the Close button has the same position as the Menu button in the mobile menu
  • Reduce spacing between the countries in the mobile menu (to display more countries on smaller screen), as discussed

@MoritzWeber0 MoritzWeber0 requested a review from therobrob July 27, 2025 13:47
@netlify
Copy link

netlify bot commented Jul 27, 2025 •
edited
Loading

✅ Deploy Preview for fipguide ready!

Name Link
🔨 Latest commit af2f998
🔍 Latest deploy log https://app.netlify.com/projects/fipguide/deploys/688795d43b93c100086774bf
😎 Deploy Preview https://deploy-preview-262--fipguide.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 project configuration.

@MoritzWeber0 MoritzWeber0 force-pushed the fix/close-button-positioning branch from 1ef5962 to 476fdda Compare July 27, 2025 13:49
@MoritzWeber0 MoritzWeber0 added the technical Technical issues, e.g. related to Hugo, HTML, CSS, deployment, etc. label Jul 27, 2025
@therobrob therobrob merged commit fa53a47 into main Jul 28, 2025
7 checks passed
@therobrob therobrob deleted the fix/close-button-positioning branch July 28, 2025 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@therobrob therobrob therobrob approved these changes

Assignees

No one assigned

Labels

technical Technical issues, e.g. related to Hugo, HTML, CSS, deployment, etc.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@MoritzWeber0 @therobrob