Vincent van Ravesteijn wrote:
>Is the attached ok ?

Is strikes me a good step. I detected one failure only:

> Index: src/insets/InsetIndex.h
> ===================================================================
> --- src/insets/InsetIndex.h     (revision 29316)
> +++ src/insets/InsetIndex.h     (working copy)
> @@ -45,7 +45,7 @@
>         static void string2params(std::string const &, InsetIndexParams &);
>  private:
>         ///
> -       EDITABLE editable() const { return HIGHLY_EDITABLE; }
> +       bool hasSettings() const { return false; }

This should be the same as PrintIndex: InsetIndex hasSettings if multiple 
indices is selected.

The old, always IS_EDITABLE setting is probably a remainder from the pre-
collapsable time of IndexInset, when it always had a dialog.

Jürgen

Reply via email to