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

Bug 343273

Summary: [Compatibility] Quick Access dialog opens on wrong monitor
Product: [Eclipse Project] e4 Reporter: Andrew Niefer <aniefer>
Component: UIAssignee: Remy Suen <remy.suen>
Status: RESOLVED FIXED QA Contact: Remy Suen <remy.suen>
Severity: normal    
Priority: P3 CC: pwebster, remy.suen
Version: unspecified   
Target Milestone: 4.1 RC3   
Hardware: PC   
OS: Linux   
Whiteboard:
Attachments:
Description Flags
screenshot
none
SearchField patch v1
none
SearchField patch v2
none
SearchField patch v03 none

Description Andrew Niefer CLA 2011-04-19 10:32:57 EDT
Created attachment 193585 [details]
screenshot

Running on Linux with two monitors, and Eclipse on the second monitor, 
when I use the quick access dialog, it appears on the primary monitor instead.

See attached screenshot.
Comment 1 Remy Suen CLA 2011-05-20 09:31:27 EDT
Created attachment 196216 [details]
SearchField patch v1

Instead of relying on getMonitor() I'm opting to just iterate over all the monitors returned by the display. I'm hoping this will do the trick.
Comment 2 Remy Suen CLA 2011-05-25 14:41:31 EDT
Created attachment 196594 [details]
SearchField patch v2

Fallback to the original code if the loop checking is not satisfied.
Comment 3 Remy Suen CLA 2011-05-25 14:42:12 EDT
(In reply to comment #2)
> Created attachment 196594 [details]
> SearchField patch v2

Patch released to CVS HEAD, let's see how it goes...
Comment 4 Paul Webster CLA 2011-05-27 08:08:07 EDT
Created attachment 196747 [details]
SearchField patch v03

This takes the same approach as JFace window.

PW
Comment 5 Remy Suen CLA 2011-05-27 08:19:29 EDT
(In reply to comment #4)
> Created attachment 196747 [details]
> SearchField patch v03

Looks good, thanks Paul. I've released this patch with javadoc additions to note where we copy/pasted those methods from.