Commit e5ab1b23 authored by Xiaoqian Dai's avatar Xiaoqian Dai Committed by Commit Bot

Don't show the selection widget for the first overview item if...

... the selection widget is already active or the filter string is empty.

Bug: 893384
Change-Id: I793e083991ef60361a4ca3056347a4023919b240
Reviewed-on: https://chromium-review.googlesource.com/c/1292713Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Xiaoqian Dai <xdai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#601346}
parent ce7fd8ee
......@@ -879,9 +879,9 @@ void WindowSelector::ContentsChanged(views::Textfield* sender,
for (std::unique_ptr<WindowGrid>& grid : grid_list_)
grid->FilterItems(new_contents);
// If the selection widget is not active, execute a Move() command so that it
// shows up on the first undimmed item.
if (grid_list_[selected_grid_index_]->is_selecting())
// If the selection widget is not active and the filter string is not empty,
// execute a Move() command so that it shows up on the first undimmed item.
if (grid_list_[selected_grid_index_]->is_selecting() || new_contents.empty())
return;
Move(WindowSelector::RIGHT, false);
}
......
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