From 2b5b069863187b215a33bf8267c37af3aa015acf Mon Sep 17 00:00:00 2001 From: NiLuJe Date: Mon, 8 Jun 2020 16:36:26 +0200 Subject: [PATCH] Remove a dodgy scheduleIn (#6241) It was causing a double-refresh on orientation change via ConfigDialog I could only reproduce it on Kindle, for some strange reason (slower?). --- frontend/apps/reader/modules/readerrolling.lua | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/frontend/apps/reader/modules/readerrolling.lua b/frontend/apps/reader/modules/readerrolling.lua index 72be6f416..5c8b8db6e 100644 --- a/frontend/apps/reader/modules/readerrolling.lua +++ b/frontend/apps/reader/modules/readerrolling.lua @@ -786,7 +786,7 @@ function ReaderRolling:onUpdatePos() return true end -- Calling this now ensures the re-rendering is done by crengine - -- so the delayed updatePos() has good info and can reposition + -- so updatePos() has good info and can reposition -- the previous xpointer accurately: self.ui.document:getCurrentPos() -- Otherwise, _readMetadata() would do that, but the positionning @@ -794,7 +794,7 @@ function ReaderRolling:onUpdatePos() -- previously because of some bad setDirty() in ConfigDialog widgets -- that were triggering a full repaint of crengine (so, the needed -- rerendering) before updatePos() is called. - UIManager:scheduleIn(0.1, function () self:updatePos() end) + self:updatePos() end function ReaderRolling:updatePos()