Bug 323053

Summary: commit 913219a357252bfaee15d0df88918d2c6e3a12cf broke live preview if "synchronize preview position..." action is enabled
Product: kile Reporter: Eugene Shalygin <eugene.shalygin+bugzilla.kde>
Component: generalAssignee: Michel Ludwig <michel.ludwig>
Severity: normal CC: umar
Priority: NOR    
Version: trunk-kde4   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: Illustration and possible fix

Description Eugene Shalygin 2013-07-31 16:30:39 UTC
When "synchronize preview position..." action is enabled, document viewer is not updated after recompilation. 

According to note for LivePreviewManager::ensureDocumentIsOpenInViewer(), it seems to me that hadToOpen and m_synchronizeViewWithCursorAction->isChecked() conditions have to be checked independently in LivePreviewManager::updatePreviewInformationAfterCompilationFinished()

Reproducible: Always
Comment 1 Eugene Shalygin 2013-07-31 16:31:34 UTC
Created attachment 81485 [details]
Illustration and possible fix

Thes changes fix the problem for me
Comment 2 Michel Ludwig 2013-09-01 18:53:47 UTC
Git commit a212fe70364e2e59af911a8619b7c272d1ce8876 by Michel Ludwig.
Committed on 01/09/2013 at 18:51.
Pushed by mludwig into branch 'master'.

Fix the reloading of the document after live preview compilations

Thanks to Eugene Shalygin for suggesting the fix!

M  +8    -6    src/livepreview.cpp

Comment 3 Michel Ludwig 2013-09-01 20:39:35 UTC
*** Bug 322527 has been marked as a duplicate of this bug. ***