WolfwithSword / Bambu-HomeAssistant-Flows

A collection of HomeAssistant Dashboards and NodeRed Flows with a configurator to connect a BambuLabs printer to HomeAssistant through MQTT - based off of my Gists
32 stars 3 forks source link

UI Improvements for A1 Mini #5

Closed informagico closed 2 weeks ago

informagico commented 2 weeks ago

Hello, as anticipated here I made some changes to the A1 Mini cards for the HACS integration.

Files changed:

individual_ams_lite.yaml:

individual_control.yaml:

individual_printer_image_a1mini.yaml:


I also added the individual_ams_lite_details.yaml card template which includes the following entities:


Preview

image

image


Things I cannot verify:

Please let me know if anything is messed up and/or if you need me to make some changes πŸ˜„

NOTE: I tested and verified these changes only with the HACS Integration in Lan Mode Configuration (I have 2FA on my account and cannot use Bambu Cloud).

WolfwithSword commented 2 weeks ago

Great work! Been meaning to get a cleaner ams-lite image at some point but had trouble finding one.

Going to take a deeper look later and test it out locally (I don't have an A1 but that's mostly just cosmetic. Will load in my X1's data into it).

Couple things to note:

Addressing questions:

Also - feel free to keep your work to just the HACs version of the yamls. I will attempt to port them to my NodeRed versions for the approved changes for parity :)

informagico commented 2 weeks ago

Sure! Tomorrow I'll take a look to all your comments 😊

WolfwithSword commented 2 weeks ago

FYI I also changed the target branch to https://github.com/WolfwithSword/Bambu-HomeAssistant-Flows/tree/a1_mini_update such that I can sync equivalent nodered version changes before pushing to main branch all at once.

informagico commented 2 weeks ago
WolfwithSword commented 2 weeks ago

Ah okay good, just wanted to make sure the AMS tray order is matching the new angle you have, so that's all good

For the button aspect ration, I think that depends on the text you have for the title. Mine without your changes (aspect_ratio:2): image

With aspect ratio 1:
image

Hmm, if that's the case I might be inclined to push that out for a different investigation to modify, best to leave it at 2 for now.

And finally for the fan temp control, I wonder if for the A1 Mini the HACs integration is using the mqtt name for aux fan as the name for cooling fan instead. But regardless, they are all still present as fan entities so it is a bit weird.

Once you address some of the comment questions then I'll check again, merge to the a1mini branch and then work on the nodered versions. After that I will start process for the update version / release / merge to main.

informagico commented 2 weeks ago

For the aspect-ratio thing I have fiddled around trying to find a suitable solution (layout-card / grids / ...) but apparently there is no pratical solution I am aware of. Will think about it later, maybe it is even not that necessary to have that title and just show the button (with custom description?) if user has the switch available, otherwise just show nothing. What's your opinion?

informagico commented 2 weeks ago

Ok, finished on my side just take your time to review changes and let me know πŸ˜ƒ

Also, opened issue here for the fans

WolfwithSword commented 2 weeks ago

maybe it is even not that necessary to have that title and just show the button (with custom description?) if user has the switch available, otherwise just show nothing. What's your opinion?

Eh some people liked having the title there and treat it like some "labelled remote". Personally I'm indifferent but it is handy visually to weigh out the power button too. I think then it might be a case of "user can try tweaking the value if it's off in their screen res / ratio".

Will review the new changes today and test them out!

WolfwithSword commented 2 weeks ago

Everything looks good - just one comment on aspect_ratio. Hoping to tackle merging the changes to the nodered versions as well this weekend and then next week merge to main (which auto updates the configurators on the site - they just pull from the raw files from main branch and do string replacement on everything in {TEXT}'s FYI).