Merge sprint 3 changes into main #63
Merged
+2,147
−129
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pruned the table with all the values related to the point selected, in order to delete all unused and useless attributes while keeping origianl behaviour
Write out name of each measurement tool and move measurements and saved positions above appearance
…group-4/MolloyExplorer into 48-make-sidebar-more-intuitive
…group-4/MolloyExplorer into 48-make-sidebar-more-intuitive
15 show compass by default
fixed a graphic bug that made not possible to see x,y and z coordinates on other measurement tools sections in the sidebar, and that showed Latitude, Longitude and Elevation in those sections
…into 47-improve-inspected-point-visualization
…bar-more-intuitive
…ud on the globe add a simple label with the name of the measured object, taken from the sidebar measurement list. It automatically gets removed once the measured object is removed, and it moves with the moving of the points.
add background to make the labels visible even while displaying black or white datasets
… 18-write-the-readme
now it is not possible to see both name labels and value labels of the measurements, and it's possible to switch between these visualization thanks to two new buttons. There is also the possibility to hide both with a third button, all in the same section of the sidebar
fixed behaviour after deleting measurements. Now UI is immediately refreshed, not needing anymore to move the camera position to see the labels of deleted points disappear
…t-visualization 47 improve inspected point visualization
…ls-tabbable-for-accessibility 50 make navigation controls tabbable for accessibility
removed a note in text
…creenshots to public/img #18
…loyExplorer into 18-write-the-readme
18 write the readme
…a-animations-show-up fix(#61): 🐛 Ensure potreeViewer is always enabled
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. Comment |
adriahso
approved these changes
Nov 10, 2025
Sign in
to join this conversation on GitHub.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Merge dev into main. Merging sprint 3 changes into main