#1461 closed enhancement (fixed)
[PATCH] Initial zoom to bounding box instead of downloaded data borders
| Reported by: | Owned by: | framm | |
|---|---|---|---|
| Priority: | minor | Milestone: | |
| Component: | Core | Version: | latest |
| Keywords: | Cc: |
Description
If I get the data for the area around here (51.76 1.0 -> 51.95 1.3) then there is a trace for the ferry route across to Holland. Ideally I'd like the initial view to default to the "yellow box" area, rather than whole data area/layer area, which represents the bounds I specified. At present I have to zoom into it. It's just a bit of an annoyance.
Attachments (3)
Change History (13)
comment:1 by , 17 years ago
| Summary: | Annoying tracks → Initial zoom to bounding box instead of downloaded data borders |
|---|
comment:2 by , 17 years ago
comment:3 by , 17 years ago
Personally I think we don't have to worry about multiple bounding boxes at the moment as the "initial zoom" is only applied on the first download (no OSM layer exists). Subsequent load operation do not affect map center or zoom.
I already tried to figure out where the logic of this initial zoom is located in the JOSM source but until now I wasn't successful.
comment:4 by , 17 years ago
Heureka, I finnally found the place where it zooms to the data - and wrote a workaround (see attached patch).
by , 17 years ago
| Attachment: | ZoomToBoundingBoxInsteadOfData_patch.txt added |
|---|
Changes: Zoom to bounding box, applied JOSM source code style and additional minor manual style changes.
comment:5 by , 17 years ago
| Summary: | Initial zoom to bounding box instead of downloaded data borders → [PATCH] Initial zoom to bounding box instead of downloaded data borders |
|---|
follow-up: 7 comment:6 by , 17 years ago
Please don't make patches which contain formating changes. At least not if you want them to be applied. I cannot check what this patch really does without a lot of work, so I myself will not apply it.
by , 17 years ago
| Attachment: | ZoomToBoundingBoxInsteadOfData_patch.2.txt added |
|---|
Rewritten patch without foratting changes
comment:7 by , 17 years ago
Please don't make patches which contain formating changes.
Ok, please see my rewritten patch - it is now clearly readable in a text editor.
by , 17 years ago
| Attachment: | ZoomToBoundingBoxInsteadOfData_patch.3.txt added |
|---|
This modified patch works with rev. 1013 and has correct indentation
follow-up: 9 comment:8 by , 17 years ago
I just tested the patch and as far as I can see, it works fine.
Also tried with files without bounding box. It zooms to data then, which is fine.
What I'd like to see now is a possibility to zoom to the bounding box via the menu. Question is: what to do, if there is more than one layer. Best would be to have two choices:
1.) Zoom to overall bounding box(es)
2.) Zoom to layer bounding box(es)
comment:9 by , 17 years ago
Replying to anonymous:
Question is: what to do, if there is more than one layer.
Or one layer with multiple bounding boxes...
I also would like to have this option.
Question is: What to do, if there is more than one bounding box?
Possible solution: Zoom to a "virtual" bounding box which enfolds all bounding boxes