On Tue, Dec 19, 2006 at 02:50:46PM +0100, Jean-Marc Lasgouttes wrote: > >>>>> "Abdelrazak" == Abdelrazak Younes <[EMAIL PROTECTED]> writes: > > Abdelrazak> Abdelrazak Younes wrote: > >> Jean-Marc Lasgouttes wrote: > >>> When selecting a collapsable inset with the mouse, it opens at > >>> mouse release time. This is very very annoying... > >> Why is that annoying? > >> > >>> The following pair of patches fix this. Objections? > >> Yes. I would imagine that "mouse click on inset" should be > >> reserved for dragging this inset with the mouse (if coupled with > >> mouse-motion), independently from the open state. Your patch > >> prevent from implementing that. > > Abdelrazak> I think I misread the patch and that my understanding > Abdelrazak> above was wrong. In that case, please forget my objection. > > Yes, I think you got it wrong. What the patch changes is what happens > on mouse1 release. Normally it is 'toggle the inset', the patch is > supposed to inhibit that when there is a selection (since we know we > are currently making this selection, since mouse1 is down).
I fear that you exchanged an annoyance with a crash... 1) load the attached file 2) select all (including the ERT inset) 3) without releasing mouse1, slide the cursor over the ERT inset 4) release mouse1: LyX asserts Assertion triggered in void lyx::LyXText::dispatch(lyx::LCursor&, lyx::FuncRequest&) by failing check "cur.text() == this" in file ../../src/text3.C:323 I am not able to get a backtrace, though. -- Enrico
#LyX 1.5.0svn created this file. For more info see http://www.lyx.org/ \lyxformat 255 \begin_document \begin_header \textclass article \language english \inputencoding auto \font_roman default \font_sans default \font_typewriter default \font_default_family default \font_sc false \font_osf false \font_sf_scale 100 \font_tt_scale 100 \graphics default \paperfontsize default \papersize default \use_geometry false \use_amsmath 1 \use_esint 1 \cite_engine basic \use_bibtopic false \paperorientation portrait \secnumdepth 3 \tocdepth 3 \paragraph_separation indent \defskip medskip \quotes_language english \papercolumns 1 \papersides 1 \paperpagestyle default \tracking_changes false \output_changes false \author "Enrico Forestieri" \end_header \begin_body \begin_layout Standard aaaa \begin_inset ERT status collapsed \begin_layout Standard \backslash relax \end_layout \end_inset \end_layout \end_body \end_document
