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

Additional marker/icon symbolizers are misplaced in 3D view #9431

Closed
1 task done
allyoucanmap opened this issue Sep 12, 2023 · 3 comments
Closed
1 task done

Additional marker/icon symbolizers are misplaced in 3D view #9431

allyoucanmap opened this issue Sep 12, 2023 · 3 comments
Assignees
Labels
3D All issues related to the 3D rendering in CesiumJs Accepted bug C027-COMUNE_FI-2023-SUPPORT investigation
Milestone

Comments

@allyoucanmap
Copy link
Contributor

Description

Found in #9397 (review)

The additional marker and icon symbolizer are not correctly displaced because of the use of billboard.eyeOffset property (eg. as polyline start/end points). We introduced the eyeOffset property as a way to order multiple billboard applied to the same feature. Inside this comment #9362 (comment) there are other information about the use of eyeOffset including a link to a Cesium issue still open about to introduce the possibility to add a zIndex to entities.

We need to investigate and understand if it possible to replace eyeOffset or completely remove it because it seems it causing more problems rather than benefits

How to reproduce

additional-symbolizers-issue-3d.mp4
  • create an line annotation in 3D on top of 3D feature
  • add a style icon to the annotation line as starting point

Expected Result

The icon is visible at the correct location
image

Current Result

image

The icon is misplaced behind the feature

  • Not browser related
Browser info (use this site: https://www.whatsmybrowser.org/ for non expert users)
Browser Affected Version
Internet Explorer
Edge
Chrome
Firefox
Safari

Other useful information

@tdipisa tdipisa added this to the 2023.02.01 milestone Sep 13, 2023
@tdipisa tdipisa added investigation 3D All issues related to the 3D rendering in CesiumJs labels Sep 13, 2023
@tdipisa tdipisa assigned allyoucanmap and unassigned tdipisa Nov 3, 2023
@tdipisa tdipisa modified the milestones: 2023.02.01, 2023.02.02 Nov 3, 2023
@tdipisa tdipisa assigned dsuren1 and unassigned mahmoudadel54 Jan 9, 2024
@dsuren1
Copy link
Contributor

dsuren1 commented Jan 11, 2024

@tdipisa
Unable to reproduce this issue anymore probably changes made as part of #9807 handled this case, as I don't see use of eyeOffset as zIndex
image

@allyoucanmap Could you please confirm. Thanks!

@allyoucanmap
Copy link
Contributor Author

@dsuren1 yes, probably in that refactor for extrusion I removed completely the eyeOffset

@tdipisa
Copy link
Member

tdipisa commented Jan 11, 2024

Thank you @dsuren1 and @allyoucanmap. Moving to Test to verify again also with @ElenaGallo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3D All issues related to the 3D rendering in CesiumJs Accepted bug C027-COMUNE_FI-2023-SUPPORT investigation
Projects
None yet
Development

No branches or pull requests

5 participants