Review Request 124786: [Panelconfig] Add more / fix constraints in the position/size handles

David Kahles david.kahles96 at gmail.com
Wed Aug 26 15:56:25 UTC 2015



> On Aug. 24, 2015, 2:06 a.m., Marco Martin wrote:
> > the middle thing was done in order to not cross the handles, but the other check you afded should prevent this anyways

That was done by `offsetHandle.position + units.gridUnit * 3` in
    minimumPosition: Math.max(offsetHandle.position + units.gridUnit * 3, [middle thing])

But I think it's really not needed.


- David


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124786/#review84238
-----------------------------------------------------------


On Aug. 17, 2015, 12:50 p.m., David Kahles wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/124786/
> -----------------------------------------------------------
> 
> (Updated Aug. 17, 2015, 12:50 p.m.)
> 
> 
> Review request for Plasma.
> 
> 
> Repository: plasma-desktop
> 
> 
> Description
> -------
> 
> Prevent users from moving the panel off the monitor, and prevent users
> from moving a panel with center alignment to the left (right) and then
> pull one of the right (left) sliders to the right (left). Because the
> left and right sliders are coupled, this would make the left (right)
> sliders to go off the monitor.
> 
> Also fix that the left (right) sliders cannot be moved over the middle.
> (In fact they can by using the offset slider, but as soon as on of the
> left (right) handles are dragged, it sets the position back to the
> middle).
> I really don't know the reason for this behavior, and everything
> works fine (or even better) without it. Does someone know whether this is needed and why?
> 
> Also delete the default values for min/maxPosition in the SliderHandle,
> as every handle sets its own min/maxPositions now.
> 
> And should I add such checks also to panelview.cpp to prevent loading an invalid panel position/size from the config?
> (Though this should only happen when a user changes the config manually) There are some checks, but not everything is checked..
> 
> 
> Diffs
> -----
> 
>   desktoppackage/contents/configuration/panelconfiguration/Ruler.qml a31feb40598ba24a107f41ff3b3f823afaa89da6 
>   desktoppackage/contents/configuration/panelconfiguration/SliderHandle.qml 3480ce65a961d498055b3952d2d57b4f5d881a9a 
> 
> Diff: https://git.reviewboard.kde.org/r/124786/diff/
> 
> 
> Testing
> -------
> 
> I know no way to set an invalid position/size as I could before. It works for horizontal and vertical panels.
> 
> 
> Thanks,
> 
> David Kahles
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150826/40e4f108/attachment.html>


More information about the Plasma-devel mailing list