Merge lp://qastaging/~zsombi/ubuntu-ui-toolkit/55-snap-options into lp://qastaging/~zsombi/ubuntu-ui-toolkit/listitem-master

8 + // track drag dirrection, so we know in which direction we should snap

diRRection

As a developer, I like to configure the overshoot when I use a custom delegate. Is that possible now? I have this delegate:

            delegate: Rectangle {
                color: delegateArea.pressed ? "orange" : "yellow"
                width: units.gu(10)
                ListItemActions.overshoot: 0
                Label {
                    text: action.text
                    anchors.centerIn: parent
                }
                MouseArea {
                    anchors.fill: parent
                    id: delegateArea
                    onPressed: print("press")
                }
            }

which is yellow, but when dragging I can pull a bit further and show the default white background of the panel. Can I disable the overshoot completely to avoid this?

why is overshoot an attached property and not a property of ListItemActions directly?

> As a developer, I like to configure the overshoot when I use a custom
> delegate. Is that possible now? I have this delegate:
>
> delegate: Rectangle {
> color: delegateArea.pressed ? "orange" : "yellow"
> width: units.gu(10)
> ListItemActions.overshoot: 0
> Label {
> text: action.text
> anchors.centerIn: parent
> }
> MouseArea {
> anchors.fill: parent
> id: delegateArea
> onPressed: print("press")
> }
> }
>
> which is yellow, but when dragging I can pull a bit further and show the
> default white background of the panel. Can I disable the overshoot completely
> to avoid this?

If you want to see the white background, you have to have a different actionsDelegate style component (perhaps we should name that differently?). Changing the overshoot to 0 would also mean that the panel cannot be dragged further than the panel size, so you won't see any background.

> why is overshoot an attached property and not a property of ListItemActions
> directly?

that property is needed for the panel for sure. I thought we should expose that in the style, but we can expose it as property to ListItem rather to ListItemActions. As it is not actions specific, it is more like ListItem specific.

> 8 + // track drag dirrection, so we know in which direction we should
> snap
>
> diRRection

I was highlighting the mistake, direction is with 1 r.

add ListItem.swipeOvershoot, and the style can have a default value for that.

> add ListItem.swipeOvershoot, and the style can have a default value for that.

Now, we have a slight problem with getting the value from the style... as the style component is loaded only when someone starts swiping the content... so I can only get the value from the style when the style is loaded. Of course this doesn't make any harm when a user value is set on ListItem, but may cause property changed signalled when the style gets loaded.

286 + * \qmlproperty real ListItem::swipeOvershoot
287 + * The property configures the overshoot value on swiping. Its default value is configured by the style.
288 + */

You can link "style" to ListItemStyle

> 286 + * \qmlproperty real ListItem::swipeOvershoot
> 287 + * The property configures the overshoot value on swiping. Its
> default value is configured by the style.
> 288 + */
>
> You can link "style" to ListItemStyle

Done.

Perfect. >=0 overshoot works as expected now, and <0 overshoot sets the default value.