New issue
Advanced search Search tips

Issue 683140 link

Starred by 6 users

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

shortcuts to switch devtool panel left/right do not work any more

Reported by liuway...@gmail.com, Jan 20 2017

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2986.0 Safari/537.36

Steps to reproduce the problem:
1.  option+command+j to open devtool
2. command+[/ command+] to go to panel to the left/right

What is the expected behavior?
command+[/command+] supposed to go to the panel left/right as demonstrated in the devtool setting shortcuts tab.

What went wrong?
instead of switch devtool panel, it goes forward / backforward of browser history.

Did this work before? Yes I tried chromium 51 and chrome 55.0.2883.95 and both work

Chrome version: 57.0.2986.0  Channel: canary
OS Version: OS X 10.12.2
Flash Version: Shockwave Flash 24.0 r0
 
chromium-bug-panel-switch-shortcuts.jpg
88.3 KB View Download

Comment 1 by caseq@chromium.org, Jan 20 2017

Labels: -Pri-2 Pri-1
Owner: pfeldman@chromium.org
Status: Assigned (was: Unconfirmed)
Bisected down to https://chromium.googlesource.com/chromium/src/+/2cbfc042012e00bc5863caa8c55de9d25f71d5e3

This is a regression from stable, so consider merging.

Comment 3 by woxxom@gmail.com, Jan 26 2017

See   issue 685674  - typing [] in an Italian keyboard layout requires pressing AltGr (right-side Alt), which gets misinterpreted as a tab-switch modifier after r445638.
Labels: Merge-Request-57
Status: Started (was: Assigned)
Project Member

Comment 5 by sheriffbot@chromium.org, Jan 31 2017

Labels: -Merge-Request-57 Hotlist-Merge-Approved Merge-Approved-57
Your change meets the bar and is auto-approved for M57. Please go ahead and merge the CL to branch 2987 manually. Please contact milestone owner if you have questions.
Owners: amineer@(clank), cmasso@(bling), ketakid@(cros), govind@(desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 6 by gov...@chromium.org, Jan 31 2017

Please merge your change to M57 branch 2987 ASAP (latest before 5:00 PM PT on Wednesday, 02/01/17) so we can pick it up for M57 Beta promotion release this week. Thank you.
Labels: TE-Verified-M57 TE-Verified-57.0.2987.21
Verified this issue on Mac OS 10.12 using chrome latest Dev M57-57.0.2987.21 by following steps mentioned in the original comment. By pressing shortcut key
(command+[) and (command+]) it switches the panel from left/right as expected in dev-tools. Hence adding TE-Verified label.
683140.mp4
1.1 MB View Download
Labels: -Merge-Approved-57 merge-merged-2987
Seems like this is already merged to M57 - https://chromium.googlesource.com/chromium/src.git/+/637fede12821cf7a4770fe33e937ad9e90a6ea78
Status: Fixed (was: Started)
Cc: nyerramilli@chromium.org
 Issue 702080  has been merged into this issue.

Sign in to add a comment