malnvenshorn / OctoPrint-FilamentManager

OctoPrint plugin to manage your filament inventory
GNU Affero General Public License v3.0
131 stars 59 forks source link

[Request] Change Dialog to Confirm Filament Selection #85

Closed bwright86 closed 6 years ago

bwright86 commented 6 years ago

I have started using the Filament Manager plugin for Octoprint, and it is a very helpful plugin for keeping track of my inventory.

I have enabled the setting to "Show dialog to confirm selected spools before starting/resuming the print job", and noticed that when a print starts, you are required to select the filament that was selected in the left navigation bar. If you do not select the same filament, it does not allow you to proceed with the print.

If the selected filament is incorrect for the print, the user has to:

  1. Cancel the dialog
  2. Update the filament in the left navigation bar, and
  3. Click Print again.

This increases the amount of time it takes to get a print running, if a subsequent prints require different filaments, and the user forgets to update the filament in the left navigation bar.

My suggestion would be to change this to a "Last chance to update your selected filament(s)" dialog, and allow users to see what is currently selected, and if wrong, update the selection to the correct filaments. It would then make the changes, and track the now updated filament selection during the print.

This would remove the need to:

malnvenshorn commented 6 years ago

This has been discusses several times and I'm not going to change the behavior of the confirmation dialog. The dialog is designed to prevent prints with the wrong spool(s) selected.

You said yourself "This increases the amount of time it takes to get a print running" this implies that you want to start your print as fast as possible. Now consider the following situation:

bwright86 commented 6 years ago

That is fair, if it has already been discussed, I can go along with that.

I was realizing how this worked, and thought of the above to help reduce friction to start a print. I didn't see previous issues about this feature, so i figured i would write in and share.

Thanks for responding and sharing your viewpoint.