Discussion:
[Freeciv-Dev] [patch #3756] RFC: UI for building generic roads, bases, etc
Jacob Nevins
2014-08-31 13:12:46 UTC
Permalink
Update of patch #3756 (project freeciv):

Status: None => In Progress
Assigned to: None => jtn
Planned Release: => 2.6.0

_______________________________________________________

Follow-up Comment #5:

Here is a very rough prototype of this feature for trunk. I'd be very
interested what people think.

Major restrictions of this prototype:
* Gtk2 only (and other clients are probably broken).
* Distinction between fortress/airbase UI is lost (both Shift+F/E and menu
items do the same thing).
** If we want to keep this then the distinction has to become an extra_cause,
I think.
* Only covers extra creation, not removal (pollution/pillage etc).
* Not very discoverable (menu items don't list "R, 3" style shortcuts -- and
it doesn't like like GtkAccelLabel can be abused to do so)
* Pop-up obscures view (just like caravan/diplomat dialog).
** I think correct fix is to find a solution for all these "choice_dialogs"
(maybe an option to put it in the unit display area of the main window? Would
only work for large displays)

Discovered consequences of this design:
* Currently you can use non-keypad number keys to move units (added in r1351
<http://svn.gna.org/viewcvs/freeciv?revision=1351&view=revision>); with this
design, you won't be able to do that any more (nor can you use keypad to
choose extras).
* With multiple units selected, and "Mine" activity, where some units can
build extras and others convert terrain, need to decide how to present this.
I've brought conversion into the pop-up menu (special key "0" =>
mine/irrigation can only cause 9 extras each)
* Current version complains if you define more extras in ruleset than there
are keys available. Alternative is that these are only reachable from menus.
(The latter will probably be necessary to enable pillage selection to move to
this system, as otherwise it'll likely have >10 options.)

Ways to test without changing rulesets:
* classic ruleset: Since fortress/airbase are merged, arrange to have
Construction+Radio and press Shift+F or Shift+E on suitable unit/terrain.
* classic ruleset: Select multiple units on different terrains where one has
mine extra and other has conversion (e.g. Forest, Hills), press M.
* alien ruleset: Native Engineer on radiating terrain with Strong Resistance
known, press R (choice of Road and Tunnel).

I would have liked to get this in 2.5, but the extra rework on trunk makes it
sufficiently much easier that I think it's only going to appear in 2.6 at the
earliest. So rulesets with lots of choices of roads won't really be practical
in 2.5.

(file #21962)
_______________________________________________________

Additional Item Attachment:

File name: trunk-extra-ui-prototype.mbox Size:43 KB


_______________________________________________________

Reply to this item at:

<http://gna.org/patch/?3756>

_______________________________________________
Message sent via/by Gna!
http://gna.org/
Jacob Nevins
2014-10-25 14:21:32 UTC
Permalink
Follow-up Comment #6, patch #3756 (project freeciv):

Rebased, and patches now in other tickets removed from stack

Is anyone likely to object if I proceed with this idea?

(file #22736)
_______________________________________________________

Additional Item Attachment:

File name: trunk-extra-ui-prototype-bis.patch Size:36 KB


_______________________________________________________

Reply to this item at:

<http://gna.org/patch/?3756>

_______________________________________________
Message sent via/by Gna!
http://gna.org/
Marko Lindqvist
2014-10-25 14:52:07 UTC
Permalink
Post by Jacob Nevins
Is anyone likely to object if I proceed with this idea?
I'm still a bit uncertain, but I'm yet to come up with counter-proposal, and
I'm unlikely to think this a lot before S2_6 has been safely branched.

_______________________________________________________

Reply to this item at:

<http://gna.org/patch/?3756>

_______________________________________________
Message sent via/by Gna!
http://gna.org/
Marko Lindqvist
2015-04-21 22:28:25 UTC
Permalink
Update of patch #3756 (project freeciv):

Planned Release: 2.6.0 => 3.0.0

_______________________________________________________

Follow-up Comment #8:

It seems the discussion about going forward with this in 3.0 was never
recorded to this ticket.

_______________________________________________________

Reply to this item at:

<http://gna.org/patch/?3756>

_______________________________________________
Message sent via/by Gna!
http://gna.org/
Jacob Nevins
2015-10-24 11:31:26 UTC
Permalink
Update of patch #3756 (project freeciv):

Depends on: => patch #6475


_______________________________________________________

Reply to this item at:

<http://gna.org/patch/?3756>

_______________________________________________
Message sent via/by Gna!
http://gna.org/
Jacob Nevins
2015-10-24 13:09:48 UTC
Permalink
Additional Item Attachment, patch #3756 (project freeciv):

File name: trunk-extra-ui-prototype-ter.mbox Size:33 KB


_______________________________________________________

Reply to this item at:

<http://gna.org/patch/?3756>

_______________________________________________
Message sent via/by Gna!
http://gna.org/
Jacob Nevins
2015-10-24 13:10:53 UTC
Permalink
Follow-up Comment #9, patch #3756 (project freeciv):

Rebased on top of current trunk + patch #6475.
No change to functionality.

_______________________________________________________

Reply to this item at:

<http://gna.org/patch/?3756>

_______________________________________________
Message sent via/by Gna!
http://gna.org/
Jacob Nevins
2015-12-12 14:38:35 UTC
Permalink
Additional Item Attachment, patch #3756 (project freeciv):

File name: trunk-extra-ui-prototype-4.mbox Size:33 KB


_______________________________________________________

Reply to this item at:

<http://gna.org/patch/?3756>

_______________________________________________
Message sent via/by Gna!
http://gna.org/
Jacob Nevins
2016-03-10 00:51:07 UTC
Permalink
Additional Item Attachment, patch #3756 (project freeciv):

File name: trunk-extra-ui-prototype-r32218.mbox Size:33 KB


_______________________________________________________

Reply to this item at:

<http://gna.org/patch/?3756>

_______________________________________________
Message sent via/by Gna!
http://gna.org/
Marko Lindqvist
2017-05-03 23:07:54 UTC
Permalink
Update of patch #3756 (project freeciv):

Status: In Progress => Duplicate
Open/Closed: Open => Closed

_______________________________________________________

Follow-up Comment #10:

Handled at hostedredmine: https://www.hostedredmine.com/issues/657403

_______________________________________________________

Reply to this item at:

<http://gna.org/patch/?3756>

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

Loading...