Skip to content

Conversation

@McGiverGim
Copy link
Contributor

@McGiverGim McGiverGim commented Dec 6, 2025

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

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

Copilot AI review requested due to automatic review settings December 6, 2025 09:21
@home-assistant home-assistant bot added has-parent This PR has a parent PR in another repo next This PR goes into the next branch labels Dec 6, 2025
@netlify
Copy link

netlify bot commented Dec 6, 2025

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 2f07e5b
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/6933fc48b9b4ea0008dd9f08
😎 Deploy Preview https://deploy-preview-42348--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

Copilot AI left a 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 invert parameter (boolean, defaults to true) 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
Copy link

Copilot AI Dec 6, 2025

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.

Copilot uses AI. Check for mistakes.
Copy link
Contributor Author

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.

@McGiverGim McGiverGim force-pushed the add_invert_cover_feature branch from 2a5eb0a to 9d6378e Compare December 6, 2025 09:46
@McGiverGim
Copy link
Contributor Author

The branch was "incorrect", fixed it now, thanks!

@timmo001 timmo001 removed their request for review December 6, 2025 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-parent This PR has a parent PR in another repo next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant