New issue
Advanced search Search tips

Issue 615284 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

[Media Router WebUI] Can't select search input or button in filter view

Project Member Reported by btolsch@chromium.org, May 27 2016

Issue description

Version: 52.0.2731.0
OS: Linux, Windows (but should affect all)

What steps will reproduce the problem?
(1) Open MR dialog where search is enabled and there are sinks present.
(2) Click the search icon to enter filter view.
(3) Observe that you can't click the search icon or input anymore, though you can still tab to it.
 
Components: Blink>PresentationAPI
Project Member

Comment 2 by bugdroid1@chromium.org, May 27 2016

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

commit c93f3868acb563e2babd9a64e883ea9832fb471b
Author: btolsch <btolsch@chromium.org>
Date: Fri May 27 06:59:20 2016

[Media Router WebUI] Put z-index of search over search results

This change gives the search div tag a z-index so that the mouse can
interact with it when the search results are being shown, which
currently cover the search div with padding.

BUG= 615284 
R=apacible@chromium.org
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2019543002
Cr-Commit-Position: refs/heads/master@{#396414}

[modify] https://crrev.com/c93f3868acb563e2babd9a64e883ea9832fb471b/chrome/browser/resources/media_router/elements/media_router_container/media_router_container.css

Status: Fixed (was: Started)
Labels: Merge-Request-52

Comment 5 by tin...@google.com, Jun 1 2016

Labels: -Merge-Request-52 Merge-Approved-52 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M52 (branch: 2743)
Project Member

Comment 6 by bugdroid1@chromium.org, Jun 1 2016

Labels: -merge-approved-52 merge-merged-2743
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3c798c8bffceeadc10d969d485323e166925267e

commit 3c798c8bffceeadc10d969d485323e166925267e
Author: btolsch <btolsch@chromium.org>
Date: Wed Jun 01 22:39:45 2016

[Media Router WebUI] Put z-index of search over search results

This change gives the search div tag a z-index so that the mouse can
interact with it when the search results are being shown, which
currently cover the search div with padding.

BUG= 615284 
R=apacible@chromium.org
NOTRY=true
NOPRESUBMIT=true

Review-Url: https://codereview.chromium.org/2019543002
Cr-Commit-Position: refs/heads/master@{#396414}
(cherry picked from commit c93f3868acb563e2babd9a64e883ea9832fb471b)

Review-Url: https://codereview.chromium.org/2035473002
Cr-Commit-Position: refs/branch-heads/2743@{#177}
Cr-Branched-From: 2b3ae3b8090361f8af5a611712fc1a5ab2de53cb-refs/heads/master@{#394939}

[modify] https://crrev.com/3c798c8bffceeadc10d969d485323e166925267e/chrome/browser/resources/media_router/elements/media_router_container/media_router_container.css

Sign in to add a comment