Skip to content

chore: less HTML, JS, & CSS for search modal - #314

Merged
flying-sheep merged 2 commits into
mainfrom
html-modal
Sep 15, 2026
Merged

flying-sheep merged 2 commits into
mainfrom
html-modal

Conversation

@flying-sheep

@flying-sheep flying-sheep commented Sep 15, 2026

Copy link
Copy Markdown
Member

Modals exist in HTML

  • they’re already centered
  • they already have a backdrop
  • autofocus exists to focus the input element

only difference: the search modal is initially vertically centered and vertically adapts to its contents on search

@netlify

netlify Bot commented Sep 15, 2026

Copy link
Copy Markdown

Deploy Preview for jade-cajeta-1bcca0 ready!

Name Link
🔨 Latest commit df47d49
🔍 Latest deploy log https://app.netlify.com/projects/jade-cajeta-1bcca0/deploys/6aa912c4c7c4900008f8e35a
😎 Deploy Preview https://deploy-preview-314--jade-cajeta-1bcca0.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.

@flying-sheep
flying-sheep merged commit a92f794 into main Sep 15, 2026
6 checks passed
@flying-sheep
flying-sheep deleted the html-modal branch September 15, 2026 09:42
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

Successfully merging this pull request may close these issues.

1 participant