Code review comment for lp:~hid-iwata/tortoisebzr/cmenu_customize

Hi!

On Sat, 2010-12-04 at 03:59 +0000, iwata wrote:
> iwata has proposed merging lp:~hid-iwata/tortoisebzr/cmenu_customize into lp:tortoisebzr.
>
> Requested reviews:
> TortoiseBZR Developers (tortoisebzr-developers)
>
>
> Context menu customizing feature
> ---------------------------------
> Make it possible to reorder context menus or hide unused menus as users like.
> Related bug : #591774
>
> Drive type filter and path filter for context menu
> -----------------------------------------------------------
> As same as icon overlays, make it possible to define folders on which tbzr avoids showing context menus, with drive type filters and path wildcards.
>
> .. note::
> Some of menus (e.g. ``settings``) are always shown regardless of these settings.
>
> A little improvement for context menu filtering
> ------------------------------------------------
> Hide obviously useless menus (e.g. ``annotate`` when no file selected) even when status of files/folders are unavailable.
Thanks for the patch, this seems like a useful addition. Is there any
chance you can split this branch up a bit ? That might make it a bit
easier for reviewer(s) to review and merge it.

Cheers,

Jelmer