New issue
Advanced search Search tips

Issue 893093 link

Starred by 1 user
Status: Fixed
Owner:
joelhockey@chromium.org
Closed: Nov 22
Cc:
ajha@chromium.org
lucmult@chromium.org
weifangsun@chromium.org
fukino@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug-Regression



Sign in to add a comment

Regression: 'File info' tooltip stays after selecting it in Files app

Project Member Reported by rkalavakuntla@chromium.org, Oct 8 
Chrome Version: 71.0.3567.0/11137.0.0 dev channel  Gnawty,Celes,Peppy
OS:Chrome OS

What steps will reproduce the problem?
(1)Sign into user >> Take a Screenshot >> Get info of it in files app
(2)By tabbing,bring the focus on 'File info' >> hit enter and observe 

Actual: 'File info' tooltip stays even after hitting enter on it 
Expected: 'File info' tooltip should go off after sometime

This is a Regression issue as same is working fine in M-68

Note: Issue is seen in M-69 Stable, M-70 beta

Attached the screencast for reference..
Actual.mp4
9.3 MB View Download
Expected.mp4
9.2 MB View Download

Comment 1 by weifangsun@chromium.org, Oct 8

Labels: -Pri-1 -M-71 CrOSFilesCategory-UI CrOSFilesFeature-QuickView Pri-3
Status: Available (was: Untriaged)

Comment 2 by slangley@chromium.org, Oct 31

Cc: lucmult@chromium.org

Comment 3 by weifangsun@chromium.org, Nov 7

Labels: Files-Fixit-2018

Comment 4 by joelhockey@chromium.org, Nov 20

Owner: joelhockey@chromium.org
Status: Started (was: Available)
Project Member

Comment 5 by bugdroid1@chromium.org, Nov 22 

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

commit fce468e7e8fb0511a7a7a503d036dec79e86aa25
Author: Joel Hockey <joelhockey@chromium.org>
Date: Thu Nov 22 07:09:45 2018

FilesApp quickview: test to ensure that info box opens/closes on enter key

* Added quickview integration test pressEnterOnInfoBoxToOpenClose.
* Updated test.util.sync.deepQueryAllElements to accept targetQuery
  as either string or !Array<string>.
* Changed RemoteCall waitForElement* functions to call
  deepQueryAllElements to allow for usage with nested elements.
* Updated javadoc on RemoteCall functions to show which already
  support query as string|!Array<string>.


Bug:  893093 
Change-Id: I7c542e7a2bc914e16fa8142fb696f5b3fe370752
Reviewed-on: https://chromium-review.googlesource.com/c/1345693
Reviewed-by: Luciano Pacheco <lucmult@chromium.org>
Commit-Queue: Joel Hockey <joelhockey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#610326}
[modify] https://crrev.com/fce468e7e8fb0511a7a7a503d036dec79e86aa25/chrome/browser/chromeos/file_manager/file_manager_browsertest.cc
[modify] https://crrev.com/fce468e7e8fb0511a7a7a503d036dec79e86aa25/ui/file_manager/file_manager/background/js/runtime_loaded_test_util.js
[modify] https://crrev.com/fce468e7e8fb0511a7a7a503d036dec79e86aa25/ui/file_manager/integration_tests/file_manager/quick_view.js
[modify] https://crrev.com/fce468e7e8fb0511a7a7a503d036dec79e86aa25/ui/file_manager/integration_tests/remote_call.js

Comment 6 by joelhockey@chromium.org, Nov 22

Status: Fixed (was: Started)
►

Sign in to add a comment