Modify

Opened 3 years ago

Last modified 3 years ago

#21994 needinfo defect

barrier=chain icon is absolutely massive

Reported by: anonymous Owned by: anonymous
Priority: normal Milestone:
Component: Internal mappaint style Version:
Keywords: template_report barrier chain icon size Cc:

Description (last modified by skyper)

What steps will reproduce the problem?

  1. Tag a node with "barrier=chain" while the default map paint style is active ("JOSM default (MapCSS)")

What is the expected result?

The node will be drawn with a similar style/size to every other node tagged as a barrier.

What happens instead?

The image of a chain is several times larger than every other node.

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

source:trunk/resources/images/presets/barrier/chain.svg

Enabling/disabling different paint styles has no effect; only disabling the JOSM default style makes the giant chain icon go away.

Revision:18360
Build-Date:2022-01-02 23:53:33

Identification: JOSM/1.5 (18360 en) Windows 8.1 64-Bit
OS Build number: Windows 8.1 (9600)
Memory Usage: 1588 MB / 3044 MB (339 MB allocated, but free)
Java version: 17.0.1+12-LTS, Azul Systems, Inc., OpenJDK 64-Bit Server VM
Look and Feel: com.sun.java.swing.plaf.windows.WindowsLookAndFeel
Screen: \Display0 1920×1080 (scaling 1.25×1.25) \Display1 1600×900 (scaling 1.25×1.25)
Maximum Screen Size: 1920×1080
Best cursor sizes: 16×16→32×32, 32×32→32×32
System property file.encoding: Cp1252
System property sun.jnu.encoding: Cp1252
Locale info: en_US
Numbers with default locale: 1234567890 -> 1234567890
VM arguments: [-Djpackage.app-version=1.5.18360, --add-modules=java.scripting,java.sql,javafx.controls,javafx.media,javafx.swing,javafx.web, --add-exports=java.base/sun.security.action=ALL-UNNAMED, --add-exports=java.desktop/com.sun.imageio.plugins.jpeg=ALL-UNNAMED, --add-exports=java.desktop/com.sun.imageio.spi=ALL-UNNAMED, --add-opens=java.base/java.lang=ALL-UNNAMED, --add-opens=java.base/java.nio=ALL-UNNAMED, --add-opens=java.base/jdk.internal.loader=ALL-UNNAMED, --add-opens=java.base/jdk.internal.ref=ALL-UNNAMED, --add-opens=java.desktop/javax.imageio.spi=ALL-UNNAMED, --add-opens=java.desktop/javax.swing.text.html=ALL-UNNAMED, --add-opens=java.prefs/java.util.prefs=ALL-UNNAMED, -Djpackage.app-path=%UserProfile%\AppData\Local\JOSM\JOSM.exe]
Dataset consistency test: No problems found

Plugins:
+ apache-commons (35893)
+ apache-http (35893)
+ buildings_tools (35916)
+ ejml (35893)
+ geotools (35906)
+ jaxb (35893)
+ jna (35893)
+ jts (35893)
+ opendata (35910)
+ print (35893)
+ pt_assistant (1ff2e15)
+ public_transport (35893)
+ reverter (35893)
+ turnrestrictions (35893)
+ undelete (35893)
+ utilsplugin2 (35893)

Tagging presets:
+ https://josm.openstreetmap.de/josmfile?page=Presets/Simple_Indoor_Tagging&zip=1

