Commit 25c060af authored by Alex Danilo's avatar Alex Danilo Committed by Commit Bot

Disable scroll on focus

CL:1593885 changed scrolling on focus behavior for the directory
tree.

A bug triggered by this change scrolls the entire client area
of the FilesApp under some focus changes.

This CL disables the scroll on focus completely.

Bug: 955566
Change-Id: I3b5d8b67501308ab5b5f3a69d3d373ec1127719b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1605382Reviewed-by: default avatarLuciano Pacheco <lucmult@chromium.org>
Commit-Queue: Alex Danilo <adanilo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#658394}
parent 5f7eb14a
...@@ -1791,9 +1791,6 @@ class DirectoryTree extends cr.ui.Tree { ...@@ -1791,9 +1791,6 @@ class DirectoryTree extends cr.ui.Tree {
fileOperationManager, 'entries-changed', fileOperationManager, 'entries-changed',
this.onEntriesChanged_.bind(this)); this.onEntriesChanged_.bind(this));
// Add a listener so we can scroll selected item into view.
this.addEventListener('focus', this.onFocus_.bind(this));
this.addEventListener('click', (event) => { this.addEventListener('click', (event) => {
// Chromevox triggers |click| without switching focus, we force the focus // Chromevox triggers |click| without switching focus, we force the focus
// here so we can handle further keyboard/mouse events to expand/collapse // here so we can handle further keyboard/mouse events to expand/collapse
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment