Bug 467205 - [Cocoa][10.10] FileDialog/DirectoryDialog with SWT.SHEET misplaced after slide-down animation
Summary: [Cocoa][10.10] FileDialog/DirectoryDialog with SWT.SHEET misplaced after slid...
Status: VERIFIED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: SWT (show other bugs)
Version: 4.5   Edit
Hardware: PC Mac OS X
: P3 normal with 1 vote (vote)
Target Milestone: 4.7.1   Edit
Assignee: Lakshmi P Shanmugam CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 467397 478927 492249 493080 508927 (view as bug list)
Depends on:
Blocks: 516837
  Show dependency tree
 
Reported: 2015-05-13 08:21 EDT by Markus Keller CLA
Modified: 2017-09-12 15:14 EDT (History)
10 users (show)

See Also:


Attachments
Directory dialog is detached (280.56 KB, image/png)
2017-08-14 12:35 EDT, Brian de Alwis CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Markus Keller CLA 2015-05-13 08:21:44 EDT
4.4.2 and 4.5 (master / I20150512-2100)
OS X 10.10.3 (didn't test earlier OS versions, but probably a new problem)

A FileDialog/DirectoryDialog with SWT.SHEET is misplaced after the slide-down animation. The sheet dialog gets centered horizontally and slightly higher than the center of the screen vertically.

Can be reproduced in the ControlExample, or in Eclipse, e.g. in File > Import... > Archive File > upper Browse... button. The lower Browse... button opens a custom Shell with SHEET. There, the sheet shell stays where it should.

Does not happen in the Save... sheet dialogs in TextEdit.app and Terminal.app. However, the bad location of SWT's sheet dialogs looks the same as the location of the (non-sheet) Open... dialogs in TextEdit/Terminal.
Comment 1 Arun Thondapu CLA 2015-05-15 14:22:23 EDT
*** Bug 467397 has been marked as a duplicate of this bug. ***
Comment 2 Rafael Karst CLA 2015-10-09 09:49:00 EDT
Is there anyone giving attention to this? I recommend increase the importance of this issue otherwise they will never fix this... It is always the same story...
Comment 3 Lakshmi P Shanmugam CLA 2015-10-13 02:43:37 EDT
*** Bug 478927 has been marked as a duplicate of this bug. ***
Comment 4 Lakshmi P Shanmugam CLA 2015-10-13 02:47:08 EDT
(In reply to Rafael Karst from comment #2)
> Is there anyone giving attention to this? 

The bug is targeted for 4.6 and we are already working on this.
Comment 5 Rafael Karst CLA 2015-10-13 08:06:04 EDT
That is Great! Thanks for the reply Lakshmi! =] (In reply to Lakshmi Shanmugam from comment #4)
> (In reply to Rafael Karst from comment #2)
> > Is there anyone giving attention to this? 
> 
> The bug is targeted for 4.6 and we are already working on this.
Comment 6 Eclipse Genie CLA 2015-10-23 04:14:12 EDT
New Gerrit change created: https://git.eclipse.org/r/58786
Comment 7 Lakshmi P Shanmugam CLA 2015-10-23 04:31:07 EDT
(In reply to Eclipse Genie from comment #6)
> New Gerrit change created: https://git.eclipse.org/r/58786

The patch fixes the misplaced SHEET problem in FileDialog. Same thing has to be done to DirectoryDialog.
The one part that is not working currently is that the patch doesn't capture user's response from the Sheet. That requires finding a way to use a objective-c block from Java code for the completion handler.

Will continue to work on this for M4.
Comment 8 Lakshmi P Shanmugam CLA 2015-12-08 04:21:37 EST
This change requires the build setup to move to OSX 10.6 or higher. Targetting for M5.
Comment 9 Lakshmi P Shanmugam CLA 2016-01-25 04:11:40 EST
Build machine has not yet moved up, need to defer to M6.
Comment 10 Lakshmi P Shanmugam CLA 2016-03-09 12:09:48 EST
(In reply to Lakshmi Shanmugam from comment #9)
> Build machine has not yet moved up, need to defer to M6.

Opened Bug 489305 to move builds to the EF build machine. If that doesn't work before M7, we'll move the builds to the local OSX 10.7 build machine.
Comment 11 Lakshmi P Shanmugam CLA 2016-04-23 02:43:36 EDT
*** Bug 492249 has been marked as a duplicate of this bug. ***
Comment 12 Lakshmi P Shanmugam CLA 2016-04-23 02:46:17 EDT
Will try for RC1.
Comment 13 Lakshmi P Shanmugam CLA 2016-05-06 06:08:00 EDT
Unfortunately couldn't complete this due to the work involved in the HiDPI related bugs. Will take it up for 4.6.1.
Comment 14 Steven Darnell CLA 2016-08-26 15:51:24 EDT
@Lakshmi, given the late date, I am assuming this bug will not make 4.6.1. Can you confirm?
Comment 15 Arun Thondapu CLA 2016-08-28 14:59:37 EDT
(In reply to Steven Darnell from comment #14)
> @Lakshmi, given the late date, I am assuming this bug will not make 4.6.1.
> Can you confirm?

Hi Steven,

Lakshmi is away for the next few months and I'll continue the work on this bug, but as you guessed it right, that won't happen in time for 4.6.1. Moving the target to 4.6.2.
Comment 16 Steven Darnell CLA 2016-08-29 12:14:08 EDT
@Arun, thanks for taking on the mantle of this issue.
Comment 17 Steven Darnell CLA 2016-10-25 17:22:09 EDT
@Arun, with the 4.6.2 release data about 60 days away, can you comment on the likelihood this issue will be addressed for its current target milestone?
Comment 18 Arun Thondapu CLA 2016-11-22 03:30:39 EST
(In reply to Steven Darnell from comment #17)
> @Arun, with the 4.6.2 release data about 60 days away, can you comment on
> the likelihood this issue will be addressed for its current target milestone?

Hi Steven, I had started looking into this issue but could not complete the fix yet due to a few other issues that cropped up in between. Its already too late for 4.6.2 (RC3 tomorrow) but I'll try to prioritize this for a fix in 4.7 M4 which will then be backported into 4.6.3.
Comment 19 Arun Thondapu CLA 2016-12-15 13:25:04 EST
*** Bug 508927 has been marked as a duplicate of this bug. ***
Comment 20 Lakshmi P Shanmugam CLA 2017-01-25 01:18:14 EST
Taking it up for 4.6.3
Comment 21 Lakshmi P Shanmugam CLA 2017-01-27 06:48:46 EST
*** Bug 493080 has been marked as a duplicate of this bug. ***
Comment 22 Lakshmi P Shanmugam CLA 2017-02-24 06:00:07 EST
More work is required in getting the native block code to work. Keeping it for 4.7.
Comment 23 Lakshmi P Shanmugam CLA 2017-05-25 07:27:29 EDT
(In reply to Lakshmi Shanmugam from comment #22)
> More work is required in getting the native block code to work. Keeping it
> for 4.7.

Wanted to fix this for 4.7, but had to look into other issues. Will target 4.7.1.
Comment 25 Lakshmi P Shanmugam CLA 2017-08-11 08:18:40 EDT
The above commit is pushed to the master branch for 4.8. The fix involves native changes and needs some good testing as part I-builds before it can be merged to the maintenance branch for 4.7.1.
Comment 26 Brian de Alwis CLA 2017-08-11 09:07:30 EDT
\o/

Looking forward to testing it out!
Comment 27 Lakshmi P Shanmugam CLA 2017-08-14 05:33:41 EDT
Verified with Build I20170813-2000 from (http://download.eclipse.org/eclipse/downloads/drops4/I20170813-2000/)

Looking forward to some community testing before backporting to 4.7.1.
Comment 28 Brian de Alwis CLA 2017-08-14 12:35:02 EDT
Created attachment 269839 [details]
Directory dialog is detached

Great, seems to work fine.

It turns out that there are a number of places that don't use SWT.SHEET that initially threw me off:

  - New Java Project's location > Browse
  - Import Maven Projects > Browse 

Will file bugs.
Comment 29 Eclipse Genie CLA 2017-08-16 03:59:41 EDT
New Gerrit change created: https://git.eclipse.org/r/103139
Comment 30 Lakshmi P Shanmugam CLA 2017-08-16 04:01:20 EDT
(In reply to Brian de Alwis from comment #28)
> Created attachment 269839 [details]
> Directory dialog is detached
> 
> Great, seems to work fine.
> 
> It turns out that there are a number of places that don't use SWT.SHEET that
> initially threw me off:
> 
>   - New Java Project's location > Browse
>   - Import Maven Projects > Browse 
> 
Thanks for testing, I too noticed that several dialogs don't use the SHEET style.

> Will file bugs.
Thanks!
Comment 32 Lakshmi P Shanmugam CLA 2017-08-16 06:20:14 EDT
Fixed for 4.7.1 & 4.8M2.
Comment 33 Lakshmi P Shanmugam CLA 2017-08-24 05:38:38 EDT
Verified in M20170823-1700
Comment 34 Lakshmi P Shanmugam CLA 2017-09-12 15:14:59 EDT
Verified with I20170911-2000.