Modify

Opened 7 weeks ago

Last modified 3 weeks ago

#15639 new defect

JOSM not respecting layers in display

Reported by: dieterdreist Owned by: team
Priority: normal Milestone:
Component: Internal mappaint style Version:
Keywords: template_report, layer, rendering order Cc: naoliv

Description (last modified by Klumbumbus)

What is the expected result?

layer order in screen rendering respected (draw first lower layers)

What happens instead?

underlying way rendered above bridge

Please provide any additional information below. Attach a screenshot if possible.


URL:http://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2017-12-05 00:58:56 +0100 (Tue, 05 Dec 2017)
Build-Date:2017-12-05 02:33:31
Revision:13194
Relative:URL: ^/trunk

Identification: JOSM/1.5 (13194 en) Mac OS X 10.13.1
OS Build number: Mac OS X 10.13.1 (17B1003)
Memory Usage: 1211 MB / 1820 MB (329 MB allocated, but free)
Java version: 1.8.0_144-b01, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Screen: Display 188875522 1920x1080, Display 458628992 1920x1080
Maximum Screen Size: 1920x1080
VM arguments: [-Dsun.java2d.opengl=true]
Dataset consistency test: No problems found

Plugins:
+ ColorPlugin (1511895663)
+ Create_grid_of_ways (33856)
+ ImportImagePlugin (33563)
+ PicLayer (33822)
+ ShapeTools (1230)
+ apache-commons (33668)
+ buildings_tools (33735)
+ editgpx (33782)
+ ejml (32680)
+ fieldpapers (v0.4.3)
+ geochat (33796)
+ geojson (73)
+ geotools (33930)
+ imagery_offset_db (33774)
+ jts (32699)
+ log4j (32699)
+ mbtiles (a3e491d)
+ measurement (33760)
+ o5m (33566)
+ osmarender (33916)
+ pbf (33568)
+ photo_geotagging (33878)
+ reltoolbox (33708)
+ reverter (33865)
+ undelete (33835)
+ utilsplugin2 (33742)

Last errors/warnings:
- W: java.net.UnknownHostException: ecn.t1.tiles.virtualearth.net
- W: Already here java.net.UnknownHostException: ecn.t1.tiles.virtualearth.net
- W: java.net.UnknownHostException: ecn.t3.tiles.virtualearth.net
- W: Already here java.net.UnknownHostException: ecn.t3.tiles.virtualearth.net
- W: java.net.UnknownHostException: ecn.t0.tiles.virtualearth.net
- W: Already here java.net.UnknownHostException: ecn.t0.tiles.virtualearth.net
- W: java.net.UnknownHostException: ecn.t1.tiles.virtualearth.net
- W: Already here java.net.UnknownHostException: ecn.t1.tiles.virtualearth.net
- W: java.net.UnknownHostException: ecn.t3.tiles.virtualearth.net
- W: Already here java.net.UnknownHostException: ecn.t3.tiles.virtualearth.net

Attachments (1)

Screen Shot 2017-12-06 at 12.11.06.png (62.3 KB) - added by dieterdreist 7 weeks ago.

Download all attachments as: .zip

Change History (9)

Changed 7 weeks ago by dieterdreist

comment:1 Changed 7 weeks ago by Klumbumbus

Description: modified (diff)

What's the object id?

comment:2 Changed 7 weeks ago by Klumbumbus

Component: CoreInternal mappaint style

comment:3 Changed 3 weeks ago by Don-vip

Cc: naoliv added
Milestone: 17.12

Isn't it a regression from r13125 as well?

comment:4 Changed 3 weeks ago by Klumbumbus

No, layer tags were never taken into account for the internal style until now. (I once (some years ago) tried to create a patch, but failed.)

comment:5 Changed 3 weeks ago by Don-vip

Ah ok, my bad. Well you have learned a lot since ;)

comment:6 Changed 3 weeks ago by Klumbumbus

Well, in mapcss it should be pretty simple like that: way[layer]::* {z-index: tag("layer");}
However the bigger problem is that we either need to free one of major-z-index, z-index or object-z-index to use for this or invent a fourth like layer-tag-z-index. Currently I think all the major-/z-/object-z-indexes are a bit confused in our style and it is hard to understand as a whole so it is hard to make any change without braking something.

comment:7 Changed 3 weeks ago by Don-vip

Milestone: 17.12

ok, not so easy!

comment:8 Changed 3 weeks ago by Klumbumbus

As a side effect r13125 "fixed" the same problem for waterway tunnels in most cases, see gif in ticket:15483#comment:5. Maybe this makes the not perfect bridge rendering more obvious.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set.
to The owner will be changed from team to the specified user.
The owner will change to dieterdreist
as duplicate The resolution will be set to duplicate.The specified ticket will be cross-referenced with this ticket
The owner will be changed from team to anonymous.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.