Skip to content
New issue
Jump to bottom

Save camera positions US:5 #4

Closed
gautegf opened this issue Sep 17, 2025 · 0 comments · Fixed by #38
Closed

Save camera positions US:5 #4

gautegf opened this issue Sep 17, 2025 · 0 comments · Fixed by #38
Assignees
@gautegf @franmagn

Comments

@gautegf
Copy link
Member

gautegf commented Sep 17, 2025 •
edited by franmagn
Loading

User story #5
Move the annotation tool, annotation list and all the annotation properties to a separate section in the sidebar.
Add the delete button for each saved annotation.
Add the button to create a new annotation folder

  • when you create an annotation, you should be able to select in which folder you want to store it
    Add a save button to save a specific folder into a .JSON
  • in the .JSON file there should be: Name, Description, Camera Position, Camera Target and Position (-> important for traveling)
franmagn added a commit that referenced this issue Oct 13, 2025
@franmagn
style(#4): ran prettier
1b5543f
franmagn added a commit that referenced this issue Oct 13, 2025
@franmagn
style(#4): make the old annotation button invisible
f92fb4c 
add annotationPanel.css file and just inserted the css code to make the original annotation button not visible in the toolbar
franmagn added a commit that referenced this issue Oct 13, 2025
@franmagn
refactor(#4): changed texts from "annotations" to "saved locations"
052e946
franmagn added a commit that referenced this issue Oct 14, 2025
@franmagn
refactor(#4): add "Saved Point coordinates" to the list of annotations
cec8774
franmagn added a commit that referenced this issue Oct 15, 2025
@franmagn
refactor(#4): add monotonic numbering (in sidebar) to created annotat…
53343ed 
…ions

add an increasing number to each new annotation created. As for now, only in the sidebar (jstree)
franmagn added a commit that referenced this issue Oct 15, 2025
@franmagn
refactor(#4): add monotonic numbering (in live scene) to created anno…
1634ea8 
…tations

add an increasing number to each new annotation created. Now works also in the live scene (for each UUID)
franmagn added a commit that referenced this issue Oct 15, 2025
@franmagn
feat(#4): 📝 add possibility to rename the title of a saved position f…
94a7959 
…rom the sidebar

it is now possible to double-click on a saved position's name to rename it. The update is cascaded to the live-scene elements
franmagn added a commit that referenced this issue Oct 15, 2025
@franmagn
fix(#4): 🐛 address NotFoundError
939af8d 
This fix solves the NotFoundError, throwed whenever there was an attempt of removing an already removed target
franmagn added a commit that referenced this issue Oct 15, 2025
@franmagn
fix(#4): 🐛 there was a conflict with "Data for point X" table
b3acc9d 
fixed behaviour conflict triggered by "Add a new location" button
franmagn added a commit that referenced this issue Oct 15, 2025
@franmagn
fix(#4): 🐛 initially hide "Saved Point Coordinates" value and wait fo…
71614e8 
…r label placement to update

Before of this fix, whenever there was the creation of a new saved position, it had Placeholder values from potree's build, that were updated only after the creation/deletion of another position. Now plaeholder value is substituted by "---" and there is a eventListener that waits for the label to be placed before updating its values
@gautegf gautegf mentioned this issue Sep 17, 2025
Closed
franmagn added a commit that referenced this issue Oct 15, 2025
@franmagn
feat(#4): ✏️ add description in the sidebar and make it editable + sm…
8b701d3 
…all refactor

Now it is possible to see the Annotation Description in the sidebar, and if double-clicked, it can be edited. The Annotation is mirrored in the liev scene object. Lastly, there is a small refactor of the code to have some helpers
franmagn added a commit that referenced this issue Oct 15, 2025
@franmagn
refactor(#4): remove camera pivot coordinates
7d804b7
franmagn added a commit that referenced this issue Oct 15, 2025
@franmagn
style(#4): 🎨 add stylings for annotationPanel in the sidebar
4dad112
gautegf added a commit that referenced this issue Oct 20, 2025
@gautegf
feat(#4): ✨ Made a small express server for saving annotations
89d65f4
gautegf added a commit that referenced this issue Oct 20, 2025
@gautegf
feat(#4): ✨ Made a function for storing annotations
77b328c 
The data is sent by API to the small express server and stored in a json. the objects in the json will be fetched when opening the app and displayed
gautegf added a commit that referenced this issue Oct 20, 2025
@gautegf
build(#4): 🏗️ Added some dependencies
527a9af
franmagn added a commit that referenced this issue Oct 20, 2025
@franmagn
fix(#4): small fix to merge with other branch
67de0a1
gautegf added a commit that referenced this issue Oct 20, 2025
@gautegf
feat(#4): ✨ update annotation persitence to work with the UI
b626233
gautegf added a commit that referenced this issue Oct 20, 2025
@gautegf
fix(#4): 🐛 fixed import
1350936
franmagn added a commit that referenced this issue Oct 22, 2025
@franmagn
style(#4): 🎨 some small UI changes
138d58b 
- made it possible to click everywhere on the single annotation panel to open it, and not only on the toggle triangle - added a small pencil icon to indicate the possibility to edit the title text - switched “camera coordinates” with “saved point coordinates” in the annotation panel - made it that the coordinates are listed below the “type of coordinate” title, and are on different lines
franmagn added a commit that referenced this issue Oct 22, 2025
@franmagn
style(#4): 🎨 ran prettier
f0bbe5c
franmagn added a commit that referenced this issue Oct 22, 2025
@franmagn
docs(#4): 📝 improved inline documentation in the code
294bc97
gautegf added a commit that referenced this issue Oct 23, 2025
@gautegf
docs(#4): 📝 Update readme
681caeb
gautegf added a commit that referenced this issue Oct 23, 2025
@gautegf
style(#4): 💄 Update the UI for editing title and description of an an…
0b98a45 
…notation
@gautegf gautegf mentioned this issue Oct 23, 2025
Merged
@gautegf gautegf linked a pull request Oct 23, 2025 that will close this issue
4 save camera positions us5 #38
Merged
franmagn added a commit that referenced this issue Oct 23, 2025
@franmagn
docs(#4): 📝 fixed and moved inline documentation
30a89bc
franmagn added a commit that referenced this issue Oct 23, 2025
@franmagn
style(#4): fix annotation description edit box behaviour
fa8725e
franmagn added a commit that referenced this issue Oct 23, 2025
@franmagn
docs(#4): 📝 change name of two functions
6f45475
franmagn added a commit that referenced this issue Oct 23, 2025
@franmagn
refactor(#4): guard jQuery according to the scary rabbit
4795405
franmagn added a commit that referenced this issue Oct 23, 2025
@franmagn
style(#4): 🎨 ran format
b084e93
Sign in to join this conversation on GitHub.
Assignees

@gautegf gautegf

@franmagn franmagn

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

4 save camera positions us5 TDT4290-group-4/MolloyExplorer
2 participants
@gautegf @franmagn