Comment #13 on issue 5738 by bugdro...@chromium.org: RTL: App drop-down  
menu should be displayed below the favicon
http://code.google.com/p/chromium/issues/detail?id=5738

The following revision refers to this bug:
     http://src.chromium.org/viewvc/chrome?view=rev&revision=12631

------------------------------------------------------------------------
r12631 | x...@chromium.org | 2009-03-26 18:16:20 -0700 (Thu, 26 Mar 2009) |  
20 lines
Changed paths:
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/frame/opaque_browser_frame_view.cc?r1=12631&r2=12630
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/views/window/custom_frame_view.cc?r1=12631&r2=12630
    M  
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/views/window/window_win.cc?r1=12631&r2=12630

This CL fix issue 5724 - RTL: Windows context menu with Alt+Space Bar  
should be displayed on right side of the window.
(http://crbug.com/5724)

and issue 5738 - RTL: App drop-down menu should be displayed below the  
favicon
(http://crbug.com/5738)

In RTL UI, the x-axis of the system menu was computed as the  
MirroredXCoordinateInsideView(x-axis in LTR UI);
Also, the menu was right-aligned relative to the x-axis.

It fixes the system menu position in the following 3 situations:
1. press alt+space (such as in application short-cut, in chrome browser, in  
pop-up dialog such as option/clear browsing data), the system menu appears  
on the right side of the window (instead of left side, without the fix).

2. click favicon of application shortcut, the system menu appears on the  
right side of the window (instead of left side, without the fix).

3. right click on chrome browser tab, system menu appears at
left side of the click (instead of right side of the click).

Please be noted that:
System menu itself is not yet mirrored. This CL only fixes its display  
position.
Review URL: http://codereview.chromium.org/53120
------------------------------------------------------------------------


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---

Reply via email to