[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH v3] Conflict option labels

From: Stefan <luke1410_at_posteo.de>
Date: Fri, 14 Oct 2016 11:15:07 +0200

On 10/14/2016 10:27 AM, Stefan Sperling wrote:
> On Thu, Oct 13, 2016 at 05:59:01PM +0200, Stefan wrote:
>> On 10/13/2016 5:26 PM, Patrick Steinhardt wrote:
>>> sion re-adds the result pool to
>>> `svn_client_conflict_option_get_lazel`.
>>> diff --git a/subversion/include/svn_client.h
>>> b/subversion/include/svn_client.h
>>> index 9bbe62b..f456c92 100644
>>> --- a/subversion/include/svn_client.h
>>> +++ b/subversion/include/svn_client.h
>>> @@ -4718,6 +4718,20 @@ svn_client_conflict_option_id_t
>>> svn_client_conflict_option_get_id(svn_client_conflict_option_t *option);
>>>
>>> [...]
>>>
>>> add_resolution_option(*options, conflict,
>>> svn_client_conflict_option_merged_text,
>>> + _("Mark as resolved"),
>>> _("accept binary file as it appears in the working copy"),
>>> resolve_text_conflict);
>> Not sure whether "Mark as resolved" means much to the user. Maybe
>> "Accept/Use current version" would be easier to get? Or maybe
>> "Accept/Use current" to keep the label consistent with the style of the
>> others.
> To avoid yet another round-trip for Patrick, I'd prefer us to focus on
> the mechanical nature of this change, which is about moving these labels
> as-is into the library and exposing them via API.
>
> We can still tweak these strings after this change is committed (and I
> agree they should be changed).

+1

Received on 2016-10-14 11:15:13 CEST

This is an archived mail posted to the Subversion Dev mailing list.