New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 755615 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: 1
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Add UMA for new launcher app grid pagination

Project Member Reported by newcomer@chromium.org, Aug 15 2017

Issue description

Record all methods of changing pages in the app grid in order to inform future UX decisions.

 
Project Member

Comment 1 by sheriffbot@chromium.org, Aug 15 2017

Labels: -Merge-Request-61 Merge-Review-61 Hotlist-Merge-Review
This bug requires manual review: M61 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), ketakid@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Review-61 Merge-Approved-61
Approving merge to M61 Chrome OS.
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 17 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/366fd939c002486a16da9f5953e60619c1a84f8a

commit 366fd939c002486a16da9f5953e60619c1a84f8a
Author: Alex Newcomer <newcomer@chromium.org>
Date: Thu Aug 17 18:47:57 2017

cros: Added UMA for pagination in the app list

Added UMA for pagination in the app list. It covers the
different ways to change pages:
1. Swiping
2. Flicking
3. Mousepad scroll
4. Mousewheel scroll
5. Clicking the pagination indicators
6. Tapping the pagination indicators.
7. Dragging an app to the border and wating

Bug:  755615 
Change-Id: I49da37ec2f0ed3fee114168b2a3a6e67f8969b5a
Reviewed-on: https://chromium-review.googlesource.com/616245
Commit-Queue: Alex Newcomer <newcomer@chromium.org>
Reviewed-by: Mark Pearson <mpearson@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#495255}
[modify] https://crrev.com/366fd939c002486a16da9f5953e60619c1a84f8a/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/366fd939c002486a16da9f5953e60619c1a84f8a/tools/metrics/histograms/histograms.xml
[modify] https://crrev.com/366fd939c002486a16da9f5953e60619c1a84f8a/ui/app_list/app_list_constants.cc
[modify] https://crrev.com/366fd939c002486a16da9f5953e60619c1a84f8a/ui/app_list/app_list_constants.h
[modify] https://crrev.com/366fd939c002486a16da9f5953e60619c1a84f8a/ui/app_list/pagination_controller.cc
[modify] https://crrev.com/366fd939c002486a16da9f5953e60619c1a84f8a/ui/app_list/pagination_model.cc
[modify] https://crrev.com/366fd939c002486a16da9f5953e60619c1a84f8a/ui/app_list/pagination_model.h
[modify] https://crrev.com/366fd939c002486a16da9f5953e60619c1a84f8a/ui/app_list/pagination_model_unittest.cc
[modify] https://crrev.com/366fd939c002486a16da9f5953e60619c1a84f8a/ui/app_list/views/apps_grid_view.cc
[modify] https://crrev.com/366fd939c002486a16da9f5953e60619c1a84f8a/ui/app_list/views/page_switcher_vertical.cc

Project Member

Comment 4 by sheriffbot@chromium.org, Aug 21 2017

Cc: keta...@chromium.org
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 5 by sheriffbot@chromium.org, Aug 24 2017

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Fixed (was: Started)

Comment 7 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)
Labels: -Merge-Approved-61

Sign in to add a comment