Uwe Stöhr wrote:
> >> My patch is ready to go in I think. The only thing that is missing is
> >> visual feedback for the indentation width. As LyX source code is not
> >
> > you have totally ignored my previous questions, and i'm especially
> > interested in the first one.
>
> What do you mean? Everything you mentioned in 
> http://www.mail-archive.com/lyx-devel@lists.lyx.org/msg153128.html is now 
> fixed.

you are right, setting indentation to 0 now works which was my main concern.
thank you.

i still see small glitch with this code:

+               if (indentation == "default")
+                       pimpl_->indentation = Length(Length::DEFAULT);
+               else
+                       pimpl_->indentation = Length(indentation);

imho the right code would look like just:
        pimpl_->indentation = Length(indentation);

and the switch for "default" would be inside Length constructor.
anyway thats only minor thing.

pavel

Reply via email to