[Freeciv-Dev] [patch #4866] Provide alternate semantics to the client to replace move_type

2014-06-28 Thread Emmet Hikory
URL:
  http://gna.org/patch/?4866

 Summary: Provide alternate semantics to the client to replace
move_type
 Project: Freeciv
Submitted by: persia
Submitted on: Sat 28 Jun 2014 07:18:56 PM JST
Category: client
Priority: 5 - Normal
  Status: None
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 

___

Details:

Attached are a pair of patches that demonstrate adding new features to the
ruleset to provide semantic values to be used by the client to replace
move_type checks.

The unit class group concept is demonstrated by use in editor tool grab
ordering, but may also be useful in other unit selection contexts (e.g. patch
#3721).  I'm not sure how to make the provided group names translatable, but
an adjustment to provide this would improve things in the case where the
strings were exposed to the UI.

While I've only demonstrated use of ruleset-specified unit background colors
in the Xaw client, it might also be useful in the Qt client (which currently
has it's own logic to calculate RGBA values depending on move_type).  I'm also
not entirely sure about the data storage model: casting color_std to int for
storage seems to work in my testing, but could potentially cause issues with
Qt (as enum is less likely to be int in C++).

While these semantic concepts seemed to match the client needs from my
viewpoint, I'll defer to folk actually working on clients to determine if
these are the right concepts, or if there are better ways to solve this class
of problems.



___

File Attachments:


---
Date: Sat 28 Jun 2014 07:18:56 PM JST  Name: introduce-unit-class-groups.patch
 Size: 34kB   By: persia

http://gna.org/patch/download.php?file_id=21181
---
Date: Sat 28 Jun 2014 07:18:56 PM JST  Name:
assign-background-color-in-the-ruleset.patch  Size: 24kB   By: persia

http://gna.org/patch/download.php?file_id=21182

___

Reply to this item at:

  http://gna.org/patch/?4866

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #4866] Provide alternate semantics to the client to replace move_type

2014-06-28 Thread Emmet Hikory
Update of patch #4866 (project freeciv):

  Depends on: = patch #4865


___

Reply to this item at:

  http://gna.org/patch/?4866

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev