Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 327200 - Diff-Window opens in background
Summary: Diff-Window opens in background
Status: RESOLVED INVALID
Alias: None
Product: EGit
Classification: Technology
Component: UI (show other bugs)
Version: 0.10.0   Edit
Hardware: Macintosh Mac OS X - Carbon (unsup.)
: P3 normal with 1 vote (vote)
Target Milestone: ---   Edit
Assignee: Jens Baumgart CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-10-07 05:40 EDT by leif.bladt CLA
Modified: 2013-05-04 16:38 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description leif.bladt CLA 2010-10-07 05:40:03 EDT
When I open the diff dialog with a double-click from the commit dialog, it sometimes happens that this dialog is opened behind the commit dialog.

The most annoying part is, that when I close the diff window, the commit dialog is "hidden". I need to use Expose (on OS X) the get back.

Unfortunately this happens not all the time and I don't have a clue, what triggers this behavior.
Comment 1 Jens Baumgart CLA 2010-10-27 04:14:42 EDT
- Code looks ok. Parent widget of diff window is the active workbench window.
- Could not reproduce the problem both on Windows 7 and Mac.
Comment 2 leif.bladt CLA 2010-10-27 04:59:32 EDT
(In reply to comment #1)
> - Code looks ok. Parent widget of diff window is the active workbench window.
> - Could not reproduce the problem both on Windows 7 and Mac.
Hmm,  with the newest nightly build (EGit 0.10.0.201010261520), this still occurs. What I did is:
 - Right-click on the project name
 - Team -> Commit...
 - Double-click on a modified file

Sometimes the compare window opens behind the commit window. Interestingly this depends on the commit dialog "session": Either all compare windows are opened in the background or all are opened correctly.

What other information could be useful to narrow this problem down?
Comment 3 Jens Baumgart CLA 2010-10-27 10:43:37 EDT
(In reply to comment #1)
> - Code looks ok. Parent widget of diff window is the active workbench window.
> - Could not reproduce the problem both on Windows 7 and Mac.

This is not correct.

org.eclipse.compare.CompareUI.openCompareDialog
always uses the active workbench window as parent. But the parent should be
the commit dialog. So we have the window hierarchy

Workbench Window
  |____Commit Dialog
  |____Compare Dialog

and it's not clearly defined which hwindow should be on top.
Comment 4 Robin Rosenberg CLA 2013-05-04 16:38:05 EDT
Closed due to being too old to investigate.