New try to minimaps improvements #340
Conversation
| {255,165,0},//29 orange | ||
|
|
||
| {139,69,19},//30 maroon medium | ||
| {210,105,30},//31 choccolate |
There was a problem hiding this comment.
This table is exactly the same as in MapView.cpp. I think this should be deduplicated.
There was a problem hiding this comment.
yes i know, but i dont know whether it is better position to put it in: MapView.cpp or ScoreboardView.cpp.
Second question. This requires something in particular about the place, or the syntax, where it needs to be declared the color palette?
i need help to do this, i'm not a real coder. i go to try!
There was a problem hiding this comment.
think it is appropriate to define the table within the namespace MapView.h openspades?
something like this:
http://pastebin.com/xhFB0H1C
*EDIT
or maybe is better use "extern"?
what is the correct thing?
There was a problem hiding this comment.
Ok now there is only one definition of the array palette.
other issues?
|
No one have other issues? |
Edit:
the DevPaks27.zip is now in my dropbox, all works fine, but, please feel free to bring back DevPaks.zip in the old places