Re: [PATCH] TortoiseMerge: report if files are identical and optionally exit

Stefan Küng wrote:
> Flavio Stanchina wrote:
>> Ian Anderson wrote:
>>> Personally I would find this dialog annoying as well - message boxes
>>> interrupt workflow. If this does get implemented as an option, I would
>>> like to see it disabled by default.
>>
>> Completely agree on both counts. As I said in the previous reply, we
>> want this feature only when we're working from within the Visual Studio
>> IDE, so it would be disabled by default and it could be enabled
>> permanently with a checkbox or just for this run with a command line
>> option.
>
> If you only want this for your VS plugin, then I'd say we should just
> show the checkbox if the command line option is given.

If it is only for a VS plugin, you don't need a checkbox at all, do you?
Users can set up their IDE with or without the command line option.

The warning dialog needs a bit of care. You probably want ESC to close
both the dialog and TMerge in one hit, and Enter to continue viewing. So
you might have a default button [ View files ] and a [ Cancel ] button.
Sorry Flavio, I didn't look at the patch - you may already have done it
this way.

Simon

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org