----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/124782/ -----------------------------------------------------------
(Updated Aug. 26, 2015, 5:52 p.m.) Review request for Plasma. Changes ------- Improved Comment Repository: plasma-desktop Description ------- Add some comments to this IHMO hard to understand code. I hope I understood the code correctly... The maximum and minimum position the slider can be moved to is not relative to offset, so min/maxValue is a wrong term. Instead, use min/maxPosition as the borders are absolute. Diffs (updated) ----- desktoppackage/contents/configuration/panelconfiguration/Ruler.qml a31feb40598ba24a107f41ff3b3f823afaa89da6 desktoppackage/contents/configuration/panelconfiguration/SliderHandle.qml 3480ce65a961d498055b3952d2d57b4f5d881a9a Diff: https://git.reviewboard.kde.org/r/124782/diff/ Testing ------- Changing the size and position works as before. Thanks, David Kahles
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
