Modify

Opened 10 years ago

Last modified 3 years ago

#10435 new enhancement

Improvements to user-settings for mapcss mappaint styles

Reported by: Klumbumbus Owned by: team
Priority: normal Milestone:
Component: Core mappaint Version:
Keywords: style settings mapcss Cc: skyper, imagic, richlv

Description (last modified by skyper)

User-settings for mapcss mappaint styles were included in JOSM with very basic functionality (see #10421). While this was ok for its first use, it could need some improvements which I want to sum up here:

  • Currently it is only accessible in mappaint dialog. Many users do not use the dialog or even do not know how to get it. (Although I described where to change the user setting and attached a screenshot in the forum, users searched in the preferences window to change the user setting and gave up searching). It would be useful if the user-settings are also accessible in the preferences window and in the view menu (maybe also for buttons placed in the toolbar). To make it accessible in the preferences window (which is maybe indeed the first place to search for this setting) is maybe relevant for milestone:14.08 since it includes the big changes in default mappaint style.
  • Enable to set a keyboard shortcut to change a specific (boolean) user setting.
  • Enable grouping of similar settings for enabling/disabling several settings at once. Done in r15289

Attachments (0)

Change History (30)

comment:1 by Don-vip, 10 years ago

I agree on point 1 we should make it more accessible. I don't understand the difference between point 4 and #10426 you created earlier?

in reply to:  1 comment:2 by Klumbumbus, 10 years ago

Replying to Don-vip:

I don't understand the difference between point 4 and #10426 you created earlier?

#10426 was not about user settings but about strings to display on nodes, ways and areas. I would like to use it in Styles/Coloured_Streets. You can see there in the screenshoots that it displays strings on nodes, ways and areas like "street?" or "number?" or "addr:interpolation shouldn't have other addr:*=* tags!"

I don't know if there is a difference in implementation if tr() is used in a mappaint user setting or in a normal block like way[highway=residential][!name]{ text: tr("name?"); } (Help/Styles/MapCSSImplementation says tr()is only for strings in the JOSM user interface).
If there is no difference, then #10426 can be closed as duplicate of this ticket.

comment:3 by skyper, 10 years ago

Cc: skyper added

in reply to:  description comment:4 by bastiK, 10 years ago

Replying to Klumbumbus:

  • Currently it is only accessible in mappaint dialog. Many users do not use the dialog or even do not know how to get it. (Although I described where to change the user setting and attached a screenshot in the forum, users searched in the preferences window to change the user setting and gave up searching). It would be useful if the user-settings are also accessible in the preferences window and in the view menu (maybe also for buttons placed in the toolbar). To make it accessible in the preferences window (which is maybe indeed the first place to search for this setting) is maybe relevant for milestone:14.08 since it includes the big changes in default mappaint style.

Technically, it is not so easy to have this setting available in the preference window. (At the moment, it is only aware of the metadata, either downloaded from the server or saved to the preferences.xml. The mapcss file is not parsed before the user hits Okay.)
I won't have time for this before the 14.08 release.

comment:5 by Don-vip, 10 years ago

No problem, the version is stable enough, release in progress :)

comment:6 by Klumbumbus, 10 years ago

Description: modified (diff)

added to ticket text: Add documentation at Help/Styles/MapCSSImplementation

comment:7 by skyper, 10 years ago

Cc: imagic added

comment:8 by Klumbumbus, 10 years ago

Description: modified (diff)

comment:9 by G_Fyyar, 5 years ago

Is anyone still working on this? Or is there a newer ticket pertaining to enhancing mapcss user-settings?

comment:10 by Don-vip, 5 years ago

I plan to work on making style settings more accessible (first point of this ticket).

comment:11 by Don-vip, 5 years ago

Description: modified (diff)
Keywords: style settings mapcss added

comment:12 by G_Fyyar, 5 years ago

Do you know if anyone is/was planning on expanding the user settings (second point of this ticket)?

comment:13 by Don-vip, 5 years ago

In 15288/josm:

see #10435 - keep the style settings menu open after one has been selected, like for mappaint menu

in reply to:  13 comment:14 by Klumbumbus, 5 years ago

Replying to Don-vip:

keep the style settings menu open

Thats a nice feature.

comment:15 by Don-vip, 5 years ago

In 15289/josm:

see #10435 - Enable grouping of similar settings for enabling/disabling several settings at once

comment:16 by Don-vip, 5 years ago

Description: modified (diff)

comment:17 by Klumbumbus, 5 years ago

Description: modified (diff)

comment:18 by G_Fyyar, 5 years ago

I'm sorry--there was some miscommunication on my end. I meant to say, do you know if anyone is/was planning on expanding the user settings (second point of this ticket)? If not, I'll be working on it.

comment:19 by Don-vip, 5 years ago

I don't plan to work on it right now, a patch would be welcome.

in reply to:  18 comment:20 by Klumbumbus, 5 years ago

Replying to G_Fyyar:

expanding the user settings (second point of this ticket)? If not, I'll be working on it.

Possible use case see ticket:18095#comment:4

comment:21 by richlv, 5 years ago

Cc: richlv added

comment:22 by Don-vip, 5 years ago

In 15330/josm:

see #10435 - add more "stay open" menu item classes

comment:23 by Don-vip, 5 years ago

In 15331/josm:

fix #18032, see #10435 - proper update of StayOpenCheckBoxMenuItemUI - fix rendering issue on macOS

comment:24 by Don-vip, 5 years ago

In 15339/josm:

see #10435 - fix Findbugs violation VO_VOLATILE_REFERENCE_TO_ARRAY

comment:25 by Don-vip, 5 years ago

In 15492/josm:

see #18032, see #10435 - fix mappaint popup menu disappearance on macOS

macOS triggers a spurious sun.awt.UngrabEvent that is catched by BasicPopupMenuUI.MouseGrabber
and makes the popup menu disappear. Probably related to https://bugs.openjdk.java.net/browse/JDK-8225698

comment:26 by simon04, 5 years ago

In 15731/josm:

see #10435, fix #18095 - MapCSS: add settings of type string/double

in reply to:  26 comment:27 by Klumbumbus, 5 years ago

Replying to simon04:

In 15731/josm:

see #10435, fix #18095 - MapCSS: add settings of type string/double

Could you please add documentation on wiki:/Help/Styles/MapCSSImplementation#Stylesettings?

comment:29 by Klumbumbus, 5 years ago

Thx.

comment:30 by skyper, 3 years ago

Description: modified (diff)

mark implemented parts

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.