| Summary: | [Compatibility] Quick Access dialog opens on wrong monitor | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] e4 | Reporter: | Andrew Niefer <aniefer> | ||||||||||
| Component: | UI | Assignee: | 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: |
|
||||||||||||
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.
Created attachment 196594 [details]
SearchField patch v2
Fallback to the original code if the loop checking is not satisfied.
(In reply to comment #2) > Created attachment 196594 [details] > SearchField patch v2 Patch released to CVS HEAD, let's see how it goes... Created attachment 196747 [details]
SearchField patch v03
This takes the same approach as JFace window.
PW
(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. |
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.