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

Issue 768492 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Dec 11
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: ----

Blocked on:
issue 764509


Participants' hotlists:
Launcher-Tech-Debt


Sign in to add a comment

refactor scroll handling to use events instead of offsets and event types

Project Member Reported by newcomer@chromium.org, Sep 25 2017

Issue description

Currently, in order to enable scrolling from the shelf to change AppList states, we resolve scroll events into offsets and types, then send over mojo IPC to the AppListView.

This is propogating to other parts of AppListView (AppsGridView::OnScrollEvent) and we should remember to go back to processing events once we can do it from all sources.


When we merge AppList into the ash process, we will no longer need to resolve scroll events, and we can get rid of AppListView::HandleScroll, and AppsGridView::HandleScroll.
 
Blocking: 764509
Labels: M-65 Pri-2
Cc: newcomer@chromium.org
Owner: ----
Labels: Touch-Friendly-Launcher-Triaged

Comment 5 by vadimt@chromium.org, Oct 10 2017

Owner: newcomer@chromium.org

Comment 6 by vadimt@chromium.org, Oct 10 2017

Blockedon: 764509
Blocking: -764509

Comment 7 by vadimt@chromium.org, Oct 10 2017

Owner: ----
Labels: -Restrict-View-Google -Pri-2 -M-65 M-72 Hotlist-GoodFirstBug Pri-3
Labels: -M-72 M-73
Status: Fixed (was: Untriaged)
Fixed by mmourgous@

Sign in to add a comment