#10468 closed defect (fixed)
Highway label issues in new map style [patch]
Reported by: | AMDmi3 | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | 14.09 |
Component: | Internal mappaint style | Version: | tested |
Keywords: | Cc: | Klumbumbus |
Description (last modified by )
New map style brings several problems:
1) It's dependent on zoom. This is not acceptable for advanced editor, as a mapper should always see all objects in area on any zoom, including their types (e.g. icons) and complexity (e.g. nodes). At least there should be an option to do so.
2) Road labels are unreadable when antialiasing is disabled (and here it is disabled, as JOSM is too slow to render dense map data even without it). There's no way to disable labels either.
I haven't found a way to fix these problems in the style, it seems that zooms and labels are handled inside the renderer. I haven't found any relevant knobs in `Advanced Preferences' either. So for now there's no other way than to stick with old XML style.
So, minimal customization abilities should be added, which don't look too complex:
- an option to fix zoom level for renderer
- an option to disable labels for ways
Attachments (2)
Change History (14)
comment:1 by , 10 years ago
comment:2 by , 10 years ago
Description: | modified (diff) |
---|---|
Summary: | New map style dependent on zoom, not customizable → Highway label issues in new map style |
follow-up: 4 comment:3 by , 10 years ago
This can be configured, see style settings.
This works, thanks. However, the location of that settings is really counter-intuitive - I would never open Map Paint Styles window. It probably should be either in Preferences or View->Map paint styles menu.
Could you describe the machine where you have performance problems (CPU, memory)? Please add a status report.
I don't think that has anything to do with my environment. I just work with large data sets.
comment:4 by , 10 years ago
comment:5 by , 10 years ago
Cc: | added |
---|
comment:6 by , 10 years ago
Component: | Core mappaint → Internal mappaint style |
---|
by , 10 years ago
Attachment: | neefestraße.png added |
---|
comment:7 by , 10 years ago
by , 10 years ago
Attachment: | 7470 style setting display_street_labels.diff added |
---|
follow-up: 10 comment:8 by , 10 years ago
Summary: | Highway label issues in new map style → Highway label issues in new map style [patch] |
---|
I made a patch for this.
comment:10 by , 10 years ago
Replying to Klumbumbus:
I made a patch for this.
Thanks! The changes I did are with efficiency in mind. The idea is that simple style conditions need not be evaluated at runtime, but there could be a pre-processing step. Any rules where the setting condition does not match can be removed from the style sheet in advance. (Not implemented yet.)
comment:11 by , 10 years ago
Milestone: | → 14.09 |
---|
comment:12 by , 10 years ago
Thank you! The style is now usable. Now the only regression to the old style I can notice is that selected nodes are not filled (and mappaint.node.fill-selected doesn't affect it), but this is very minor.
Replying to AMDmi3:
This can be configured, see style settings.
Could you describe the machine where you have performance problems (CPU, memory)? Please add a status report.