-
-
Notifications
You must be signed in to change notification settings - Fork 8k
Add invert cover feature #42348
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
base: next
Are you sure you want to change the base?
Add invert cover feature #42348
Conversation
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request adds an invert parameter to the cover position feature for dashboards, allowing users to control whether the position slider is inverted (default behavior) or displayed in a non-inverted manner.
Key changes:
- Adds optional
invertparameter (boolean, defaults totrue) to the cover-position feature configuration - Updates several dashboard-related screenshot images
- Minor improvement to zoneminder integration documentation (adds glossary term reference)
Reviewed changes
Copilot reviewed 179 out of 186 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| source/dashboards/features.markdown | Adds documentation for new invert parameter in cover-position feature |
| source/_integrations/zoneminder.markdown | Adds glossary term reference for "integration" |
| source/images/dashboards/*.png | Updated dashboard screenshots (binary files, not reviewed) |
| ```yaml | ||
| features: | ||
| - type: "cover-position" | ||
| invert: false |
Copilot
AI
Dec 6, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The indentation is inconsistent with the YAML configuration block. The invert property should be indented with 2 spaces (matching the type property on line 313), but it appears to have 4 spaces of indentation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is how it's done in other parts of this card, for exaple in the Bar Gauge part.
2a5eb0a to
9d6378e
Compare
|
The branch was "incorrect", fixed it now, thanks! |
Proposed change
The feature for cover position has the slider "inverted". I understand the reason of this decision, but this is driving me crazy since the beginning and I'm not the only one: https://community.home-assistant.io/t/reverse-slider-display/746748 so I decided to add an "optional" parameter to disable the inversion.
Type of change
currentbranch).currentbranch).nextbranch).nextbranch).Additional information
Checklist
currentbranch.nextbranch.