Bug 323053

Summary:	commit 913219a357252bfaee15d0df88918d2c6e3a12cf broke live preview if "synchronize preview position..." action is enabled
Product:	[Applications] kile	Reporter:	Eugene Shalygin <eugene.shalygin+bugzilla.kde>
Component:	general	Assignee:	Michel Ludwig <michel.ludwig>
Status:	RESOLVED FIXED
Severity:	normal	CC:	umar
Priority:	NOR
Version:	trunk-kde4
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:	http://commits.kde.org/kile/a212fe70364e2e59af911a8619b7c272d1ce8876	Version Fixed In:
Sentry Crash Report:
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

http://commits.kde.org/kile/a212fe70364e2e59af911a8619b7c272d1ce8876

Comment 3 Michel Ludwig 2013-09-01 20:39:35 UTC

*** Bug 322527 has been marked as a duplicate of this bug. ***