Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Win32] Restore historical FileDialog behavior #142

Merged
merged 1 commit into from
May 13, 2022
Merged

[Win32] Restore historical FileDialog behavior #142

merged 1 commit into from
May 13, 2022

Conversation

nnemkin
Copy link
Contributor

@nnemkin nnemkin commented May 13, 2022

Miration to COM API in Bug 571571 resulted in unexpected changes in behavior. This patch restores original behavior.

  • Allow non-existent file names in SWT.OPEN dialog (Bug 579359).

  • Let the system remember last used directory and override FileDialog.setFilterPath (Bug 577773 and FileDialog does not remember previous directory #95).
    This change reverts the fix for Bug 577190 and Bug 578415.

@niraj-modi please see my comment https://bugs.eclipse.org/bugs/show_bug.cgi?id=577190#c17

Miration to COM API in Bug 571571 resulted in unexpected changes in
behavior. This patch restores original behavior.

 * Allow non-existent file names in SWT.OPEN dialog (Bug 579359).

 * Let the system remember last used directory and override
   FileDialog.setFilterPath (Bug 577773 and Issue #95).
   This change reverts the fix for Bug 577190 and Bug 578415.
@github-actions
Copy link
Contributor

Unit Test Results

     596 files       596 suites   10m 11s ⏱️
  4 009 tests   3 995 ✔️     5 💤   8  1 🔥
23 998 runs  23 845 ✔️ 134 💤 15  4 🔥

For more details on these failures and errors, see this check.

Results for commit 8af1758.

Copy link
Member

@niraj-modi niraj-modi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This patch works and it effectively reopens bug:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=577190

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FileDialog does not remember previous directory
2 participants