patchFreeciv - Patches: patch #3711, Caravan popups as part of unit...

Show feedback again

patch #3711: Caravan popups as part of unit focus queue?

Submitted by:  Jacob Nevins <jtn>
Submitted on:  Sun 17 Feb 2013 10:23:00 AM UTC  
Category: clientPriority: 5 - Normal
Status: In ProgressPrivacy: Public
Assigned to: Sveinung Kvilhaugsvik <sveinung>Open/Closed: Open
Planned Release: 3.0.0, 2.6.0

Add a New Comment (Rich MarkupRich Markup):

You are not logged in

Please log in, so followups can be emailed to you.


Thu 19 Nov 2015 09:26:48 AM UTC, SVN revision 30683:

Get rid of separate action selection dialog queue

Get rid of diplomat_arrival_queue by merging it into the unit focus queue.

Requested by Jacob Nevins

(This patch doesn't implement everything asked for in patch #3711)

See patch #6602

(Browse SVN revision 30683)

Sveinung Kvilhaugsvik <sveinung>
Project MemberIn charge of this item.
Tue 19 Feb 2013 04:25:32 AM UTC, comment #2:

About obscuring the action, i wonder if it wouldn't be better if all other unit movements pause until the popup has been dealt with. But, i understand that would come with its own problems.

About forgetting that a caravan is ready, i don't think that's such a big problem. The caravan gets its focus back next turn and essentially just loses a turn. The worst case scenario is if the player forgets sending the caravan there and sends it out again.

David Lowe <doctorjlowe>
Sun 17 Feb 2013 10:33:27 AM UTC, comment #1:

Sounds much better than the minimal fix I've been wanting for the case when player is not logged in during turn change, misses the popups, and never gets them once (s)he logs in.
Currently caravan either automatically creates traderoute (if target was non-allied city) or just goes inside the city (if own or allied city).

Marko Lindqvist <cazfi>
Project Administrator
Sun 17 Feb 2013 10:23:00 AM UTC, original submission:

Following on from discussion in patch #2206:

In general, it does feel like we need a new workflow for caravan orders (and maybe other things like diplomat actions).

Having the popup occur in the flurry of turn-change activities was bad for various reason (e.g. taking focus, obscuring the action), which is why we have "Pop up caravan actions" as an option (r15533), but if you turn that off it's easy to forget that a caravan is ready.

Commercial CivIV has a fairly forced workflow, where it takes you through decisions like city production, promotions, and units in some fixed order. This helps not forget things, but it's still not ideal; if you don't want to deal with a thing at the point it prompts because you need other information (you're locked out of seeing all information due to the forced workflow), you have to cancel and remember to go back and review the thing later.

I wonder if there's scope for putting pending-caravan-decisions on the unit focus queue, and popping up the caravan dialog whenever focus gets to a unit, and allowing the usual unit focus keys like "W" to dismiss the popup non-permanently (so it will appear again next time the unit comes up)? In this world, a unit capable of an action would stay on the queue even if it had no movement points left.

That still doesn't quite fix the case where your first unit after turn change needs a popup, but you don't want the popup obscuring your view of the post-turn action -- although it'll help a bit as the popup will naturally pop up a bit later. Perhaps you want the option of some unobtrusive proxy for the popup, like a particular modification to the unit icon.

(Obviously having the specify-action-in-advance envisaged by bug #16905 would also help some players, but it would still be good to have a better workflow for those not making plans in advance.)

Jacob Nevins <jtn>
Project Administrator


(Note: upload size limit is set to 1024 kB, after insertion of the required escape characters.)

Attach File(s):

No files currently attached


   patch dependencies.

Items that depend on this one: None found


Carbon-Copy List
  • -unavailable- added by sveinung (Updated the item)
  • -unavailable- added by doctorjlowe (Posted a comment)
  • -unavailable- added by cazfi (Posted a comment)
  • -unavailable- added by jtn (Submitted the item)

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.


    Error: not logged in



    Follow 4 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 17 Nov 2015 12:51:32 PM UTCsveinungDependencies-=>Depends on patch #6602
    Fri 13 Nov 2015 02:08:50 PM UTCsveinungStatusNone=>In Progress
      Assigned toNone=>sveinung
      Planned Release=>3.0.0, 2.6.0
    Show feedback again

    Back to the top

    Powered by Savane 3.1-cleanup