[PATCH] Show only player owned systems in production window

Programmers discuss here anything related to FreeOrion programming. Primarily for the developers to discuss.

Moderator: Committer

Message
Author
zhur
Space Floater
Posts: 38
Joined: Thu Aug 09, 2012 8:15 am

Re: [PATCH] Show only player owned systems in production win

#16 Post by zhur » Sun Aug 12, 2012 4:05 pm

Here, this is a fix for small inconsistency: when a player selects in production window a system without colonized planets, previously selected planet remains selected even if it is in another system. With this patch no planet is always selected for such systems.

Edit: the patch is licensed under GPLv2 or later.
Attachments

[The extension patch has been deactivated and can no longer be displayed.]

Last edited by zhur on Sat Aug 25, 2012 7:14 pm, edited 1 time in total.

User avatar
em3
Vacuum Dragon
Posts: 510
Joined: Sun Sep 25, 2011 2:51 pm

Re: [PATCH] Show only player owned systems in production win

#17 Post by em3 » Sun Aug 12, 2012 5:38 pm

Geoff the Medio wrote:Having clicking or double-clicking a system do different things depending on not-necessarily-obvious details of the system doesn't seem like a good idea. A particular user action should have a consistent result, be that showing the system or opening the production screen to a system.

Even if a system doesn't have any valid production locations for an empire, there's no reason to make it impossible to view the system in the production screen, particularly if doing so means the same UI action would do one thing in some situations, do nothing in other situations, or do something else in yet other situations. This could be confusing for players.
This is a feature I really like in latest games in Civilization series. Clicking on your own city brings building screen. Clicking on other empire's city brings diplomacy screen. Completely different results follow seemingly same UI action... Unless you consider clicking your own asset and clicking an asset of another player separate UI actions. Context is clear in this example.

I'm not sure what is not clear in FreeOrion's example of double clicking systems. You either have colonies in a system, or you don't. If you don't know this much, you are a confused user anyway. ;)
[...] for Man has earned his right to hold this planet against all comers, by virtue of occasionally producing someone totally batshit insane. - Randall Munroe, title text to xkcd #556

Post Reply