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

Issue 749807 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: ----
Type: ----


Previous locations:
gerrit:6821


Sign in to add a comment

'Filter' text box should have focus after clicking 'Choose try jobs'

Project Member Reported by thakis@chromium.org, Jul 22 2017

Issue description

Affected Version:

What steps will reproduce the problem?
1. Click 'Choose try jobs'
2. Try to start typing to filter list of bots

What is the expected output?

Should work.


What do you see instead?

I have to manually click the 'Filter' text box first, even though nothing else has keyboard focus while the try dialog is up.


Please provide any additional information below.

 

Comment 1 by wyatta@google.com, Jul 24 2017

Cc: aga...@chromium.org

Comment 2 by aga...@chromium.org, Jul 25 2017

Cc: kaspern@google.com
Kasper: the "choose try jobs" box is a gr-overlay. Can you comment on what the best way would be to have the new gr-overlay focus a specific sub-element?

And relatedly, for accessibility, when it closes, gr-overlay needs to restore focus to wherever it was prior to the overlay being opened.

Comment 3 by kaspern@google.com, Jul 25 2017

I'd recommend following the approach done in gr-change-view here:

https://gerrit.googlesource.com/gerrit/+/master/polygerrit-ui/app/elements/change/gr-change-view/gr-change-view.js#371

Comment 4 by logan@google.com, Jul 27 2017

Components: -PolyGerrit
Labels: Proj-Gerrit-Migration
Aaron, feel free to send this back our way if you need help implementing this.

Comment 5 by logan@google.com, Jul 27 2017

Project: chromium
Moved issue gerrit:6821 to now be  issue chromium:749807 .

Comment 6 by aga...@chromium.org, Jul 27 2017

Components: Infra>Codereview>Gerrit
Labels: Milestone-Afterglow
Status: Available (was: New)
Labels: -Milestone-Afterglow
Removing Milestone-Afterglow, as it has ceased to have meaning. More refined milestones may be added back in the near future.

Comment 8 by aga...@chromium.org, Mar 27 2018

Owner: aga...@chromium.org
Status: Fixed (was: Available)
I fixed this!

Sign in to add a comment