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 -~----------~----~----~----~------~----~------~--~---