Am 19.08.19 um 23:41 schrieb Pratyush Yadav:
> Just like the user can select lines to stage or unstage, add the
> ability to revert selected lines.
> 
> Signed-off-by: Pratyush Yadav <m...@yadavpratyush.com>

> +     if {$revert} {
> +             set query "[mc "Revert changes in file %s?" \
> +                     [short_path $current_diff_path]]
> +
> +[mc "The selected lines will be permanently lost by the revert."]"
> +
> +             set reply [revert_dialog $query]

Please don't do this. This confirmation dialog is unacceptable in my
workflow. I use reversals of hunks and lines frequently, almost like a
secondary code editor. My safety net is the undo function of the IDE,
which works across reloads that are triggered by these external edits.
These confirmations get in the way.

> +             if {$reply ne 1} {
> +                     unlock_index
> +                     return
> +             }
> +     }
> +
>       set wholepatch {}
>  
>       while {$first_l < $last_l} {
> 

-- Hannes

Reply via email to