[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-frameworks-devel
Subject:    D4972: Start drag with press and hold on touch events
From:       David Edmundson <noreply () phabricator ! kde ! org>
Date:       2017-03-08 1:59:02
Message-ID: differential-rev-PHID-DREV-igtpaznighsgwq2gug6y-req () phabricator ! kde ! org
[Download RAW message or body]

davidedmundson created this revision.
Restricted Application added projects: Plasma, Frameworks.
Restricted Application added subscribers: Frameworks, plasma-devel.

REVISION SUMMARY
  This patch consists of two changes:
  
  - starting drags on press and hold
  - not starting drags when the event is from a touch event
  
  This means a user can scroll on a widget explorer with a touch screen
  correctly without it immediately starting a drag, but can still drag an
  item using a press and hold pattern.
  
  BUG: 368698

TEST PLAN
  Scrolled in Widget Explorer
  Added widget with press and hold
  
  Surprisingly that's the only Plasma usage of DragArea.

REPOSITORY
  R296 KDeclarative

BRANCH
  master

REVISION DETAIL
  https://phabricator.kde.org/D4972

AFFECTED FILES
  src/qmlcontrols/draganddrop/DeclarativeDragArea.cpp
  src/qmlcontrols/draganddrop/DeclarativeDragArea.h

To: davidedmundson, #plasma
Cc: plasma-devel, #frameworks, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol

[Attachment #3 (text/html)]

<table><tr><td style="">davidedmundson created this revision.<br />Restricted \
Application added projects: Plasma, Frameworks.<br />Restricted Application added \
subscribers: Frameworks, plasma-devel. </td><a style="text-decoration: none; padding: \
4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; \
border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to \
bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" \
href="https://phabricator.kde.org/D4972" rel="noreferrer">View \
Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This \
patch consists of two changes:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">starting drags on press and hold</li>
<li class="remarkup-list-item">not starting drags when the event is from a touch \
event</li> </ul>

<p>This means a user can scroll on a widget explorer with a touch screen<br />
correctly without it immediately starting a drag, but can still drag an<br />
item using a press and hold pattern.</p>

<p>BUG: 368698</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Scrolled \
in Widget Explorer<br /> Added widget with press and hold</p>

<p>Surprisingly that&#039;s the only Plasma usage of DragArea.</p></div></div><br \
/><div><strong>REPOSITORY</strong><div><div>R296 KDeclarative</div></div></div><br \
/><div><strong>BRANCH</strong><div><div>master</div></div></div><br \
/><div><strong>REVISION DETAIL</strong><div><a \
href="https://phabricator.kde.org/D4972" \
rel="noreferrer">https://phabricator.kde.org/D4972</a></div></div><br \
/><div><strong>AFFECTED \
FILES</strong><div><div>src/qmlcontrols/draganddrop/DeclarativeDragArea.cpp<br /> \
src/qmlcontrols/draganddrop/DeclarativeDragArea.h</div></div></div><br \
/><div><strong>To: </strong>davidedmundson, Plasma<br /><strong>Cc: \
</strong>plasma-devel, Frameworks, progwolff, lesliezhai, ali-mohamed, \
jensreuterberg, abetts, sebas, apol<br /></div>



[prev in list] [next in list] [prev in thread] [next in thread]