Skip to content

Comments

Allow setting the coordinates of survey vertices#2319

Merged
rafaellehmkuhl merged 3 commits intobluerobotics:masterfrom
rafaellehmkuhl:issue-2318-allow-setting-lat-long-survey-vertex
Jan 14, 2026
Merged

Allow setting the coordinates of survey vertices#2319
rafaellehmkuhl merged 3 commits intobluerobotics:masterfrom
rafaellehmkuhl:issue-2318-allow-setting-lat-long-survey-vertex

Conversation

@rafaellehmkuhl
Copy link
Member

@rafaellehmkuhl rafaellehmkuhl commented Jan 8, 2026

Needs a little more work. I'm trying to solve the ExpansiblePanel problem, where it starts expanded but hidden. It's almost there.

Cap.2026-01-08.at.10.29.23.mp4

Closes #2318

@rafaellehmkuhl rafaellehmkuhl force-pushed the issue-2318-allow-setting-lat-long-survey-vertex branch 4 times, most recently from d671b2c to 0bda46a Compare January 13, 2026 22:23
@rafaellehmkuhl rafaellehmkuhl marked this pull request as ready for review January 13, 2026 22:23
Copy link
Contributor

@ArturoManzoli ArturoManzoli left a comment

Choose a reason for hiding this comment

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

Awesome!

It's working well, but I would suggest some small adjustments:

  • (1) move up the divider to line 36, so it will create a header on the card. Also, to have a lighter look, you could add a font-[16px] to the trash can icon. Take a look on the last image 🔽
  • (2) The content of the panel isn't scrollable.
Image Image

@rafaellehmkuhl rafaellehmkuhl force-pushed the issue-2318-allow-setting-lat-long-survey-vertex branch from 0bda46a to 4d92453 Compare January 14, 2026 16:42
@rafaellehmkuhl
Copy link
Member Author

@ArturoManzoli nice catches! Both solved!

Copy link
Contributor

@ArturoManzoli ArturoManzoli left a comment

Choose a reason for hiding this comment

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

All good now!

@rafaellehmkuhl rafaellehmkuhl merged commit f5ef75c into bluerobotics:master Jan 14, 2026
11 checks passed
@rafaellehmkuhl rafaellehmkuhl deleted the issue-2318-allow-setting-lat-long-survey-vertex branch January 14, 2026 16:56
@ES-Alexander ES-Alexander added the docs-needed Change needs to be documented label Jan 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-needed Change needs to be documented

Projects

None yet

Development

Successfully merging this pull request may close these issues.

frontend: mission planning: allow lat/long coordinate editing of survey boundary vertexes

3 participants