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

fix: add viewport-fix=cover to index.html as well #20943

Merged
merged 1 commit into from
Feb 3, 2025

Conversation

vursen
Copy link
Contributor

@vursen vursen commented Feb 3, 2025

Description

It turns out that viewport-fix=cover also needs to be added to index.html, in addition to @Viewport annotations, which were addressed in #20836.

Fixes vaadin/web-components#8449

Type of change

  • Bugfix

@vursen vursen changed the title fix: add viewport-fix=cover to index.html fix: add viewport-fix=cover to index.html as well Feb 3, 2025
Copy link

github-actions bot commented Feb 3, 2025

Test Results

1 165 files  ± 0  1 165 suites  ±0   1h 39m 36s ⏱️ - 1m 54s
7 639 tests ± 0  7 582 ✅ ± 0  57 💤 ±0  0 ❌ ±0 
7 999 runs  +55  7 933 ✅ +49  66 💤 +6  0 ❌ ±0 

Results for commit fa5524a. ± Comparison against base commit 21a4145.

@mshabarov mshabarov merged commit fbff8ae into main Feb 3, 2025
26 of 34 checks passed
@mshabarov mshabarov deleted the add-viewport-fit-cover-to-index-html branch February 3, 2025 14:07
vaadin-bot pushed a commit that referenced this pull request Feb 3, 2025
It turns out that viewport-fix=cover also needs to be added to index.html, in addition to @Viewport annotations, which were addressed in #20836.

Fixes vaadin/web-components#8449
vaadin-bot added a commit that referenced this pull request Feb 3, 2025
It turns out that viewport-fix=cover also needs to be added to index.html, in addition to @Viewport annotations, which were addressed in #20836.

Fixes vaadin/web-components#8449

Co-authored-by: Sergey Vinogradov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AppLayout marginals look broken on iPhone 15
3 participants