Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(18)

Issue 774053003: Remove TouchAdjustment. (Closed)

Created:
6 years ago by esprehn
Modified:
6 years ago
Reviewers:
rafaelw, ojan
CC:
abarth-chromium, mojo-reviews_chromium.org, jdduke (slow)
Base URL:
git@github.com:domokit/mojo.git@master
Project:
mojo
Visibility:
Public.

Description

Remove TouchAdjustment. TouchAdjustment adjusts the hit test rects to nearby elements so that when touch targets are too small they can still be hit by users. Instead applications should just make their touch targets big enough, and if they need adjustment it should be implemented at the framework level. R=ojan@chromium.org Committed: https://chromium.googlesource.com/external/mojo/+/21ce97a2c082187f587e1ca8c0d24c4472b81637

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -486 lines) Patch
M sky/engine/core/core.gni View 1 chunk +0 lines, -2 lines 0 comments Download
M sky/engine/core/frame/Settings.in View 1 chunk +0 lines, -6 lines 0 comments Download
M sky/engine/core/page/EventHandler.h View 3 chunks +1 line, -5 lines 0 comments Download
M sky/engine/core/page/EventHandler.cpp View 5 chunks +1 line, -80 lines 0 comments Download
D sky/engine/core/page/TouchAdjustment.h View 1 chunk +0 lines, -37 lines 0 comments Download
D sky/engine/core/page/TouchAdjustment.cpp View 1 chunk +0 lines, -339 lines 0 comments Download
M sky/engine/platform/PlatformGestureEvent.h View 1 chunk +0 lines, -10 lines 0 comments Download
M sky/engine/public/web/WebSettings.h View 1 chunk +0 lines, -1 line 0 comments Download
M sky/engine/web/WebSettingsImpl.h View 1 chunk +1 line, -1 line 0 comments Download
M sky/engine/web/WebSettingsImpl.cpp View 1 chunk +0 lines, -5 lines 0 comments Download

Messages

Total messages: 7 (1 generated)
esprehn
6 years ago (2014-12-03 02:57:23 UTC) #2
ojan
+rbyers to let us know if removing this is crazy or a good idea. otherwise, ...
6 years ago (2014-12-03 03:04:23 UTC) #3
Rick Byers
On 2014/12/03 03:04:23, ojan wrote: > +rbyers to let us know if removing this is ...
6 years ago (2014-12-03 14:36:57 UTC) #4
esprehn
Committed patchset #1 (id:1) manually as 21ce97a2c082187f587e1ca8c0d24c4472b81637 (presubmit successful).
6 years ago (2014-12-03 18:51:31 UTC) #5
ojan
On 2014/12/03 at 14:36:57, rbyers wrote: > On 2014/12/03 03:04:23, ojan wrote: > > +rbyers ...
6 years ago (2014-12-03 21:20:45 UTC) #6
esprehn
6 years ago (2014-12-04 07:26:38 UTC) #7
Message was sent while issue was closed.
On 2014/12/03 at 21:20:45, ojan wrote:
> On 2014/12/03 at 14:36:57, rbyers wrote:
> > ...
> 
> > However, I might be wrong about this.  Does iOS or Android have any built-in
notion (outside of webview of course) of fuzzy hit-testing?  Apple has
traditionally been unwilling to discuss this publicly due to IP issues.
> 
> Not sure.

Android lets you implement this yourself:
https://github.com/android/platform_frameworks_base/blob/master/core/java/and...

The framework also supports some slop features for gestures like dragging from
the edge. See all the slop properties in ViewConfiguration:
http://developer.android.com/reference/android/view/ViewConfiguration.html#ge...

iOS also lets you do this yourself:
http://stackoverflow.com/questions/808503/uibutton-making-the-hit-area-larger...

and some of the built in widgets do it too:
http://stackoverflow.com/questions/9079907/why-does-uinavigationbar-steal-tou...

but neither platform has a general concept of touch adjustment.

Powered by Google App Engine
This is Rietveld 408576698