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

Opening Rapid with id=... query parameter does not zoom to object #1108

Closed
elevenchars opened this issue Sep 10, 2023 · 7 comments
Closed

Opening Rapid with id=... query parameter does not zoom to object #1108

elevenchars opened this issue Sep 10, 2023 · 7 comments
Assignees
Labels
bug-regression It worked before but now it doesn't good first issue Good for newcomers
Milestone

Comments

@elevenchars
Copy link

Description

When I open Rapid with an id query parameter, it shows up in the left "Edit feature" pane but it does not zoom the map to the location of the object. However, pressing "zoom to selection" zooms to the object correctly.

Version

2.1.1

What browser are you seeing the problem on? What version are you running?

Firefox 117

The OS you're using

MacOS 13.5.2

Steps to reproduce

  1. Open a Rapid url which contains an valid id query parameter. (Here is the one that I originally noticed this with, generated by OSMCha: https://mapwith.ai/rapid#id=w1204555438)
  2. Observe that the editor does not zoom to the object as expected.

The browser URL at the time you encountered the bug

https://rapideditor.org/rapid#background=Bing&datasets=fbRoads,msBuildings&disable_features=boundaries&id=w1204555438&map=2.00/0.0/0.0

The auto-detected useragent string for your browser (leave blank if you're manually filling this form out)

No response

@bhousel bhousel self-assigned this Sep 11, 2023
@bhousel bhousel added the bug-regression It worked before but now it doesn't label Sep 11, 2023
@Bonkles Bonkles added the good first issue Good for newcomers label Oct 11, 2023
@Bonkles
Copy link
Contributor

Bonkles commented Oct 20, 2023

Reassigning to @voscarmv, our third cohort member for the MLH pod joining us this fall

@voscarmv
Copy link
Contributor

voscarmv commented Oct 21, 2023

Reassigning to @voscarmv, our third cohort member for the MLH pod joining us this fall

Glad to be here!

@bhousel
Copy link
Contributor

bhousel commented Oct 24, 2023

I actually think this issue might have been fixed already.

@elevenchars
Copy link
Author

I just tested it on latest commit of main and the issue is still present. Is the fix on another branch ?

@bhousel
Copy link
Contributor

bhousel commented Oct 24, 2023

Thanks for confirming - I just tested and I agree, it's wrong! 😄
http://127.0.0.1:8080/#background=Bing&id=w1204555438&map=2.00/0.0/0.0

I'll take care of this myself - it's related to code that I recently moved from Context into MapSystem, but UrlHashSystem is involved too. This relates to the order that things start up and events fire.

@bhousel bhousel assigned bhousel and unassigned voscarmv Oct 24, 2023
@voscarmv
Copy link
Contributor

voscarmv commented Oct 24, 2023

Hi @bhousel I just subitted a PR for this!

Here it is: #1170

@voscarmv
Copy link
Contributor

voscarmv commented Oct 25, 2023

Hi @bhousel I just subitted a PR for this!

Here it is: #1170

PR #1170 is 14 commits behind the main branch.

I made another PR but this time it's rebased on the latest Rapid commit, here: #1173

I'd appreciate your code review please. Thank you!

@Bonkles Bonkles added this to the v2.2 milestone Nov 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-regression It worked before but now it doesn't good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

4 participants