Skip to content

PB-2436: load icon images in 3D from their URL - #1602

Merged
rotwurstesser merged 1 commit into
developfrom
fix-3d-icon-billboards
Oct 14, 2026
Merged

rotwurstesser merged 1 commit into
developfrom
fix-3d-icon-billboards

Conversation

@rotwurstesser

@rotwurstesser rotwurstesser commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Fixes PB-2436

Switching to 3D with a GeoJSON layer using icons (e.g. Ladepunkte für Elektroautos) stops Cesium from rendering. On prod the map just stays black, on int/dev you see "Expected image.width to be greater than or equal to 1".

We gave Cesium the OpenLayers icon image before it was loaded (width 0). Now Cesium gets the icon URL and loads it itself.

Same view side by side, switch to 3D in both:

Test link

Test link

Test link

@github-actions github-actions Bot added the bug Something isn't working as expected label Oct 14, 2026
Switching to 3D with a GeoJSON layer using icons (e.g. Ladepunkte für
Elektroautos) stopped Cesium from rendering. We gave Cesium the OpenLayers
icon image before it was loaded (width 0). Cesium now gets the icon URL and
loads the image itself.
@rotwurstesser
rotwurstesser force-pushed the fix-3d-icon-billboards branch from 6e696bc to dc35092 Compare October 14, 2026 09:16
@cypress

cypress Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

web-mapviewer    Run #6632

Run Properties:  status check passed Passed #6632  •  git commit dc35092d92: fix: load icon images in 3D from their URL
Project web-mapviewer
Branch Review fix-3d-icon-billboards
Run status status check passed Passed #6632
Run duration 04m 58s
Commit git commit dc35092d92: fix: load icon images in 3D from their URL
Committer Raphi
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 19
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 222
View all changes introduced in this branch ↗︎

@rotwurstesser

Copy link
Copy Markdown
Contributor Author
image

@rotwurstesser rotwurstesser changed the title fix: load icon images in 3D from their URL PB-2436: load icon images in 3D from their URL Oct 14, 2026
@rotwurstesser
rotwurstesser marked this pull request as ready for review October 14, 2026 09:49

@romankaravia romankaravia left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

@rotwurstesser
rotwurstesser merged commit 23568ee into develop Oct 14, 2026
31 checks passed
@rotwurstesser
rotwurstesser deleted the fix-3d-icon-billboards branch October 14, 2026 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working as expected

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants