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

[Bug] Cancelling out of "Open Image" file explorer makes the program unresponsive #70

Open
KyleKing54 opened this issue Oct 6, 2021 · 5 comments
Labels
bug Something isn't working

Comments

@KyleKing54
Copy link

KyleKing54 commented Oct 6, 2021

Describe the bug
When you select the File > Open option and launch a file explorer and then close the file explorer without selecting an image, the program window becomes unresponsive.

To Reproduce
Steps to reproduce the behavior:

  1. Go to File
  2. Click on Open
  3. Click "Cancel" on the file explorer window
  4. Program window is now unresponsive to inputs.

Expected behavior
The program should allow you to attempt to open a file again instead of freezing, or an error popup telling you no file was selected.

Desktop (please complete the following information):

  • OS: MacOS
  • Version: 10.13.6
@KyleKing54 KyleKing54 changed the title Cancelling out of "Open Image" file explorer makes the program unresponsive [Bug] Cancelling out of "Open Image" file explorer makes the program unresponsive Oct 6, 2021
@jody jody added the bug Something isn't working label Oct 7, 2021
@jody
Copy link
Contributor

jody commented Oct 7, 2021

All currently open windows also become unresponsive (except to minimize/restore actions).
Testing with Java versions 9 and 16 under macOS 11.6 produced the same results.

@lonespartan12
Copy link

checkpoint: We think the listener is not looking for the possible file explorer exit code and returning to where the file explorer was called from.

@KaizenPool
Copy link

I found the same problem as well. I clicked on the file menu option -> opened an image from the repository -> a new window opened with the picture, and I could quit the window. This makes me think that it is an interaction with the MAC OS rather than the program itself. In JMenu there is no code that shows the connection between the Finder Window and the application program.

KaizenPool added a commit to KaizenPool/imagelab that referenced this issue Oct 15, 2023
…elected is not the right file type it freezes. the goal of this commit to fix this problem. It also worked help fix issue MetroCS#70 that says when you click cancel on the dialog window the program crashes. This update that was focused on fix issue MetroCS#83 also fixe issue MetroCS#70. The fix to issue MetroCS#83 is not complete. It is eable to discern between the right type of file extension. It does not send a message to the user that it is the wrong file type, but it does allow the user to select another file. I worked on this edit on bluej and Mac OS
@Li-es
Copy link

Li-es commented Oct 31, 2023

I would like to be assigned to this issue.

@jody
Copy link
Contributor

jody commented Oct 31, 2023

I would like to be assigned to this issue.

Done!

@jody jody mentioned this issue Nov 20, 2023
Li-es added a commit to Li-es/imagelab that referenced this issue Nov 30, 2023
Fixed an error when cancelling out of selecting an item, the program froze and become unresponsive whilst throwing exceptions.
Was told to that it was too hard-coded so I'm looking to fix that.
Li-es added a commit to Li-es/imagelab that referenced this issue Nov 30, 2023
Li-es added a commit to Li-es/imagelab that referenced this issue Dec 5, 2023
When cancelling out of selecting an item, the program froze and become unresponsive whilst throwing exceptions, also when closing the program on intellij the program does not stop running. This fixed both of these issues while making them not hard-coded
@jody jody unassigned Li-es Dec 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: To do
Development

Successfully merging a pull request may close this issue.

5 participants