mainsail-crew / mainsail

Mainsail is the popular web interface for managing and controlling 3D printers with Klipper.
https://docs.mainsail.xyz
GNU General Public License v3.0
1.72k stars 370 forks source link

Additional filtering on Spoolman integration #1702

Open adamcstorm opened 10 months ago

adamcstorm commented 10 months ago

Requested feature:

I'm loving spoolman and the Mainsail integration! I have some suggestions for the spool selection dialog:

Solves the following problem:

When there are a large number of spools, it can be cumbersome to find the right spool. This would help speed up the process, particularly on mobile/small screens.

Additional information:

No response

meteyou commented 10 months ago

You can use the current search bar to filter everything. So filament type, color, and brand can all be filtered already.

adamcstorm commented 10 months ago

That is a handy feature. However, I am of the opinion that having a dropdown+checkbox for a filter is more useful, particularly on a touch interface. Having the ability for a filter to be automatically applied based on filament detected in gcode file would be super useful IMO. It's certainly not a deal breaker. Just something I thought could improve usability. There's already a sort-by option, and I suppose there's a path to extend that functionality to a filtered list.

rackrick commented 10 months ago

Presetting the search with the filament typ if a wrong filament is deteced might be an option. It comes with the downside does names containing the keyword also will be shown but this is proably an edge case. Anyways... I think additional dropdowns or checkboxes will overload the interface. If we have to hide them behind a search menu or something you also could just type the name into the searchbar even on mobile devices.

meteyou commented 10 months ago

I also wanted to add filament type. This will be implemented relatively quickly.

In terms of the type of implementation, the manufacturer should also be the same and fast. But I think it's the same as @rackrick. It could become too much with too little added benefit.

I will add the filament type as a dropdown and check the UI...

adamcstorm commented 10 months ago

I trust you all with the UI. It’s been a pleasure to use! I appreciate you considering the request and look forward to seeing what you come up with if it’s deemed an improvement.