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

New Search Bar + Fly To works #226

Merged
merged 8 commits into from
Jun 3, 2024
Merged

New Search Bar + Fly To works #226

merged 8 commits into from
Jun 3, 2024

Conversation

CelineMP
Copy link
Contributor

@CelineMP CelineMP commented Jun 3, 2024

New expandable Search Bar on top left on map.
Favorites always fixed on top.
All results can be added to favorites now.

In MapOptions I kept the old button that triggers the SearchPopUp in comments so we can quickly change if we prefer the old design.

I didn't integrate the latitude-longitude search as I want to discuss it first. My opinion is to leave it like that because users copy paste coordinates anyway and don't need a seperate input mode.

I was not sure about the folder structure. SearchBar.tsx is now in the PopUp folder, has to be changed eventually.

Signed-off-by: Celine Pöhl <celine.poehl@fau.de>
Signed-off-by: Celine Pöhl <celine.poehl@fau.de>
@CelineMP CelineMP added the frontend Issues related to the user interface of a web application label Jun 3, 2024
@CelineMP CelineMP requested review from Corgam and Lucas-Nan June 3, 2024 13:49
Signed-off-by: Celine Pöhl <celine.poehl@fau.de>
Signed-off-by: Celine Pöhl <celine.poehl@fau.de>
Signed-off-by: Celine Pöhl <celine.poehl@fau.de>
Lucas-Nan
Lucas-Nan previously approved these changes Jun 3, 2024
Copy link
Contributor

@Lucas-Nan Lucas-Nan left a comment

Choose a reason for hiding this comment

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

looks good enough for release. small things like corners can be changed later. FlyTo is doing (random) stuff. still todo for later

Signed-off-by: Celine Pöhl <celine.poehl@fau.de>
@Lucas-Nan Lucas-Nan self-requested a review June 3, 2024 15:19
Lucas-Nan
Lucas-Nan previously approved these changes Jun 3, 2024
Copy link
Contributor

@Lucas-Nan Lucas-Nan left a comment

Choose a reason for hiding this comment

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

looks good enough for release. small things like corners can be changed later. FlyTo is doing (random) stuff - still todo for later

Signed-off-by: Celine Pöhl <celine.poehl@fau.de>
@CelineMP
Copy link
Contributor Author

CelineMP commented Jun 3, 2024

Added some commits that make flying to a location from the search possible now

@CelineMP CelineMP changed the title New Search Bar New Search Bar + Fly To works Jun 3, 2024
@Lucas-Nan Lucas-Nan self-requested a review June 3, 2024 15:39
Copy link
Contributor

@Lucas-Nan Lucas-Nan left a comment

Choose a reason for hiding this comment

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

looks good enough for release. small things like corners can be changed later. I really like the flyTo

@Lucas-Nan Lucas-Nan merged commit 6f41fb4 into sprint-release Jun 3, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend Issues related to the user interface of a web application
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants