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

TailwindUI Template Spotlight Footer Overflow - Needs 'flex-wrap' #1454

Closed
djsjr opened this issue May 9, 2023 · 1 comment
Closed

TailwindUI Template Spotlight Footer Overflow - Needs 'flex-wrap' #1454

djsjr opened this issue May 9, 2023 · 1 comment
Assignees

Comments

@djsjr
Copy link

djsjr commented May 9, 2023

What version of Tailwind CSS are you using?

For example: v3.3.1

What build tool (or framework if it abstracts the build tool) are you using?

For example: Next.js latest

What version of Node.js are you using?

For example: n/a

What browser are you using?

All browsers

What operating system are you using?

macOS

Reproduction URL

https://spotlight.tailwindui.com

Describe your issue

Footer text overflows when text size too large, screen width too small, or number of links too great. Go to Spotlight example, increase page size, and narrow window to demonstrate. First noticed this on an iPhone which had slightly enlarged large text enabled on Safari for the readability of the user. Caused overflow and horizontal scroll (no good!). Added flex-wrap to innermost flex box containing page links to solve.

@djsjr djsjr changed the title TailwindUI Template Spotlight Footer - needs flex-wrap TailwindUI Template Spotlight Footer Overflow - Needs 'flex-wrap' May 9, 2023
@thecrypticace thecrypticace transferred this issue from tailwindlabs/tailwindcss May 10, 2023
@reinink reinink self-assigned this May 11, 2023
@reinink
Copy link
Member

reinink commented May 11, 2023

@djsjr Hey thanks for sharing this feedback! You're right, we could probably make the footer nav in this template a little more adaptable on mobile. I've just deployed an update to this template to improve this.

If you want to make the same changes in your own project, update the src/components/Footer.jsx file like this:

- <div className="flex gap-6 text-sm font-medium text-zinc-800 dark:text-zinc-200">
+ <div className="flex flex-wrap justify-center gap-x-6 gap-y-1 text-sm font-medium text-zinc-800 dark:text-zinc-200">

Thanks again! 🤙

@reinink reinink closed this as completed May 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants