Re: [PATCH] make mergeinfo diagram column headings l10n friendly

On 24.01.2013 23:16, Andreas Stieger wrote:
> On 24/01/13 15:53, Ben Reser wrote:
>> On Wed, Jan 23, 2013 at 4:43 PM, Branko Čibej <brane_at_wandisco.com> wrote:
>>> On 24.01.2013 00:42, Andreas Stieger wrote:
>>>> Would something like this not be better?
>>>>
>>>> youngest common ancestor
>>>> | last full merge
>>>> | | tip of branch
>>>> | | | repository path
>>>>
>>>> 2 6
>>>> | |
>>>> --| |------------ branches/branch1
>>>> / /
>>>> / /
>>>> -------| |------------ trunk
>>>> | |
>>>> 5 6
>> +0 on this.
>>
>>> The patch itself still makes translators deal with the position of the
>>> vertical bars. If we're going to change the layout, we may as well make
>>> it even easier for translators so that they don't have to worry about
>>> positioning, just translating the headings.
>> +1 on this.
> Based on that feedback, the updates patch will produce to the same
> output, but with the column headings translated as separate strings.

Thanks! This patch looks good to me. +1 to commit, but I'll leave it to
Julian to review.

-- Brane

-- 
Branko Čibej
Director of Subversion | WANdisco | www.wandisco.com