Skip to content
New issue
Jump to bottom

Point Filtering #8

Closed
gautegf opened this issue Sep 17, 2025 · 2 comments · Fixed by #39
Closed

Point Filtering #8

gautegf opened this issue Sep 17, 2025 · 2 comments · Fixed by #39
Assignees
@95f5ca5b369f78f4fc2a22efa554cf068195794

Comments

@gautegf
Copy link
Member

gautegf commented Sep 17, 2025 •
edited by 95f5ca5b369f78f4fc2a22efa554cf068195794
Loading

User story #11
Add filtering for points that are:

  • "accepted"/not.
  • TVU
  • THU
  • source ID

Should be able to handle switching between different filters without a massive rerender.
@gautegf gautegf mentioned this issue Sep 17, 2025
Closed
@95f5ca5b369f78f4fc2a22efa554cf068195794
Copy link
Member

Current suggestion:

  • slider for <= TVU and THU with an AND/OR option
  • radio group for accepted/not accepted/all
  • slider for source ID (a bit unsure about this one)

95f5ca5b369f78f4fc2a22efa554cf068195794 pushed a commit that referenced this issue Oct 13, 2025
feat(#8): ✨ Made functionaliry for Accepted filtering
e2b778c 
Created a separate panel for accepted filtering. When the button is clicked the colors of the points are either black for not accepted and white for accepted points. The panel also have a description for this.
95f5ca5b369f78f4fc2a22efa554cf068195794 pushed a commit that referenced this issue Oct 13, 2025
perf(#8): ⚡ Optimize elevation control and accepted filtering
eae6463 
Changes styles, hooks, css, scrolling, visuals to make it a better user experience
@95f5ca5b369f78f4fc2a22efa554cf068195794
Copy link
Member

Todo next time for Elevation control and accepted filtering:

  • clean the code
  • write tests

95f5ca5b369f78f4fc2a22efa554cf068195794 pushed a commit that referenced this issue Oct 14, 2025
style(#8): 💡 Update the comments and clean the code
3b79c44
95f5ca5b369f78f4fc2a22efa554cf068195794 pushed a commit that referenced this issue Oct 14, 2025
test(#8): 🧪 Made tests for elevation control and accepted filtering
76a18d8
95f5ca5b369f78f4fc2a22efa554cf068195794 pushed a commit that referenced this issue Oct 22, 2025
fix(#8): 🐛 Elevation needs to be activated upon initiation
7fcbf1e
95f5ca5b369f78f4fc2a22efa554cf068195794 pushed a commit that referenced this issue Oct 22, 2025
feat(#8): ✨ Filter based on acceted/not accepted
9ef64a3 
Refactor: moved elevation panel into same file as accepted panel for structural purposes
95f5ca5b369f78f4fc2a22efa554cf068195794 pushed a commit that referenced this issue Oct 23, 2025
revert(#8): 🔥 Remove elevation control and accepted filtering files a…
2ea2c33 
…s they now exist in new joined file
Merged
@95f5ca5b369f78f4fc2a22efa554cf068195794 95f5ca5b369f78f4fc2a22efa554cf068195794 linked a pull request Oct 23, 2025 that will close this issue
8 point filtering #39
Merged
95f5ca5b369f78f4fc2a22efa554cf068195794 pushed a commit that referenced this issue Oct 24, 2025
refactor(#8): 🩹 Adjusted to PR comments
8b54df5
@coderabbit coderabbit bot mentioned this issue Oct 29, 2025
Merged
Sign in to join this conversation on GitHub.
Assignees

@95f5ca5b369f78f4fc2a22efa554cf068195794 95f5ca5b369f78f4fc2a22efa554cf068195794

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@gautegf @95f5ca5b369f78f4fc2a22efa554cf068195794