Map paint styles:
+ https://www.openrailwaymap.org/styles/maxspeed.zip
- https://josm.openstreetmap.de/josmfile?page=Styles/Lane_and_Road_Attributes&zip=1
+ %UserProfile%\Documents\Styles_Sidewalks2.zip
- https://josm.openstreetmap.de/josmfile?page=Styles/Sidewalks&zip=1
+ https://josm.openstreetmap.de/josmfile?page=Styles/PublicTransport&zip=1
- https://josm.openstreetmap.de/josmfile?page=Styles/Cycleways&zip=1
+ https://josm.openstreetmap.de/josmfile?page=Styles/HiDPISupport&zip=1
+ https://josm.openstreetmap.de/josmfile?page=Styles/Potlach2_access&zip=1
- https://github.com/GlassOceanos/indoor-JOSM-style/archive/master.zip
- https://josm.openstreetmap.de/josmfile?page=Styles/PublicTransportV2&zip=1
- https://josm.openstreetmap.de/josmfile?page=Styles/Coloured_buildings&zip=1
- https://josm.openstreetmap.de/josmfile?page=Styles/Surface-DataEntry&zip=1
+ %UserProfile%\Documents\osm\styles\Styles_Coloured_buildings_en-style.mapcss
+ https://raw.githubusercontent.com/yopaseopor/traffic_signs_style_JOSM/master/Styles_Traffic_signs_AME.zip
+ https://josm.openstreetmap.de/josmfile?page=Styles/ParkingLanes&zip=1
+ https://raw.githubusercontent.com/species/josm-preset-traffic_sign_direction/master/direction.mapcss

Last errors/warnings:
- 00000.600 W: extended font config - overriding 'filename.Raavi=raavi.ttf' with 'RAAVI.TTF'
- 00000.601 W: extended font config - overriding 'filename.Gautami=gautami.ttf' with 'GAUTAMI.TTF'
- 00000.601 W: extended font config - overriding 'filename.Vrinda=vrinda.ttf' with 'VRINDA.TTF'
- 00000.602 W: extended font config - overriding 'filename.Kartika=kartika.ttf' with 'KARTIKA.TTF'
- 00000.602 W: extended font config - overriding 'filename.Latha=latha.ttf' with 'LATHA.TTF'
- 00001.077 E: java.security.KeyStoreException: Windows-ROOT not found. Cause: java.security.NoSuchAlgorithmException: Windows-ROOT KeyStore not available
- 00009.944 E: Id 'standard' is not unique - used by 'OpenStreetMap Carto (Standard)' and 'OpenStreetMap Carto (Standard)'!
- 00009.945 E: Id 'USGS-Imagery' is not unique - used by 'USGS Imagery' and 'USGS Imagery'!
- 00019.352 W: Warning: Failed to scan file 'site-svn.openstreetmap.org-_applications_editors_josm_plugins_opendata_modules.txt' for module information. Skipping.
- 00019.547 E: Failed to locate image 'bus.png'

Attachments (2)

chain.png (658.9 KB ) - added by anonymous 3 years ago.
Screenshot showing chain and bollard
chain_and_bollard.jpg (4.0 KB ) - added by skyper 3 years ago.
screenshot with new (empty) preferences

Download all attachments as: .zip

Change History (7)

comment:1 by skyper, 3 years ago

Description: modified (diff)
Keywords: barrier chain icon size added

It might be better to scale down the original icon to a width of 16px than using icon-width in the style.

Last edited 3 years ago by skyper (previous) (diff)

comment:2 by stoecker, 3 years ago

Owner: changed from team to anonymous
Status: newneedinfo

Looks ok here. Please provide a screenshot.

by anonymous, 3 years ago

Attachment: chain.png added

Screenshot showing chain and bollard

by skyper, 3 years ago

Attachment: chain_and_bollard.jpg added

screenshot with new (empty) preferences

comment:3 by skyper, 3 years ago

Cannot reproduce, either.

screenshot with new (empty) preferences

First of all, please, disable all other mappaint styles.
It might be a problem with general display scaling as I find Screen: \Display0 1920×1080 (scaling 1.25×1.25) \Display1 1600×900 (scaling 1.25×1.25) in your status report.

comment:4 by stoecker, 3 years ago

It's a bug anyway, question is why and where?

comment:5 by Klumbumbus, 3 years ago

The special of the chain icon is that one dimension is below and one above 64 px, which is our threshold to scale icons down to 16px.
(The chain icon is 117,87615 x 58,07882 px)
Maybe this is handled differently in Java 17 than in older versions (or a combination of Java 17 and scaling)?
Then resizing the icon would fix it but might not be the best fix.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as needinfo The owner will remain anonymous.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from anonymous to the specified user. Next status will be 'new'.
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 remain 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.