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.
> /playercolor doesn't allow to remove a defined color by the
> 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
/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).
Reply to this item at:
Nachricht geschickt von/durch Gna!
Freeciv-dev mailing list