My favorites | Sign in
Project Home Downloads Wiki Issues Code Search
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 175858: [Linux] The panel does not accept keyboard input when it is minimized and expanded
1 person starred this issue and may be notified of changes. Back to list
 
Project Member Reported by jia...@chromium.org, Feb 12, 2013
Version: 25.0
OS: Linux

What steps will reproduce the problem?
1. Launch a docked panel
2. Make sure the panel is focused and then click the minimize button to collapse the panel
3. Expand the panel by clicking on the titlebar or the restore button

What is the expected output? What do you see instead?
Expect that the panel should be painted as active and could receive keyboard input. Instead, the panel is inactive. No matter how you click on it again, it does not become active and does not accept the keyboard input.


Feb 12, 2013
#1 jia...@chromium.org
(No comment was entered for this change.)
Status: Started
Feb 12, 2013
#2 dim...@chromium.org
Since the fix is simple and very isolated, we should ask for M26 port as well, to reduce frustration of Linux users.
Labels: Mstone-26
Feb 12, 2013
#3 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=182121

------------------------------------------------------------------------
r182121 | jianli@chromium.org | 2013-02-13T04:44:02.621790Z

Changed paths:
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/ui/gtk/panels/panel_gtk.cc?r1=182121&r2=182120&pathrev=182121
   M http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/ui/gtk/panels/panel_titlebar_gtk.cc?r1=182121&r2=182120&pathrev=182121

Fix  bug 175858 : [Linux] The panel does not accept keyboard input when it is minimized and expanded

BUG=175858
TEST=Manual test by following the repro steps in the bug


Review URL: https://chromiumcodereview.appspot.com/12223114
------------------------------------------------------------------------
Feb 14, 2013
#4 jia...@chromium.org
(No comment was entered for this change.)
Status: Fixed
Feb 14, 2013
#5 jia...@chromium.org
The fix is simple and not risky. Could we please merge it into M26?
Labels: Merge-Requested
Feb 15, 2013
#6 dhar...@chromium.org
(No comment was entered for this change.)
Labels: -Merge-Requested Merge-Approved
Feb 15, 2013
#7 mbo...@chromium.org
Is this the dock panel we talking about. Please confirm.
Screenshot from 2013-02-15 13:59:12.png
55.7 KB   View   Download
Cc: mbo...@chromium.org
Labels: Action-FeedbackNeeded
Feb 15, 2013
#8 jia...@chromium.org
No.

Please use the attached test extension to launch the docked panel.
simple_panel.zip
25.4 KB   Download
Feb 15, 2013
#9 mbo...@chromium.org
Verified and not reproducible on latest Chrome unstable build for Linux Ubuntu 12.04.
Labels: -Action-FeedbackNeeded
Feb 15, 2013
#10 bugdro...@chromium.org
The following revision refers to this bug:
    http://src.chromium.org/viewvc/chrome?view=rev&revision=182851

------------------------------------------------------------------------
r182851 | jianli@chromium.org | 2013-02-15T22:48:15.376734Z

Changed paths:
   M http://src.chromium.org/viewvc/chrome/branches/1410/src/chrome/browser/ui/gtk/panels/panel_gtk.cc?r1=182851&r2=182850&pathrev=182851
   M http://src.chromium.org/viewvc/chrome/branches/1410/src/chrome/browser/ui/gtk/panels/panel_titlebar_gtk.cc?r1=182851&r2=182850&pathrev=182851

Merge 182121
> Fix  bug 175858 : [Linux] The panel does not accept keyboard input when it is minimized and expanded
> 
> BUG=175858
> TEST=Manual test by following the repro steps in the bug
> 
> 
> Review URL: https://chromiumcodereview.appspot.com/12223114

TBR=jianli@chromium.org
Review URL: https://codereview.chromium.org/12287015
------------------------------------------------------------------------
Labels: -Merge-Approved merge-merged-1410
Feb 19, 2013
#11 ligim...@chromium.org
Works fine using 26.0.1410.10.
Status: Verified
Labels: TE-Verified-26.0.1410.10
Mar 10, 2013
#12 bugdro...@chromium.org
(No comment was entered for this change.)
Labels: -Area-UI -Feature-Panels -Mstone-26 Cr-UI M-26 Cr-UI-Browser-Panels
Sign in to add a comment

Powered by Google Project Hosting