Re: [patch] fix bug 2907

2007-09-12 Thread Helge Hafting
Jürgen Spitzmüller wrote: Uwe Stöhr wrote: That the dialog is not opened has always been very annoying for me, therefore I tried to fix this. For me, it is annoying if it is always opened. What do others think? I think this depends a lot on the type of text being indexed. Some wil

Re: [patch] fix bug 2907

2007-09-10 Thread Abdelrazak Younes
Jürgen Spitzmüller wrote: Uwe Stöhr wrote: That the dialog is not opened has always been very annoying for me, therefore I tried to fix this. For me, it is annoying if it is always opened. What do others think? I think I agree with you. I personally never change the text so I remember bein

Re: [patch] fix bug 2907

2007-09-10 Thread Abdelrazak Younes
Uwe Stöhr wrote: > For me, it is annoying if it is always opened. OK, but have you already indexed some documents? Yes, otherwise I wouldn't comment. The larger the number of index entries the more it will annoy you. I admit that my document were not large... Abdel.

Re: [patch] fix bug 2907

2007-09-10 Thread Abdelrazak Younes
Jürgen Spitzmüller wrote: Jürgen Spitzmüller wrote: Furthermore, it doesn't fix bug 2907: C-i still doesn't open the inset's dialog. Do fix this (which also applies to url, nomenclature and others), you'd need the change LFUN_NEXT_INSET_TOGGLE or LFUN_INSET_TOGGLE, respectively. But then. you ha

Re: [patch] fix bug 2907

2007-09-10 Thread Jean-Marc Lasgouttes
Uwe Stöhr <[EMAIL PROTECTED]> writes: >> I'd prefer 2 lfuns. > > But then users have to edit the bind file to switch between the > functions. This is not user friendly. chaning an option in the prefs > is intuitive and quicker. No, I would propose two keyboard bindings. Menus are not relevant, si

Re: [patch] fix bug 2907

2007-09-10 Thread Uwe Stöhr
> I'd prefer 2 lfuns. But then users have to edit the bind file to switch between the functions. This is not user friendly. chaning an option in the prefs is intuitive and quicker. regards Uwe

Re: [patch] fix bug 2907

2007-09-10 Thread Jean-Marc Lasgouttes
[EMAIL PROTECTED] (Jürgen Spitzmüller) writes: > I have tested (a bit) and it works for me. The only thing that could > happen is that inset->edit() is called somewhere which would now > open the dialog unintentionally (which was the case in Text3.cpp). > > So I guess I'll check for such instances

Re: [patch] fix bug 2907

