Skip to content

PB-2227: force icon width in map buttons#1572

Merged
pakb merged 1 commit intodevelopfrom
fix-PB-2227-no-icons-in-map-buttons
Feb 24, 2026
Merged

PB-2227: force icon width in map buttons#1572
pakb merged 1 commit intodevelopfrom
fix-PB-2227-no-icons-in-map-buttons

Conversation

@pakb
Copy link
Contributor

@pakb pakb commented Feb 24, 2026

for unknown reasons they shrink to a width of zero when the project is bundled, but it works fine when served locally (without going through the build:prod target)

So I've forced the width the same we are forcing the height of the underlying FontAwesome icon

Test link

for unknown reasons they shrink to a width of zero when the project is bundled, but it works fine when served locally (without going through the build:prod target)

So I've forced the width the same we are forcing the height of the underlying FontAwesome icon
@github-actions github-actions bot added the bug label Feb 24, 2026
@cypress
Copy link

cypress bot commented Feb 24, 2026

web-mapviewer    Run #6484

Run Properties:  status check passed Passed #6484  •  git commit 88cb5d99a6: PB-2227: force icon width in map buttons
Project web-mapviewer
Branch Review fix-PB-2227-no-icons-in-map-buttons
Run status status check passed Passed #6484
Run duration 06m 37s
Commit git commit 88cb5d99a6: PB-2227: force icon width in map buttons
Committer Pascal Barth
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
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 261
View all changes introduced in this branch ↗︎

@pakb pakb requested a review from sommerfe February 24, 2026 13:29
@pakb pakb merged commit 9a93f9f into develop Feb 24, 2026
6 checks passed
@pakb pakb deleted the fix-PB-2227-no-icons-in-map-buttons branch February 24, 2026 13:49
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.

2 participants