Skip to content

Fix search modal height in Safari#2028

Merged
delucis merged 1 commit into
withastro:mainfrom
HiDeoo:hd-fix-search-dialog-height-safari
Jun 18, 2024
Merged

Fix search modal height in Safari#2028
delucis merged 1 commit into
withastro:mainfrom
HiDeoo:hd-fix-search-dialog-height-safari

Conversation

@HiDeoo
Copy link
Copy Markdown
Member

@HiDeoo HiDeoo commented Jun 18, 2024

Description

This PR fixes an issue with the search modal height in Safari (probably this issue which would also apply for max-content and min-content) by removing the grid layout and using flexbox instead.

No search query
Chrome Firefox Safari
Before chrome_before_1 firefox_before_1 safari_before_1
After chrome_after_1 firefox_after_1 safari_after_1
No results
Chrome Firefox Safari
Before chrome_before_2 firefox_before_2 safari_before_2
After chrome_after_2 firefox_after_2 safari_after_2
Some results
Chrome Firefox Safari
Before chrome_before_3 firefox_before_3 safari_before_3
After chrome_after_3 firefox_after_3 safari_after_3
Some results with overflow
Chrome Firefox Safari
Before chrome_before_4 firefox_before_4 safari_before_4
After chrome_after_4 firefox_after_4 safari_after_4

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jun 18, 2024

🦋 Changeset detected

Latest commit: f498912

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@astrojs/starlight Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 18, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
starlight ✅ Ready (Inspect) Visit Preview Jun 18, 2024 3:04pm

@github-actions github-actions Bot added the 🌟 core Changes to Starlight’s main package label Jun 18, 2024
Copy link
Copy Markdown
Member

@delucis delucis left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for catching that @HiDeoo!

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

Labels

🌟 core Changes to Starlight’s main package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants