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 , 14 years ago
Reporter: | changed from | to
---|
comment:2 by , 14 years ago
comment:3 by , 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:5 by , 14 years ago
Owner: | changed from | to
---|---|
Summary: | Switching mappaint style during runtime broken → Improve mappaint style switch |
Type: | defect → enhancement |
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 , 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.
comment:7 by , 14 years ago
comment:9 by , 13 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
It seems named styles are broken completely.