-
Notifications
You must be signed in to change notification settings - Fork 7.3k
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
Update map.markdown for geolocation focus option #35454
Conversation
✅ Deploy Preview for home-assistant-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
📝 Walkthrough📝 WalkthroughWalkthroughThe changes involve updates to the documentation for the "Map card" in the dashboard configuration. Key modifications include enhanced descriptions of YAML configuration options, with specific references to related documentation sections. A new section on "Options for geolocation sources" has been added, detailing the configuration requirements for geolocation sources, including new fields such as Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Documentation
User->>Documentation: View Map card configuration
Documentation->>User: Display updated descriptions
Documentation->>User: Show new section on geolocation sources
User->>Documentation: Review configuration options
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
Actionable comments posted: 1
🧹 Outside diff range and nitpick comments (1)
source/_dashboards/map.markdown (1)
137-137
: Remove trailing colon from heading.
According to markdown style guidelines, headings should not end with punctuation.
-## Options for geolocation sources:
+## Options for geolocation sources
🧰 Tools
🪛 Markdownlint
137-137: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
- source/_dashboards/map.markdown (3 hunks)
🧰 Additional context used
🪛 LanguageTool
source/_dashboards/map.markdown
[grammar] ~149-~149: The singular determiner ‘this’ may not agree with the plural noun ‘entities’. Did you mean “these”?
Context: ...rue description: When set to false
, this entities of this source will not be con...
(THIS_NNS)
🪛 Markdownlint
source/_dashboards/map.markdown
137-137: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
🔇 Additional comments (2)
source/_dashboards/map.markdown (2)
61-61
: LGTM! Documentation cross-references improve navigation.
The added references to detailed sections help users find relevant configuration information more easily.
Also applies to: 65-65
169-170
: LGTM! Example clearly demonstrates the new focus option.
The example effectively shows how to configure a geolocation source with the focus option set to false.
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.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (1)
source/_dashboards/map.markdown (1)
137-137
: Remove trailing punctuation from heading
The heading should not end with a colon to comply with markdown style guidelines.
-## Options for geolocation sources:
+## Options for geolocation sources
🧰 Tools
🪛 Markdownlint
137-137: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
- source/_dashboards/map.markdown (3 hunks)
🧰 Additional context used
🪛 Markdownlint
source/_dashboards/map.markdown
137-137: Punctuation: ':'
Trailing punctuation in heading
(MD026, no-trailing-punctuation)
🔇 Additional comments (3)
source/_dashboards/map.markdown (3)
61-61
: LGTM: Improved cross-referencing
The added section references enhance navigation and make the documentation more user-friendly.
Also applies to: 65-65
149-149
: Fix grammar in description
There's a grammatical error in the description.
- description: When set to `false`, this entities of this source will not be considered for determining the default zoom or fit of the map.
+ description: When set to `false`, these entities of this source will not be considered for determining the default zoom or fit of the map.
169-170
: LGTM: Clear example of focus option
The added example effectively demonstrates the usage of the new focus option for geolocation sources.
Proposed change
For home-assistant/frontend#22535
Type of change
current
branch).current
branch).next
branch).next
branch).Additional information
Checklist
current
branch.next
branch.Summary by CodeRabbit