UFO: Alien Invasion Issue Tracker
UFO: Alien Invasion
Go to the previous open issue
Go to the previous issue (open or closed)
star_faded.png
Please log in to bookmark issues
icon_project.png UFO: Alien Invasion / Closed Submit Patch #4749 Sorted aircraft list by distace to target
Go to the next issue (open or closed)
Go to the next open issue
This issue has been closed with status "Closed" and resolution "Not determined".
Issue basics
  • Type of issue
    Submit Patch
  • Category
    General
  • Targetted for
    Not determined
  • Status
    Closed
  • Priority
    3. Normal
User pain
  • Type of bug
    Not triaged
  • Likelihood
    Not triaged
  • Effect
    Not triaged
Affected by this issue (1)
People involved
Times and dates
  • Posted at
  • Last updated
Issue details
Attachments (0)
There is nothing attached to this issue
Duplicate issues (0)
This issue does not have any duplicates
Description
[http://sourceforge.net/p/ufoai/patches/622 Item 622] imported from sourceforge.net tracker on 2013-01-28 20:46:59

Very useful when player have many bases and many aircrafts.
===== Comments Ported from Sourceforge =====

====== nokim (2011-07-31 04:59:16) ======


====== geever (2011-07-31 08:34:02) ======

It could work, however I think such things should be handled in a more generic way - by UI node(s) for example.
As second this patch changes sorting order permanently, it's not switchable.
Third: it violates our coding guidelines almost everywhere.

-geever
====== tlh2000 (2011-07-31 11:02:20) ======

see http://mattn.ninex.info/patches/0001-implemented-sorting-in-ui-text-nodes.patch
====== tlh2000 (2011-07-31 11:36:40) ======

this might work, too: http://mattn.ninex.info/patches/0001-sort-the-aircraft-list-in-the-interception-popups.patch
====== tlh2000 (2011-07-31 11:36:59) ======


====== tlh2000 (2011-07-31 11:37:15) ======


====== tlh2000 (2011-07-31 11:53:17) ======

the feature is now implemented in current master - 0001-sort-the-aircraft-list-in-the-interception-popups.patch was used for this - see dccc07771e2b8cec27f53ca6697f9660db77113e
====== nokim (2011-07-31 19:26:19) ======

It works - on my build at least. But if implementation violates coding guidelines (how? for next attempt if any)... and it's implemented anyway...

But this is one feature i strongly lacked. Having two aircraft on one base one in beginning of list, other in the end and interception mission each day... Another one is sorted list of employees by one chosen stat.
====== geever (2011-07-31 21:55:06) ======

Hello

First of all: I owe an apology. These days are bit crowded with Real Life stuff an I'm happy if I can steal time for coding. I'm sorry I didn't make it clear that we really appreciate your work and hoping for more!

I like mattn's version better as it's closer to the generic way I suggested.

Yes, for sure there are other areas especially on UI that can be improved, go ahead! :)

Coding guidelines: http://ufoai.ninex.info/wiki/index.php/Coding_guidelines

-geever
Todos (0 / 0)
Issue created
footer_logo.png The Bug Genie 4.3.1 | Support | Feedback spinning_16.gif