2007-09-10 Thread Jürgen Spitzmüller
Jean-Marc Lasgouttes wrote: > > How about the attached? > > It looks good to me (but I have no time to try it out). I have tested (a bit) and it works for me. The only thing that could happen is that inset->edit() is called somewhere which would now open the dialog unintentionally (which was the

Re: [patch] fix bug 2907

2007-09-10 Thread Jean-Marc Lasgouttes
[EMAIL PROTECTED] (Jürgen Spitzmüller) writes: >> I see. I'll have a look. > > How about the attached? It looks good to me (but I have no time to try it out). JMarc

Re: [patch] fix bug 2907

2007-09-10 Thread Jürgen Spitzmüller
Jürgen Spitzmüller wrote: > > Sorry indeed. InsetCommand should maybe have something like > > InsetGraphics: > > void InsetGraphics::edit(Cursor & cur, bool) > > { > >         InsetGraphicsMailer(*this).showDialog(&cur.bv()); > > } > > > > (and then should use it when handling MOUSE_RELEASE). > > I

Re: [patch] fix bug 2907

2007-09-10 Thread Jürgen Spitzmüller
Jean-Marc Lasgouttes wrote: > Sorry indeed. InsetCommand should maybe have something like > InsetGraphics: > void InsetGraphics::edit(Cursor & cur, bool) > { >         InsetGraphicsMailer(*this).showDialog(&cur.bv()); > } > > (and then should use it when handling MOUSE_RELEASE). I see. I'll have a

Re: [patch] fix bug 2907

2007-09-10 Thread Jean-Marc Lasgouttes
[EMAIL PROTECTED] (Jürgen Spitzmüller) writes: > Jean-Marc Lasgouttes wrote: >> You should definitely not use these xxx_CODE tests. This is wrong in >> 99% of cases. Test whether inset->editable() == IS_EDITABLE and then >> call inset->edit(). The code will be much shorter IMO. > > Doesn't seem to

Re: [patch] fix bug 2907

2007-09-10 Thread Jean-Marc Lasgouttes
Uwe Stöhr <[EMAIL PROTECTED]> writes: > Concerning the dialog display, I think different types of text > requires a different action. So the only solution I see is to have an > option in the preferences where the user can switch between display or > not. I'd prefer 2 lfuns. JMarc

Re: [patch] fix bug 2907

2007-09-10 Thread Uwe Stöhr
>> Not that besides the many sub level entries you also have many entries >> at top level that consists of more than one word. So also in these >> cases, you have to modify the index entry in the dialog. > > You just select the string and insert an index entry. I didn't know this. I'll add this i

Re: [patch] fix bug 2907

2007-09-10 Thread Jürgen Spitzmüller
Jean-Marc Lasgouttes wrote: > You should definitely not use these xxx_CODE tests. This is wrong in > 99% of cases. Test whether inset->editable() == IS_EDITABLE and then > call inset->edit(). The code will be much shorter IMO. Doesn't seem to work. Inset->edit() doesn't open the dialog (not surpri

Re: [patch] fix bug 2907

2007-09-10 Thread Jean-Marc Lasgouttes
[EMAIL PROTECTED] (Jürgen Spitzmüller) writes: > Jürgen Spitzmüller wrote: >> Furthermore, it doesn't fix bug 2907: C-i still doesn't open the inset's >> dialog. Do fix this (which also applies to url, nomenclature and others), >> you'd need the change LFUN_NEXT_INSET_TOGGLE or LFUN_INSET_TOGGLE,

Re: [patch] fix bug 2907

2007-09-10 Thread Jürgen Spitzmüller
Uwe Stöhr wrote: > OK, but have you already indexed some documents? Sure. > The larger the number of index entries the more it will annoy you. I guess we simply index different types of text. Jürgen

Re: [patch] fix bug 2907

2007-09-10 Thread Jean-Marc Lasgouttes
Uwe Stöhr <[EMAIL PROTECTED]> writes: >> When you create an index you have in 90% of the cases to modify the entry. >> >> where do you get the statistics from? > > See the new UserGuide or EmbeddedObjects manual. Only about 10% of the > entries are exactly the one word that is in front of the inde

Re: [patch] fix bug 2907

2007-09-10 Thread Jean-Marc Lasgouttes
[EMAIL PROTECTED] (Jürgen Spitzmüller) writes: >> The fix I sent increases the workflow a lot and I don't know why this was >> removed in LyX 1.4.x. > > I think we even had a request for this. Yes. Not everybody has the same use of indices. So some people complained that closing the dialog everyt

Re: [patch] fix bug 2907

2007-09-10 Thread Bennett Helm
On Sep 10, 2007, at 7:57 AM, Jürgen Spitzmüller wrote: Uwe Stöhr wrote: That the dialog is not opened has always been very annoying for me, therefore I tried to fix this. For me, it is annoying if it is always opened. What do others think? I think the 90% figure for modifying index entries

Re: [patch] fix bug 2907

2007-09-10 Thread Uwe Stöhr
> For me, it is annoying if it is always opened. OK, but have you already indexed some documents? The larger the number of index entries the more it will annoy you. regards Uwe

Re: [patch] fix bug 2907

2007-09-10 Thread Jürgen Spitzmüller
Uwe Stöhr wrote: > That the dialog is not opened has always been very annoying for me, > therefore I tried to fix this. For me, it is annoying if it is always opened. What do others think? Jürgen

Re: [patch] fix bug 2907

2007-09-10 Thread Uwe Stöhr
> When you create an index you have in 90% of the cases to modify the entry. > > where do you get the statistics from? See the new UserGuide or EmbeddedObjects manual. Only about 10% of the entries are exactly the one word that is in front of the index entry box. Not that besides the many sub le

Re: [patch] fix bug 2907

2007-09-10 Thread Jürgen Spitzmüller
Uwe Stöhr wrote: > When you create an index you have in 90% of the cases to modify the entry. where do you get the statistics from? > In indexes you group entries, therefore you have lots of second and > third-level entries like e.g. > > Figures ! Floats Not always. > I indexed a lot the last t

Re: [patch] fix bug 2907

2007-09-10 Thread Uwe Stöhr
> However, I think this is the wrong approach: it opens the index dialog > unnecessarily (i.e., if the word-at-cursor is filled in, which is often the > desired behaviour, and closing the dialog each and every time is annoying). When you create an index you have in 90% of the cases to modify the

Re: [patch] fix bug 2907

2007-09-10 Thread Jürgen Spitzmüller
Jürgen Spitzmüller wrote: > Furthermore, it doesn't fix bug 2907: C-i still doesn't open the inset's > dialog. Do fix this (which also applies to url, nomenclature and others), > you'd need the change LFUN_NEXT_INSET_TOGGLE or LFUN_INSET_TOGGLE, > respectively. But then. you have to take care that

Re: [patch] fix bug 2907

2007-09-09 Thread Jürgen Spitzmüller
Uwe Stöhr wrote: > Attached is a batch for bug 2907 > http://bugzilla.lyx.org/show_bug.cgi?id=2907 > > It shows the index dialog like in LyX 1.3.x and previous. > > The code is stolen from LFUN_NOMENCL_INSERT and works as expected. > If there are no objections, can this go into branch? If any, I w

Re: [patch] fix bug 2907

2007-09-09 Thread Martin Vermeer
On Mon, Sep 10, 2007 at 12:44:56AM +0200, Uwe Stöhr wrote: > Attached is a batch for bug 2907 > http://bugzilla.lyx.org/show_bug.cgi?id=2907 > > It shows the index dialog like in LyX 1.3.x and previous. > > The code is stolen from LFUN_NOMENCL_INSERT and works as expected. > If there are no objec

[patch] fix bug 2907

2007-09-09 Thread Uwe Stöhr
Attached is a batch for bug 2907 http://bugzilla.lyx.org/show_bug.cgi?id=2907 It shows the index dialog like in LyX 1.3.x and previous. The code is stolen from LFUN_NOMENCL_INSERT and works as expected. If there are no objections, can this go into branch? regards Uwe Index: Text3.cpp ==