WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
96750
[BlackBerry] Prevent scroll adjustment of input fields when region of interest mechanism active
https://bugs.webkit.org/show_bug.cgi?id=96750
Summary
[BlackBerry] Prevent scroll adjustment of input fields when region of interes...
Arvid Nilsson
Reported
2012-09-14 04:02:28 PDT
The region of interest mechanism replaces the scrolling/zooming functionality in BlackBerry::WebKit::InputHandler::ensureFocusTextElementVisible(CaretScrollType scrollType). We accomplish this by introducing a new fine-grained setting for the various adjustment modes. Whoever enables ROI needs to disable all scroll types using the new setting. PR 208387
Attachments
Patch
(4.45 KB, patch)
2012-09-14 17:11 PDT
,
Arvid Nilsson
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Arvid Nilsson
Comment 1
2012-09-14 17:11:22 PDT
Created
attachment 164252
[details]
Patch
Antonio Gomes
Comment 2
2012-09-17 07:04:55 PDT
Comment on
attachment 164252
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=164252&action=review
> Source/WebKit/blackberry/WebKitSupport/InputHandler.cpp:1055 > - if (!Platform::Settings::instance()->allowCenterScrollAdjustmentForInputFields() && scrollType != EdgeIfNeeded) > + if (!(Platform::Settings::instance()->allowedScrollAdjustmentForInputFields() & scrollType))
I would name it scrollAdjustmentForInputFieldsPolicy, but this this also ok.
> Source/WebKit/blackberry/WebKitSupport/InputHandler.h:71 > + enum CaretScrollType { CenterAlways = BlackBerry::Platform::Settings::ScrollAdjustmentCenterAlways, > + CenterIfNeeded = BlackBerry::Platform::Settings::ScrollAdjustmentCenterIfNeeded, > + EdgeIfNeeded = BlackBerry::Platform::Settings::ScrollAdjustmentEdgeIfNeeded };
there are compile asserts we can add to ensure these values match.
WebKit Review Bot
Comment 3
2012-09-18 01:16:58 PDT
Comment on
attachment 164252
[details]
Patch Clearing flags on attachment: 164252 Committed
r128858
: <
http://trac.webkit.org/changeset/128858
>
WebKit Review Bot
Comment 4
2012-09-18 01:17:01 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug