Related: tdf#155266 Eliminate delayed scrollbar redrawing when swiping
By default, the layout idle timer in the InterimWindowItem
class is set to TaskPriority::RESIZE. That is too high of
a priority as it appears that other timers are drawing
after the scrollbar has been redrawn.
As a result, when swiping, the content moves fluidly but
the scrollbar thumb does not move until after swiping
stops or pauses. Then, after a short lag, the scrollbar
thumb finally "jumps" to the expected position.
So, to fix this scrollbar "stickiness" when swiping,
setting the priority to TaskPriority::POST_PAINT causes
the scrollbar to be redrawn after any competing timers.
Change-Id: I8c0772fc40ddc690ee59c6267c1c50971f4ff238
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161184
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Reviewed-by: Patrick Luby <plubius@libreoffice.org>
(cherry picked from commit 9a685238cfcc17bb1586b07047bb077a5ce3821b)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161216
diff --git a/include/vcl/InterimItemWindow.hxx b/include/vcl/InterimItemWindow.hxx
index 8f7efc6..ba4aa4c 100644
--- a/include/vcl/InterimItemWindow.hxx
+++ b/include/vcl/InterimItemWindow.hxx
@@ -34,6 +34,8 @@ public:
virtual void Draw(OutputDevice* pDevice, const Point& rPos,
SystemTextColorFlags nFlags) override;
void SetPriority(TaskPriority nPriority);
protected:
// bAllowCycleFocusOut of true allows focus to be moved out of the Control
// via tab key into a parent window or sibling window, false means focus
diff --git a/sc/source/ui/view/tabview.cxx b/sc/source/ui/view/tabview.cxx
index b6c7a2b..aa5bd47 100644
--- a/sc/source/ui/view/tabview.cxx
+++ b/sc/source/ui/view/tabview.cxx
@@ -231,6 +231,20 @@ void ScTabView::InitScrollBar(ScrollAdaptor& rScrollBar, tools::Long nMaxVal, co
rScrollBar.SetMouseReleaseHdl(LINK(this, ScTabView, EndScrollHdl));
rScrollBar.EnableRTL( aViewData.GetDocument().IsLayoutRTL( aViewData.GetTabNo() ) );
// Related: tdf#155266 Eliminate delayed scrollbar redrawing when swiping
// By default, the layout idle timer in the InterimWindowItem class
// is set to TaskPriority::RESIZE. That is too high of a priority as
// it appears that other timers are drawing after the scrollbar has been
// redrawn.
// As a result, when swiping, the content moves fluidly but the scrollbar
// thumb does not move until after swiping stops or pauses. Then, after a
// short lag, the scrollbar thumb finally "jumps" to the expected
// position.
// So, to fix this scrollbar "stickiness" when swiping, setting the
// priority to TaskPriority::POST_PAINT causes the scrollbar to be
// redrawn after any competing timers.
rScrollBar.SetPriority(TaskPriority::POST_PAINT);
}
// Scroll-Timer
diff --git a/vcl/source/control/InterimItemWindow.cxx b/vcl/source/control/InterimItemWindow.cxx
index 41d1466..697f787 100644
--- a/vcl/source/control/InterimItemWindow.cxx
+++ b/vcl/source/control/InterimItemWindow.cxx
@@ -187,6 +187,11 @@ void InterimItemWindow::Draw(OutputDevice* pDevice, const Point& rPos,
m_xContainer->draw(*pDevice, rPos, GetSizePixel());
}
void InterimItemWindow::SetPriority(TaskPriority nPriority)
{
m_aLayoutIdle.SetPriority(nPriority);
}
void InterimItemWindow::ImplPaintToDevice(::OutputDevice* pTargetOutDev, const Point& rPos)
{
Draw(pTargetOutDev, rPos, SystemTextColorFlags::NONE);
