diff options
| author | Caolán McNamara <caolanm@redhat.com> | 2020-03-11 14:16:20 +0000 |
|---|---|---|
| committer | Caolán McNamara <caolanm@redhat.com> | 2020-03-11 16:48:58 +0100 |
| commit | d6717f243fe4a4935c96da791799d9c62fc8f84f (patch) | |
| tree | 1df11d98f4190a45fb52b7a02b920a23f123e87e /vcl | |
| parent | cdc16e5d67ad4668cfa457fc277a796b413228a9 (diff) | |
Window::Update means to call Paint immediately
rather than wait until the next event loop. This is causing the scrollbar to be
painted at pos 0 and then appears to flicker when it is repainted at its final
position when the gen navigator treeview is cleared and refilled.
call Invalidate instead to schedule in the painting of the scrollbars at the
usual time.
Update is used since initial commit so its unknown as to the reason why.
If SetUpdateMode(true) is in use it won't happen so the flicker normally
doesn't happen, but the equivalent we're using for the native gtk widgerty
disconnects the model when frozen for performance, so setting cursor positions
and other similar operations have to happen unfrozen. Not calling Update here
would make things much easier.
Change-Id: I4575e55802c54e5b849f076f2da79dd010fb27e8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90331
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Diffstat (limited to 'vcl')
| -rw-r--r-- | vcl/source/treelist/svimpbox.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/vcl/source/treelist/svimpbox.cxx b/vcl/source/treelist/svimpbox.cxx index e1561ad12607..bc407216a696 100644 --- a/vcl/source/treelist/svimpbox.cxx +++ b/vcl/source/treelist/svimpbox.cxx @@ -1362,7 +1362,7 @@ void SvImpLBox::ShowVerSBar() m_pView->nFocusWidth = -1; AdjustScrollBars( m_aOutputSize ); if( GetUpdateMode() ) - m_aVerSBar->Update(); + m_aVerSBar->Invalidate(); } } else @@ -1385,7 +1385,7 @@ void SvImpLBox::ShowVerSBar() m_pView->nFocusWidth = -1; AdjustScrollBars( m_aOutputSize ); if( GetUpdateMode() ) - m_aHorSBar->Update(); + m_aHorSBar->Invalidate(); } else { |