Modify

Opened 10 years ago

Last modified 9 years ago

#10106 new defect

mapcss: grouping doesn't work correctly

Reported by: Klumbumbus Owned by: team
Priority: normal Milestone:
Component: Core mappaint Version: latest
Keywords: mapcss Cc:

Description (last modified by Don-vip)

For example see way/4592789
I want to hide the new cycleway mappaint of elemstyle (see #10057). So I use an additional mapcss file with the following code. This work fine and hides all cycleway tracks.

way.tR::core_cycleway
{
        width: 0;
}
way[prop("tL","core_cycleway")]::core_cycleway2
{
        width: 0;
}

However, grouping both selectors as in the following example doesn't work correctly.

way.tR::core_cycleway,
way[prop("tL","core_cycleway")]::core_cycleway2
{
        width: 0;
}

With this code there is still a track at one side of the highway. It is even not always the same side. Manually reloading the style severaltimes successively in the mappaint dialog results in showing sometimes the left track and sometimes the right track.

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2014-06-06 01:35:12
Last Changed Author: akks
Revision: 7216
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2014-06-05 10:59:30 +0200 (Thu, 05 Jun 2014)
Last Changed Rev: 7216

Identification: JOSM/1.5 (7216 en) Windows 7 32-Bit
Memory Usage: 242 MB / 247 MB (77 MB allocated, but free)
Java version: 1.7.0_55, Oracle Corporation, Java HotSpot(TM) Client VM
VM arguments: [-Djava.security.policy=file:C:\Program Files\Java\jre7\lib\security\javaws.policy, -DtrustProxy=true, -Xverify:remote, -Djnlpx.home=<java.home>\bin, -Djnlpx.origFilenameArg=C:\Program Files\josm-latest.jnlp, -Djnlpx.remove=true, -Dsun.awt.warmup=true, -Xbootclasspath/a:C:\Program Files\Java\jre7\lib\javaws.jar;C:\Program Files\Java\jre7\lib\deploy.jar;C:\Program Files\Java\jre7\lib\plugin.jar, -Djava.util.Arrays.useLegacyMergeSort=true, -Djnlpx.splashport=54625, -Djnlpx.jvm=<java.home>\bin\javaw.exe, -Djnlpx.vmargs=-Djava.util.Arrays.useLegacyMergeSort=true]
Dataset consistency test: No problems found

Plugin: DirectUpload (30416)
Plugin: FastDraw (30416)
Plugin: FixAddresses (30416)
Plugin: HouseNumberTaggingTool (30416)
Plugin: OpeningHoursEditor (30416)
Plugin: PicLayer (30436)
Plugin: RoadSigns (30438)
Plugin: Tracer2 (30416)
Plugin: buildings_tools (30485)
Plugin: download_along (30416)
Plugin: editgpx (30416)
Plugin: imagery_offset_db (30416)
Plugin: measurement (30416)
Plugin: notes (v0.9.2)
Plugin: openstreetbugs (30436)
Plugin: osmarender (30416)
Plugin: public_transport (30416)
Plugin: reltoolbox (30416)
Plugin: reverter (30436)
Plugin: terracer (30416)
Plugin: turnlanes (30416)
Plugin: turnrestrictions (30454)
Plugin: undelete (30416)
Plugin: utilsplugin2 (30460)

Attachments (0)

Change History (2)

comment:1 by bastiK, 10 years ago

See #8568.

Maybe we should just revert [6736]. Comma is not a logical or, but a shortcut for multiple identical declaration blocks (http://www.w3.org/TR/selectors/#grouping).

comment:2 by Don-vip, 9 years ago

Description: modified (diff)
Keywords: mapcss added

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from team to the specified user.
Next status will be 'needinfo'. The owner will be changed from team to Klumbumbus.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from team to anonymous. Next status will be 'assigned'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.