Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 389450

Summary: Populating menu item does not size and position well for the first time.
Product: [ECD] Orion Reporter: libing wang <libingw>
Component: ClientAssignee: libing wang <libingw>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: susan
Version: 0.5   
Target Milestone: 1.0 M2   
Hardware: PC   
OS: Windows 7   
Whiteboard:

Description libing wang CLA 2012-09-12 17:39:38 EDT
As fixed in Bug 386043, you can now populate a list saved search in the search option. But the first time when you populate that list, the position of the list is kind of shifting. From the second time, everything is OK.
Comment 1 Susan McCourt CLA 2012-09-13 10:53:05 EDT
my guess is that there is asynch population of the searches happening (after dijit calculates the size), but I'll take a look...
Comment 2 libing wang CLA 2012-09-13 12:00:06 EDT
Just a hint:  I removed the async population code and added two testing items instead. But it seems the issue still happens for the first time.
Comment 3 libing wang CLA 2012-09-18 15:48:40 EDT
fixed with http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=18c993a0728207f6085da0ad43e6663255bafe66.

The trick is that for the first time when "_openPopup" is triggered, the content of dijit.PopupMenuItem is zero.

So I filled up the content of dijit.PopupMenuItem  with the current search preference when page is loaded. That means before the "_openPopup" event is triggered, it already know roughly.
Comment 4 libing wang CLA 2012-09-18 16:04:02 EDT
I think this issue always hits on search options because it is always sitting on the right side of the browser.
However, in extreme case it happens in navigator action menu as well:
Make the browser smaller to make the action menu sit on the right side. Then for the firs time you popup the "copy to" or "move to" you will see the same issue.
Susan, what do you think?