[Tickets #10829] Re: Add missing units current wind condition

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/10829
------------------------------------------------------------------------------
  Ticket             | 10829
  Updated By         | arjen+horde at de-korte.org
  Summary            | Add missing units current wind condition
  Queue              | Horde Base
  Version            | Git master
  Type               | Bug
  State              | Feedback
  Priority           | 1. Low
  Milestone          |
  Patch              | 1
  Owners             | Michael Rubinsky
------------------------------------------------------------------------------


arjen+horde at de-korte.org (2011-12-04 21:13) wrote:

> I committed the fix for the precipitation chance, but I'm confused  
> about your other change. Why are you moving the spaces into the  
> gettext string?

Because in the forecast, the same string is used (line 321). Since not  
only the word but also the meaning is identical (wind direction and  
-speed), it made sense to me to use the same gettext string in both  
places. Since though the translation for ' at ' was already present, I  
changed the line earlier on to the same, so that other languages will  
benefit too without any effort from translators. Other than that, I  
have no preference for the space in or out of the gettext string.