My favorites | Sign in
Logo
             
New issue | Search
for
| Advanced search | Search tips
Issue 4750: RTL: Bookmarks manager context menu should be displayed next to the mouse pointer
2 people starred this issue and may be notified of changes. Back to list
Status:  Verified
Owner:  laforge@chromium.org
Closed:  Nov 2008
Cc:  anantha@chromium.org, niranjan@chromium.org, mal.chromium
Type-Bug
Pri-2
OS-All
Area-BrowserUI
Mstone-1.0


Sign in to add a comment
 
Reported by venkataramana@chromium.org, Nov 24, 2008
Build: 0.4.154.25 (r5918)

What steps will reproduce the problem?
1. Run Chrome for any RTL LANG (Arabic is fine)
2. Bring up Bookmarks Manager (Ctrl+Shift+b)
3. Right click on any row in the ListView (Right Hand side)


What is the expected output?
Context menu should be displayed, just next to the mouse pointer.

What do you see instead?
Context menu is displayed far away from from mouse pointer BUT at the same
row level. (Screenshot attached)


BookmarkManager_ContextMenuFarAWay.PNG
166 KB   View   Download
Comment 1 by niranjan@chromium.org, Nov 25, 2008
Good to fix for 1.0 (if possible)
Status: Assigned
Owner: s...@chromium.org
Cc: m...@chromium.org
Labels: Mstone-1.0
Comment 2 by sky@chromium.org, Nov 25, 2008
Fixed in r5996:

Changes table_view to get position for context menu from the current
position of the mose. I've no idea why, but it seems when RTL the
position passed to OnContextMenu is wrong in weird inconsistent
ways. I thought it was interted, but that isn't always the case. I'm
also changing the anchor position to top_right when RTL.

BUG=4750
TEST=see bug

I'm passing to Mark. I don't see this as urgent, but feel free to merged into m1 
branch if you want.
Status: Started
Owner: m...@chromium.org
Comment 3 by laforge@chromium.org, Nov 26, 2008
Merged into release branch - r6066
Status: Fixed
Comment 4 by sky@chromium.org, Dec 01, 2008
 Issue 4752  has been merged into this issue.
Comment 5 by venkataramana@chromium.org, Dec 01, 2008
Looks like, I can still reproduce the issue in release branch 0.4.154.29 (r6138).
Sure, it is merged  (?).

-Venkat.
Comment 6 by venkataramana@chromium.org, Dec 02, 2008
(No comment was entered for this change.)
Owner: lafo...@chromium.org
Comment 7 by venkataramana@chromium.org, Dec 03, 2008
Rocks in 0.4.154.31(r6264) :)

-Venkat.
Status: Verified
Sign in to add a comment

Powered by Google Project Hosting