commit | f30556be3738006a6cd285bf327f4db87c15d0f4 | [log] [tgz] |
---|---|---|
author | Fabio Tirelo <ftirelo@chromium.org> | Tue Dec 04 03:22:36 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Tue Dec 04 03:22:36 2018 |
tree | cf75e54268fea534d14a1f1d1ffb1adafce3fe3a | |
parent | e9d087a00c99e2eb1d2654133700dd6b12d15b7a [diff] |
[AF Android] Maybe omit new dropdown in landscape mode When the new dropdown is shown in landscape mode, most vertical space is occupied with the on-screen keyboard and the sticky footer options, leaving not enough space to actually show the suggestions. This CL adds a quick fix that prevents the most common cases by allowing the dropdown to only be shown in landscape mode when we are sure that we have enough vertical space (e.g. tablets). This will allow us to at least to experiment on the new dropdown in M72 and collect metrics to understand the size of the affected population and make an informed decision on how we should address this issue. Moving forward, the plan to properly fix this is: - [short term] Make the footer non-sticky; - [long term] Move away from the dropdown and show suggestions in the keyboard accessory. A follow-up CL will add a new bucket to Autofill.FormEvents to track how often that happens. Bug: 905081 Change-Id: Ia8f8b14f987e13e8d9f0afc021d156fbf4238af6 Reviewed-on: https://chromium-review.googlesource.com/c/1346734 Commit-Queue: Fabio Tirelo <ftirelo@chromium.org> Reviewed-by: Ted Choc <tedchoc@chromium.org> Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org> Cr-Commit-Position: refs/heads/master@{#613410}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .