bugFreeciv - Bugs: bug #19609, View Native Tiles & city dialog

 
 
Show feedback again

bug #19609: View Native Tiles & city dialog

Submitted by:  Marko Lindqvist <cazfi>
Submitted on:  Tue 03 Apr 2012 05:01:33 PM UTC  
 
Category: artSeverity: 3 - Normal
Priority: 5 - NormalStatus: Fixed
Assigned to: Jacob Nevins <jtn>Open/Closed: Closed
Release: Operating System: Any
Planned Release: 2.4.0, 2.5.0

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

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

 

(Jump to the original submission Jump to the original submission)

Sat 09 Feb 2013 10:22:11 AM UTC, SVN revision 22316:

Add dedicated non-native tile graphic for Amplio2, Amplio, and Cimpletoon.

Requested by Marko Lindqvist (cazfi@gna).

See gna bug #19609.

(Browse SVN revision 22316)

Jacob Nevins <jtn>
Project AdministratorIn charge of this item.
Sat 09 Feb 2013 10:21:29 AM UTC, SVN revision 22315:

Add dedicated non-native tile graphic for Amplio2, Amplio, and Cimpletoon.

Requested by Marko Lindqvist (cazfi@gna).

See gna bug #19609.

(Browse SVN revision 22315)

Jacob Nevins <jtn>
Project AdministratorIn charge of this item.
Thu 07 Feb 2013 01:13:57 AM UTC, comment #8:

Doesn't look too bad to me. I did make the stripe spacing fairly dense with this case in mind. (Will also affect experimental ruleset with units like Freight.)

I've nudged the phase of the striping slightly in this version to make it more symmetric.

Further possible improvements:

  • Fade at the edges, rather than the current hard boundary. (I didn't pursue this because I wasn't sure how consistent I could rely on the compositing to give a seamless result for large areas. It's probably fine. Also I'd have to find even more GIMPing skills.)
  • For cases where it's something like a road making a terrain native, we could allow tilesets to define a mask to apply to the nonnative marking so that just the outline of the road appears native rather than the whole tile. But that's probably overkill for this feature.

(file #17148, file #17149)

Jacob Nevins <jtn>
Project AdministratorIn charge of this item.
Thu 07 Feb 2013 12:41:20 AM UTC, comment #7:

Looks good at least with such large continuous areas. You may want to test with alien ruleset and Earthly units (startunits are Earthly) how it looks like with more fragmented native/non-native areas.

Marko Lindqvist <cazfi>
Project Administrator
Thu 07 Feb 2013 12:32:12 AM UTC, comment #6:

How about this for Amplio-based tilesets?
See attached screenshot.

Attached grid.png would go to data/amplio2/grid.png and data/amplio/grid.png on commit.

(It would be easy enough to do something similar for other tilesets.)

(file #17144, file #17145, file #17146)

Jacob Nevins <jtn>
Project AdministratorIn charge of this item.
Fri 01 Feb 2013 11:15:26 AM UTC, SVN revision 22249:

Do not display selected unit's native tile layer in citydlg map.

See gna bug #19609

(Browse SVN revision 22249)

Marko Lindqvist <cazfi>
Project Administrator
Fri 01 Feb 2013 11:15:22 AM UTC, SVN revision 22248:

Do not display selected unit's native tile layer in citydlg map.

See gna bug #19609

(Browse SVN revision 22248)

Marko Lindqvist <cazfi>
Project Administrator
Wed 30 Jan 2013 05:54:38 PM UTC, comment #3:

Setting category to "art" to appear in queries about missing art.

Marko Lindqvist <cazfi>
Project Administrator
Wed 30 Jan 2013 01:00:28 AM UTC, comment #2:

Fix for "It doesn't make sense to display information about the current unit(s) in the city map." part.

(file #17051)

Marko Lindqvist <cazfi>
Project Administrator
Sun 29 Jul 2012 08:21:56 PM UTC, comment #1:

To fix this I think we should fix both issues:

  • It doesn't make sense to display information about the current unit(s) in the city map.
  • We should have a different sprite for non-native tiles (perhaps one without borders). It shouldn't take much in the way of artistic skills...
Jacob Nevins <jtn>
Project AdministratorIn charge of this item.
Tue 03 Apr 2012 05:01:33 PM UTC, original submission:

Don't know which part should be considered a bug, but together these form a problem:

- If View Native Tiles is enabled and one has unit selected, tile nativity information is diplayed in city dialog also.
- Even default Aplio2 tileset reuses "tile unusable" sprite for non-native tiles.

End result: Having land unit selected while opening city dialog causes all ocean tiles to seem like unusable.

Marko Lindqvist <cazfi>
Project Administrator

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #17148:  nonnativealien.png added by jtn (68kB - image/png - Tweaked grid.png for symmetry, + screenshot of Alien World (isolated tiles))
file #17149:  grid.png added by jtn (11kB - image/png - Tweaked grid.png for symmetry, + screenshot of Alien World (isolated tiles))
file #17144:  grid.png added by jtn (12kB - image/png - trunk/S2_4 r22303: nonnative tile graphic for Amplio/2(/Cimpletoon), + example screenshot)
file #17145:  trunk-S2_4-amplio-nonnative.patch added by jtn (1kB - text/x-patch - trunk/S2_4 r22303: nonnative tile graphic for Amplio/2(/Cimpletoon), + example screenshot)
file #17146:  nonnativeamplio2.png added by jtn (208kB - image/png - trunk/S2_4 r22303: nonnative tile graphic for Amplio/2(/Cimpletoon), + example screenshot)
file #17051:  CitydlgNativeNot.patch added by cazfi (455B - text/x-diff)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by jtn (Posted a comment)
  • -unavailable- added by cazfi (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 12 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 09 Feb 2013 10:22:31 AM UTCjtnStatusReady For Test=>Fixed
      Assigned toNone=>jtn
      Open/ClosedOpen=>Closed
      Operating SystemNone=>Any
    Thu 07 Feb 2013 01:13:57 AM UTCjtnAttached File-=>Added nonnativealien.png, #17148
      Attached File-=>Added grid.png, #17149
    Thu 07 Feb 2013 12:32:12 AM UTCjtnAttached File-=>Added grid.png, #17144
      Attached File-=>Added trunk-S2_4-amplio-nonnative.patch, #17145
      Attached File-=>Added nonnativeamplio2.png, #17146
      StatusNone=>Ready For Test
    Wed 30 Jan 2013 05:54:38 PM UTCcazfiCategoryclient=>art
    Wed 30 Jan 2013 01:00:28 AM UTCcazfiAttached File-=>Added CitydlgNativeNot.patch, #17051
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup