Skip to content

Conversation

@Flight-Lab
Copy link
Contributor

Description

Automatically switches to kitt_bar overlay when entering camera view and restores configured default overlay when navigating away. This is to keep the camera feed(s) visible while using Assist.

Changes:

  • camera.yaml: Added var_setup_camera_overlay variable with cleanup logic
  • advancedcamera.yaml: Added var_setup_camera_overlay variable with cleanup logic
  • Both views now force kitt_bar overlay on entry via _overlay_init custom field
  • Overlay restoration happens automatically when var_current_view !== 'camera'

Automatically switches to kitt_bar overlay when entering camera view and restores configured default overlay when navigating away.
This is to keep the camera feed(s) visible while using Assist.
@dinki
Copy link
Owner

dinki commented Dec 8, 2025

I get the idea but wondering if this will create questions. Additionally, we may some day remove that overlay, rename it, prefer another, etc.

Would it be too much to ask to make this optional? How about making it selectable from the list of available?

Am I overcomplicating this unnecessarily?

@Flight-Lab
Copy link
Contributor Author

No, I don't think you're overcomplicating it at all. I also realized it shouldn't change the flashing bar to the kit bar cause that is already out of the way.

As for making it optional, we absolutely could do that with an additional attribute. We could probably use an additional attribute reflecting the header config setting to fix the #294 / #297 issue as well. That begs the question, is there a limit to how many attributes we want to add, whether through the master/device configuration or through the blueprints? I've been trying to limit the amount of attributes added, but it is certainly an easy way of passing information to the front end.

@dinki
Copy link
Owner

dinki commented Jan 3, 2026

Circling back. Is this one ready?

@Flight-Lab
Copy link
Contributor Author

You made some good points in your last comment but I haven't gotten around to implementing that logic yet.

@Flight-Lab Flight-Lab marked this pull request as draft January 3, 2026 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants