bugLordsAWar! - Bugs: bug #24388, game preferences dialog doesn't...

 
 

bug #24388: game preferences dialog doesn't show tileset of loaded maps or scenarios.

Submitted by:  Ben Asselstine <benasselstine>
Submitted on:  Sat 27 Sep 2008 01:40:00 PM UTC  
 
Category: gameSeverity: 3 - Normal
Item Group: NoneStatus: None
Privacy: PublicAssigned to: None
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

Sun 14 Jun 2009 07:04:06 PM UTC, comment #2:

I have rearranged the new game menu, so that the game preferences dialog doesn't load a map or scenario any longer.

Ben Asselstine <benasselstine>
Project Administrator
Sun 28 Sep 2008 02:53:29 PM UTC, comment #1:

I implemented this feature, but I'm not going to commit it.

It doesn't make sense to show the tile size, tile theme, city theme, army theme and shieldset in the comboboxes provided in the random map frame. It doesn't make sense because the loaded scenario isn't a random map.

Even after implementing this feature of showing the tile/army/city/shield set -- changing the tile/army/city/shield set doesn't have the desired effect (because it's for random maps).

Is it desirable to let the player change the look of the loaded scenario in the game preferences dialog? Is this a case of Scenario-writer knows best? Maybe this is best done in the scenario editor by the scenario writer.

Is this a case of providing more flexibility than will ever be used?

Attaching the patch that I'm not commiting.

(file #16580)

Ben Asselstine <benasselstine>
Project Administrator
Sat 27 Sep 2008 01:40:00 PM UTC, original submission:

Loading a scenario (e.g. punic war) doesn't show right set size in the "New Game" dialog.

Although the army/city/shield/tile sets are greyed-out because they cannot be changed, they should still reflect the army/city/shield/tile sets that are going to be used for the game.

This also applies to loading a map in the game preferences dialog.

Ben Asselstine <benasselstine>
Project Administrator

 

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

Attach File(s):
   
   
Comment:
   

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by benasselstine (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):

     

     

    Follow 2 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sun 14 Jun 2009 07:04:06 PM UTCbenasselstineOpen/ClosedOpen=>Closed
    Sun 28 Sep 2008 02:53:29 PM UTCbenasselstineAttached File-=>Added show-sets-for-loaded-maps.patch, #16580

    Back to the top


    Powered by Savane 3.1-cleanup1