Follow-up Comment #2, patch #2062 (project freeciv):
> /playercolor takes a color argument like in the
> format "0xrrggbb". However, the featured text module already
> use a different format for colors ("#rrggbb"). They should be
> the same to do not confuse the users.
I did try this but a '#' is an invalid character wrt the command line. It has
to be added to fc_isalnum() (see stdinhand.c:3971)
> /playercolor help text should mention about the red/green/blue
> format a bit clearly.
OK
> /playercolor doesn't allow to remove a defined color by the
> user.
> I guess that /playercolor should take ALLOW_ADMIN access level
> and move to /list colors the non harmful related commands.
I can move it as follows:
/playercolor => the information is included in /list players; see patch
#2063
/playercolor <player-name> => remove this
/playercolor <player-name> <color> => define the color
/playercolor <player-name> reset => clear the user-defined color; only
available in pre-game
> I really don't like the usage of magic values for the
> setting "playercolor".
> the setting "playercolor" should at least be
> named "playercolormode" to don't duplicate this name (e.g.
> for /help).
OK
_______________________________________________________
Reply to this item at:
<http://gna.org/patch/?2062>
_______________________________________________
Nachricht geschickt von/durch Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
[email protected]
https://mail.gna.org/listinfo/freeciv-dev
