[Freeciv-Dev] [bug #25022] Qt client: Unused font options

2016-09-05 Thread Marko Lindqvist
Follow-up Comment #3, bug #25022 (project freeciv):

Remember that S2_6 client options file format is frozen, so you can't make big
changes in S2_6 (or S2_5).

___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7650] civ2civ3: require a Marketplace to enter

2016-09-05 Thread Marko Lindqvist
Update of patch #7650 (project freeciv):

 Planned Release:  2.6.0-beta1, 3.0.0 => 3.0.0  

___

Follow-up Comment #2:

So maybe we leave this out from 2.6, and reconsider situation in 3.0 where
ruleset can take map size in to account.

___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [task #7963] Tracking release 2.6.0-beta1

2016-09-05 Thread Marko Lindqvist
Update of task #7963 (project freeciv):

  Dependency Removed: => patch #7650


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #24989] City bombardment leaks a lot of information to attacker

2016-09-05 Thread David Fernandez
Follow-up Comment #1, bug #24989 (project freeciv):

I confirm that bombardment can be used to get this useful information. I
personally like it, but I admit it might be too much. What would you suggest?

At least, bombers are available at the same time than spies, that can
investigate the city without dying. But this issue could be a serious
limitation to give bombardment ability to early units like catapults or
frigates.

___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7650] civ2civ3: require a Marketplace to enter

2016-09-05 Thread David Fernandez
Follow-up Comment #1, patch #7650 (project freeciv):

I liked this idea when sveinung talked about it in patch 5995, but then I
remembered that these one-time bonuses from caravans can be "exploited",
specially in big maps, to get an endless source of gold. In multiplayer games
they use to forbid it, and I did the same in patch #7137.

As I understand it, the source of the problem is that the revenue when the
caravan enters the marketplace is proportional to the distance (between the
cities), with no max limit. If the map is big enough, it exists a distance
where the revenue is much greater than the cost of buying the caravan, and it
might be overpowered.

___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #25022] Qt client: Unused font options

2016-09-05 Thread Louis Moureaux
Follow-up Comment #2, bug #25022 (project freeciv):

If we change that much, let's go the other way around and define what we
keep.

*General fonts*
* Default
* Mono -- for text formatted in ASCII-art, excluding help content
* Title -- a bigger, bold font

*Map fonts*
* City name
* City production

*Domain-specific fonts*
* Chat -- the default is mono, but some may prefer a proportional font
* Help content -- mono, but has some long texts too
* Tech tree

This brings us to 8 fonts (down from 15). Gone are:
* help label -> default,
* help title -> title
* beta label -> title,
* reports (=notify label) -> default or mono,
* tile info (=comment label) -> default,
* city label -> mono

I don't see the need for a small font: if some layout needs one, then it's
simply not good enough.
Migration from 2.5 will need to be taken care of carefully.

Does it sound good ?

___

Reply to this item at:

  

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #6108] Assert that a connection from the all_connections list aren't NULL.

2016-09-05 Thread Jacob Nevins
Update of patch #6108 (project freeciv):

 Planned Release:   3.0.0 => 2.6.0, 3.0.0   


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [bug #25038] Fuel not shown in new sidebar unit display

2016-09-05 Thread mir3x
Update of bug #25038 (project freeciv):

  Status:None => Ready For Test 
 Assigned to:None => mir3x  
 Contains string changes:None => Yes

___

Follow-up Comment #1:

Fixed in sidebar and unit selector,
but I changed it to moves_left(moves_total)
(unit_activity_astr() - shows it moves_total(moves_left))

(file #28566)
___

Additional Item Attachment:

File name: qt_mp.patchSize:2 KB


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7670] Qt client: Update the font database for all fonts

2016-09-05 Thread mir3x
Update of patch #7670 (project freeciv):

  Status:None => Ready For Test 
 Assigned to:None => mir3x  
 Planned Release: => 2.6.0, 3.0 
 Contains string changes:None => No 


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7645] Depleting Goods

2016-09-05 Thread Marko Lindqvist
Follow-up Comment #2, patch #7645 (project freeciv):

- Fix cancelling of "Depletes" trade route when it's not depleted, but cities
cannot trade for some other reason

(file #28565)
___

Additional Item Attachment:

File name: DepletingGoods-3.patch Size:12 KB


___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #7671] for(

2016-09-05 Thread Marko Lindqvist
URL:
  

 Summary: for(
 Project: Freeciv
Submitted by: cazfi
Submitted on: Mon 05 Sep 2016 12:24:37 PM EEST
Category: general
Priority: 5 - Normal
  Status: Ready For Test
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 3.0.0
 Contains string changes: None

___

Details:

Codingstyle fix: Add space to the remaining cases where it's missing between
'for' and '('. Made also some other style corrections in the neighborhood of
these corrections.



___

File Attachments:


---
Date: Mon 05 Sep 2016 12:24:37 PM EEST  Name: ForSpaceBrace.patch.bz2  Size:
7kB   By: cazfi



___

Reply to this item at:

  

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev