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

Save Module progress indicator may not appear if computer is too fast #336

Closed
8749236 opened this issue Jul 1, 2024 · 2 comments
Closed
Labels
bug Something isn't working

Comments

@8749236
Copy link

8749236 commented Jul 1, 2024

dnSpyEx version

6.5.1

Describe the Bug

The progress indicator of Save Module dialog closes immediately when completed, but the progress indicator may not appear at all if computer is way too fast. (E.g. 14700k + 2x32GB 7200MHz + small program)

How To Reproduce

  1. (Optional) Build a very fast computer with very high score
  2. Open a small C# program with dnSpyEx 6.5.1
  3. Make any changes to the program.
  • Add or delete a resource
  • Find a class and edit its source code
  • etc.
  1. File -> Save Module -> Ok
  2. The progress indicator may or may not appears, since the processing takes less time for UI to appear, and UI gets closed before it can be displayed.
  3. Open file property and observe its modified date has been changed.

Expected Behavior

The progress indicator should contain a button for dismissing it when progress bar is full, it should let user acknolwedge the completion then dismiss itself.
It should have a checkbox for auto dismissing in the future.
In options menu, there should be an option to disable auto dismissing.

Actual Behavior

The progress indicator may or may not appear.
When it appears, it closes itself immediately when progress bar is full, if user isn't paying attention, then they don't know whether the operation is finished.
When it doe not appear, there is no other feedback on whether the action actually took place.

Alternative is add a status bar and throw a prompt there. Example of VS2022's save prompt.
image

Additional Context

No response

@8749236 8749236 added the bug Something isn't working label Jul 1, 2024
@ElektroKill
Copy link
Member

Hi,

Sorry for the late response.

What do you think of adding an artificial delay to the dialog closing of 250ms, that is if the save operation completes the dialog will stay open for at least 250ms even if the saving was faster? If the saving operation takes longer, no delay will be in effect.

This should be relatively easy to implement.

Let me know what you think!

@8749236
Copy link
Author

8749236 commented Jul 31, 2024 via email

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
None yet
Development

No branches or pull requests

2 participants