[PATCH 1 of 3] monoblue: split a long line in filelogentry template

Pierre-Yves David pierre-yves.david at ens-lyon.org
Fri Sep 18 05:54:05 CDT 2015



On 09/18/2015 03:18 AM, Anton Shestakov wrote:
> On Fri, 18 Sep 2015 03:13:09 -0700
> Pierre-Yves David <pierre-yves.david at ens-lyon.org> wrote:
>
>>
>>
>> On 09/17/2015 11:42 PM, Anton Shestakov wrote:
>>> # HG changeset patch
>>> # User Anton Shestakov <av6 at dwimlabs.net>
>>> # Date 1442557424 -28800
>>> #      Fri Sep 18 14:23:44 2015 +0800
>>> # Node ID 8c583a8f33ac65e3d6ed1a08c22ed13b753891a4
>>> # Parent  3166bcc0c53840833e4223f123731fc420c0d3d3
>>> monoblue: split a long line in filelogentry template
>>>
>>> The whole td has a nowrap class, which disallows breaking the line,
>>> so it's not required to use non-breakable spaces.
>>>
>>> diff --git a/mercurial/templates/monoblue/map
>>> b/mercurial/templates/monoblue/map ---
>>> a/mercurial/templates/monoblue/map +++
>>> b/mercurial/templates/monoblue/map @@ -254,7 +254,9 @@ filelogentry
>>> = ' </a>
>>>        </td>
>>>        <td class="nowrap">
>>> -      <a
>>> href="{url|urlescape}file/{node|short}/{file|urlescape}{sessionvars%urlparameter}">file</a> | <a
>>> href="{url|urlescape}diff/{node|short}/{file|urlescape}{sessionvars%urlparameter}">diff</a> | <a
>>> href="{url|urlescape}annotate/{node|short}/{file|urlescape}{sessionvars%urlparameter}">annotate</a>
>>> +      <a
>>> href="{url|urlescape}file/{node|short}/{file|urlescape}{sessionvars%urlparameter}">file</a>
>>> |
>>> +      <a
>>> href="{url|urlescape}diff/{node|short}/{file|urlescape}{sessionvars%urlparameter}">diff</a>
>>> |
>>> +      <a
>>> href="{url|urlescape}annotate/{node|short}/{file|urlescape}{sessionvars%urlparameter}">annotate</a>
>>
>> Sounds like the non-breaking-space have been lost in the process.
>> Should we get worried?
>>
>
> No, this is intended, since they are superfluous (because of the nowrap
> class all spaces inside the td are non-breakable).

As you had nicely pointed out in the changeset description…

Pushed to the clowncopter, thanks.

-- 
Pierre-Yves David


More information about the Mercurial-devel mailing list