patchFreeciv - Patches: patch #3454, Multiple selectable flagsets per...

 
 
Show feedback again

patch #3454: Multiple selectable flagsets per tileset

Submitted by:  Jacob Nevins <jtn>
Submitted on:  Sat 28 Jul 2012 01:22:27 PM UTC  
 
Category: clientPriority: 5 - Normal
Status: NonePrivacy: Public
Assigned to: NoneOpen/Closed: Open
Planned Release: 

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

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

 

Sat 18 Aug 2012 11:56:26 PM UTC, comment #1:

You have to handle "occupied" icon (on top of flag) in Amplio2 somehow when supporting multiple flag sizes. It was fixed to match larger flag size now used in Amplio2, but that broke it for me who use smaller flags.

Marko Lindqvist <cazfi>
Project Administrator
Sat 28 Jul 2012 01:22:27 PM UTC, original submission:

I find the large flags used with Amplio2, while beautiful, do get in my way rather. They cause the citybar to be much bigger, and various dialogs / lists to be spaced out more. This is particularly noticeable on small screens.

Half-baked idea to solve this while still allowing them to be used:

Allow a single tileset to define multiple flagsets. The tileset specifies a default, but there's a client option to override it.

Since a client may switch between tilesets with different flagsets, the situation where the client-selected flagset isn't in the tileset needs handling gracefully. There should be a "default" option which always picks the tileset's default; the client UI should probably be a combo box rather than pure enumerated dropdown; and if the current non-default preference isn't in the current tileset, it should fall back silently to the tileset default. Naming conventions for flagsets will help here too.
(Unless we want to somehow support saving tileset-specific client options.)

So for Amplio2 (and probably the other supplied tilesets), we'd define "standard" and "large" flagsets, with Amplio2 at least defaulting to "large".

This will require some shuffling of where certain graphics live. For instance, the non-citybar occupied sprite is tied to the flag size, so needs to change with the selected flagset (although in principle it could still be per-flagset-per-tileset).

Jacob Nevins <jtn>
Project Administrator

 

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

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -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.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    No Changes Have Been Made to This Item
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup