Modify

Opened 14 years ago

Closed 13 years ago

#5372 closed enhancement (fixed)

Improve mappaint style switch

Reported by: stoecker Owned by: team
Priority: major Milestone:
Component: Core Version:
Keywords: Cc:

Description

The possibility to switch a mappaint style during runtime between the loaded styles is broken. Now a restart is required.

Attachments (0)

Change History (9)

comment:1 by stoecker, 14 years ago

Reporter: changed from anonymous to stoecker

comment:2 by stoecker, 14 years ago

It seems named styles are broken completely.

comment:3 by stoecker, 14 years ago

Named styles works, was my own error. Reason for broken style switch may be this: All the style caching I did has been moved into the primitive, but there is no check anymore to ensure we still have the right style. Either we need a caching or the cached styles must be cleared when the style is changed.

Additionally we could add a menu-point to choose styles, so it is no longer necessary to go to prefs.

comment:4 by stoecker, 14 years ago

(In [3466]) see #5372 - fix missing style switch

comment:5 by stoecker, 14 years ago

Owner: changed from jttt to team
Summary: Switching mappaint style during runtime brokenImprove mappaint style switch
Type: defectenhancement

Two points as improvement missing (Jiri or Sebastian, can you do these, I did not get these done for long time):

  • add JosmAction, which allows to switch style in toolbar (like search with parameter)
  • add above action as menu entry (when more than one style is named?)
  • add "resource://data/elemstyles.xml" in the styles-select-list as internal style
  • better separate the style naming by an own table column, so people recognice we have that feature.

comment:6 by bastiK, 14 years ago

I've done something like that already:

http://github.com/bastik/josm/commit/ea2395df256f348ee2208bcdda864fd15f58f3de

But I'm not too happy with it, yet. It introduces a distinction between Base Styles and Extension Styles, but I'll probably revert that part.

in reply to:  4 comment:7 by bastiK, 14 years ago

Replying to stoecker:

(In [3466]) see #5372 - fix missing style switch

I think this only does the current data layer

comment:8 by stoecker, 14 years ago

(In [3467]) see #5372 - extend clean for all layers

comment:9 by stoecker, 13 years ago

Resolution: fixed
Status: newclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


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