Opened 15 years ago
Closed 14 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 , 15 years ago
| Reporter: | changed from to |
|---|
comment:2 by , 15 years ago
comment:3 by , 15 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 , 15 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 , 15 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 , 15 years ago
comment:9 by , 14 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |



It seems named styles are broken completely.