https://bugs.kde.org/show_bug.cgi?id=390929

Peter Mühlenpfordt <de...@ukn8.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
         Resolution|---                         |FIXED
   Version Fixed In|                            |18.08.0
      Latest Commit|                            |https://commits.kde.org/gwe
                   |                            |nview/fec2cd8b655e06c6cbb62
                   |                            |4b32737651b5a45bc8c

--- Comment #7 from Peter Mühlenpfordt <de...@ukn8.de> ---
Git commit fec2cd8b655e06c6cbb624b32737651b5a45bc8c by Peter Mühlenpfordt.
Committed on 05/08/2018 at 07:14.
Pushed by muhlenpfordt into branch 'Applications/18.08'.

Fix video controls volume slider not vertically centered sometimes

Summary:
The volume slider of the video controls overlay is sometimes not
centered vertically. This happens mostly on KDE Neon or if Gwenview
is started to show a video by commandline argument.
This is caused by calculating the slider and handle positions before
initializing the widget and therefore using wrong values. After the
first slider change this is correctly updated.
This patch delays the initialization of the slider to give the
overlay widget time to setup.
FIXED-IN: 18.08.0

Before:
{F6173671, size=full}

After:
{F6173673, size=full}

Test Plan:
* Start Gwenview with video file as commandline argument
* Check that all sliders are vertically centered in controls overlay
* Check in current KDE Neon system

Reviewers: #gwenview, rkflx

Reviewed By: #gwenview, rkflx

Subscribers: rkflx

Differential Revision: https://phabricator.kde.org/D14578

M  +4    -1    lib/hud/hudslider.cpp

https://commits.kde.org/gwenview/fec2cd8b655e06c6cbb624b32737651b5a45bc8c

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to