LaTeX and chktex error handling problems.

[John C. McCabe-Dansted]

The new "LaTeX Errors" Error dialog box has some problems:

• It always jumps back to and highlights the first error when I go back to the main LyX window, regardless of which error I clicked on in the dialog box.

• It makes it hard to edit the text, as it keeps the (first) error

selected/highlighted, typing anything deletes the error text.

• Ctrl+G no longer selects the next error. This may be intentional, but it seems that it would be very handy if Ctrl+G brought the LaTeX error dialog to the foreground and jumped to the next error.

My system:
LyX-1.4.0pre3
kubuntu 5.10 (upgraded from preview)
gcc 4.02
QT 3.3.4
KDE 3.5
Intel(R) Pentium(R) 4 CPU 1500MHz

[Juergen Spitzmueller]

confirmed. Also, if you undo some action while the error dialog is open, the
par IDs change, and the dialog does not find the errors anymore.
This is really nasty.
Note, however, that the selection doesn't jump back if you really close the
dialog, and I think the dialog should be closed anyway, because the error
status changes when you edit the document.
I propose the following:

• add a menu option to reopen the error dialog (which probably has to rerun

latex or whatever the last action was)

• reimplement error-next which opens the dialog and jumps to the next error

from cursor position

• disallow editing while the error dialog is open

or, more elegant:

• the error dialog automatically updates its information when the document has

been edited. This might be not trivial, though (simply due to the par id
situation).

[lasgouttes]

The par id situation is now better, fixing the way the error dialog works would be
very welcome indeed.

[lasgouttes]

Fixed in 1.4 branch. Abdel is making changes in trunk, this will have to be
fixed later

Url: ​http://www.lyx.org/trac/changeset/14655
Log:
Fix #2179

• controllers/ControlErrorList.h (unreadlist_): new member, tells whether the errorlist has been read by the dialog.

• controllers/ControlErrorList.C (errorList, initialiseParams): update unreadlist_ as needed.

• qt2/QErrorList.C (update_contents): do nothing if error list has not been updated (#2179).

• xforms/FormErrorList.C (update): do not call updateContents if the error list has not been updated (#2179).

[Juergen Spitzmueller]

Removing "fixedintrunk" keyword, because that's exactly what it's not.

[lasgouttes]

And there is still the part that Ctrl+G does not work. This is now #2775.

[Juergen Spitzmueller]

Jean-Marc, what's the status of this? Should this be fixed for 1.5.0?

[John C. McCabe-Dansted]

It was fixed in 1.4.3, except for the bit split off as #2775. I hope it is
still fixed in 1.5.0.

[Juergen Spitzmueller]

As I understand it, the fix has been applied to the 1.4 branch back then, but
not to trunk (I have not checked that, though). If it's not too complicated, it
should be fixed in 1.5 as well before release.

[younes]

I fixed it differently in 1.5svn.

​http://www.lyx.org/trac/changeset/16848

[John C. McCabe-Dansted]

This bug appears to have reoccurred in LyX 1.6.x, in particular, I can reproduce
it in 1.6.2svn.

[Vincent van Ravesteijn]

Fixed in trunk at r32222.

[Juergen Spitzmueller]

Looks good. Go ahead.

[Vincent van Ravesteijn]

Fixed in branch at r32417.

[Juergen Spitzmueller]

1.6.6 is done.