-
Notifications
You must be signed in to change notification settings - Fork 60
Spool-Editing-Dialog: Auto weight/length feature #206
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
Conversation
…ment" and "Spool"
Data binding still missing
Reverted my auto formatting changes to reduce merge request footprint
…h-feature # Conflicts: # octoprint_SpoolManager/static/css/SpoolManager.css
Reverted my auto formatting changes to reduce merge request footprint
@OllisGit The only debatable conflict was the css for the dialog min/max height. Both your version 1.5 and this PR tried to fix the height issues. I chose my solution, as it also prevents the dialog from becoming too high on large displays. Although I'm not quite sure if the now deleted #full-sized-box div had other side effects. Solution in this PR:html:
css:
Solution of Version 1.5:html
js
|
Fyi: I will try your css solution...the minHeight is taken from OP sourcecode I only adjust the maxheight, e.g. AboutDialog
|
Hi @TiziG, I changed the ui a little bit (make it more compact). Smaller headers and replace all "Auto calculation prevents editing." with a single static note on top. ... I will change some other stuff (not related to this PR), so it will take a while when I release the next version. |
Btw. which calc.mode should be the default one? and should this be stored in plugins-settings or in each spool? |
- E release-channles - E stale-bot via github-actions - E #79, #87, #99, #119, #157, #176, #202, weight calculation during editing PR #206, thx al lot @TiziG - E #106 switching from travis to github-actions - E #107, #215 database-settings in the log will only be shown, if sql-logging is enabled - E #170 inform user about switching a spool already selected - E #213 sort/filter options in spool selection dialog - E Extrusion-Debugging: Show current extrusion during printing - B #188 touchUI conflict - B #193, #196 sorting/showing of remaining weight - B #205 changed to "browser-native" date/time picker - B #210 loadCatalogColors when color name is blank - B #211 button text changed - B #212 import CSV only if printer in idle-mode - B #216 FilamentManager and SpoolManager used same viewmodel - B #217 when print is paused the extruded filament is assigned to the spool and the counter is reseted
Hi @TiziG, your merge is now in the latest release. Thanks again, very helpful! I changed some small things...e.g. remaining field is now in readonly mode. The value is only calculated. Please test and give a feedback if you found some issues. |
Awesome, thanks for your work @OllisGit ! |
New functionality for the spool editing dialog. Let me know what you think!
New Functionality:
UI Changes
Implementation Details
The new logic is implemented solely in the dialog view model. The new input fields (for combined weight) are derived fields and do not get stored outside the view model (hence no db changes).
Related Tickets
implements #202
Screenshots