Opened 15 years ago
Closed 10 years ago
#3016 closed defect (fixed)
Zoom limitations
Reported by: | Owned by: | team | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | Core | Version: | tested |
Keywords: | zoom | Cc: | Gubaer |
Description (last modified by )
Under r1816 I can:
- Open JoSM
- Create a new document with C-n
- Click around with the drawing tool to create new nodes
All of the nodes I'm creating will be added at lat/lon 0.0, furthermore I can't see them as I'm drawing them nor the lines between them (the only hint is the updated selection dialog).
I also can't scroll the mouse wheel to zoom in/out. Nor does the lat/lon display at the bottom of the screen update from 0.0/0.0. But I can grab the zoom adjuster and zoom out that way.
This also has nothing to do with my preferences since JOSM does the same thing once I move my preferences file elsewhere.
<?xml version='1.0' encoding='UTF-8'?> <osm version='0.6' generator='JOSM'> <node id='-1' visible='true' lat='0.0' lon='0.0' /> <node id='-2' visible='true' lat='0.0' lon='0.0' /> <node id='-3' visible='true' lat='0.0' lon='0.0' /> <node id='-4' visible='true' lat='0.0' lon='0.0' /> <node id='-5' visible='true' lat='0.0' lon='0.0' /> <way id='-6' action='modify' visible='true'> <nd ref='-1' /> <nd ref='-2' /> <nd ref='-3' /> <nd ref='-4' /> <nd ref='-5' /> </way> </osm>
Attachments (2)
Change History (20)
comment:1 by , 15 years ago
Priority: | critical → blocker |
---|
follow-up: 4 comment:3 by , 15 years ago
Priority: | blocker → major |
---|---|
Summary: | JOSM produces nodes with lat/lon 0.0 when I create a new OSM document → Zoom limitations |
Press <STRG>-<N> after josm Start.
The reason is that such a new datalayer is deep-zoomed and scroll-wheel zooming does help to get out. The accuracy of variables is not high enought to have any effect.
Fix: Prevent to deep zoom as well as zooming out of the world in NavigatableComponent.
- Deepzoom: Maxzoomdepth = 0.1m
- Maxzoom: Whole world could be visible
- Zooming should not allow to zoom to positions outside of world
Remove the blocker as this affects only new layers which anyway should not be used for uploading anything.
comment:4 by , 15 years ago
Replying to stoecker:
Press <STRG>-<N> after josm Start.
The reason is that such a new datalayer is deep-zoomed and scroll-wheel zooming does help to get out. The accuracy of variables is not high enought to have any effect.
Fix: Prevent to deep zoom as well as zooming out of the world in NavigatableComponent.
- Deepzoom: Maxzoomdepth = 0.1m
- Maxzoom: Whole world could be visible
- Zooming should not allow to zoom to positions outside of world
Remove the blocker as this affects only new layers which anyway should not be used for uploading anything.
They should not, but I had in mind the inexperienced user who might create a new layer, try to edit something, not notice any changes and then discovering the download button and not choosing "Download in a new layer". That would result 0.0/0.0 nodes uploaded to the server.
follow-ups: 6 8 comment:5 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
Fixed in r1815 (I believe). Anyway is fixed now. No more deep zoom possible.
comment:6 by , 15 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
comment:8 by , 15 years ago
Version: | latest → tested |
---|
Replying to stoecker:
Fixed in r1815 (I believe). Anyway is fixed now. No more deep zoom possible.
You are right, you could not do a deep zoom yourself. But when starting josm it seems that the bbox is not set and so you starts in the deep zoom. The next Problem is that the zoom slider is in the middle position instead of being in the right. Zooming out once and you never get in the deep zoom back.
comment:9 by , 15 years ago
Summary: | Zoom limitations → [patch] Zoom limitations |
---|
Till a proper fix arrives, just set an initial value for new layers, as mentioned in #3204.
comment:10 by , 15 years ago
Owner: | changed from | to
---|---|
Status: | reopened → new |
Setting an initial scale only works for some projections. I'm still investigating a proper way to let each projection specify their own default center and zoom level, but I ran into NPEs that I suspect arise from some code parts being static (haven't found exact issue yet).
Also, fixing this is merely a minor issue, I only ever used new layers for testing. It's certainly not something the non-developer mapper will run into.
comment:11 by , 15 years ago
Priority: | major → minor |
---|---|
Summary: | [patch] Zoom limitations → Zoom limitations |
by , 15 years ago
Attachment: | wrong_deefault_zoom.patch added |
---|
Adds a default zoom level method for each projection. The center is derived from the world bounds, this is why it doesn't (yet) work with Lambert projections. The comment says we do not handle handle coordinates outside the boundaries correctly, but doesn't go into further detail.
comment:13 by , 15 years ago
Summary: | Zoom limitations → [patch?] Zoom limitations |
---|
comment:15 by , 15 years ago
Summary: | [patch?] Zoom limitations → Zoom limitations |
---|
Well, probably not the best solution, but applied anyway. But I don't think the bug is closed yet. There are still other issues regarding e.g. switching projections when the switching moves center out of world.
Regarding coordinates outside the world: Nobody knows what these will be handled like. The projection code may work or it may produce invalid results. Currently JOSM ignores these problems totally. I have a method in mind to fix this - Move the outlying nodes near the next border and draw them there. Each projection should have a method to "FIX" outlying coordinates in some way for proper display.
comment:16 by , 12 years ago
Description: | modified (diff) |
---|---|
Owner: | changed from | to
comment:18 by , 10 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
I tested this with josm-tested.jar (r1788) where the bug didn't occur but it does happen in HEAD.
Marking this as a release blocker. JOSM versions with this bug are likely to upload corrupt data do the server.