Skip to content

Conversation

@pakb
Copy link
Contributor

@pakb pakb commented Apr 28, 2025

When using a narrow layout (such as A4 portrait) the confederation text was switching to the "mobile" layout, and using a width that match only the flag.

Test link

@github-actions github-actions bot added the bug label Apr 28, 2025
@cypress
Copy link

cypress bot commented Apr 28, 2025

web-mapviewer    Run #5298

Run Properties:  status check failed Failed #5298  •  git commit 86181fd21e: PB-1522 : fix confederation text shrinking/covered in print view
Project web-mapviewer
Branch Review fix-PB-1522-print-conf-logo-text-width
Run status status check failed Failed #5298
Run duration 06m 00s
Commit git commit 86181fd21e: PB-1522 : fix confederation text shrinking/covered in print view
Committer Pascal Barth
View all properties for this run ↗︎

Test results
Tests that failed  Failures 1
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 20
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 250
View all changes introduced in this branch ↗︎

Tests for review

Failed  tests/cypress/tests-e2e/importToolMaps.cy.js • 1 failed test • e2e/chrome/mobile

View Output

Test Artifacts
The Import Maps Tool > Import external WMTS layers Test Replay Screenshots

@pakb pakb force-pushed the fix-PB-1522-print-conf-logo-text-width branch 2 times, most recently from 75f32dd to d5a9ddb Compare May 6, 2025 09:23
@pakb pakb requested a review from rebert May 7, 2025 08:16
@pakb pakb force-pushed the fix-PB-1522-print-conf-logo-text-width branch 2 times, most recently from 74e13df to 8acb40c Compare May 8, 2025 06:28
When using a narrow layout (such as A4 portrait) the confederation text was switching to the "mobile" layout, and using a width that match only the flag.
@pakb pakb force-pushed the fix-PB-1522-print-conf-logo-text-width branch from 8acb40c to 86181fd Compare May 13, 2025 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant