Add ID information to Lovelace view page.#7153
Merged
frenck merged 2 commits intohome-assistant:currentfrom Oct 29, 2018
Merged
Add ID information to Lovelace view page.#7153frenck merged 2 commits intohome-assistant:currentfrom
frenck merged 2 commits intohome-assistant:currentfrom
Conversation
bramkragten
changed the title
arsaboo
reviewed
Oct 27, 2018
source/lovelace/views.markdown
Outdated
|
|
||
| You can link to one view from another view by its ID. For this use cards that support navigation (`navigation_path`). Do not use special characters in IDs. | ||
|
|
||
| IDs are also used to identify a view for editing, we will add an ID to your view if you don't specify it. You can change this ID, as long as every view has an unique ID. |
Contributor
There was a problem hiding this comment.
I think it will help if you clarify the purpose of these IDs. May be we can just indicate that the IDs will be used for things like rearranging UI using drag-and-drop, etc.
arsaboo
reviewed
Oct 27, 2018
source/lovelace/views.markdown
Outdated
| id: | ||
| required: false | ||
| description: IDs are used in the URL, more info below. | ||
| description: IDs are used in the URL, more info below. If you don't specify an ID, it will be added automatically |
Contributor
There was a problem hiding this comment.
Is there a specific format for ID? length?
Contributor
frenck
requested changes
Oct 27, 2018
Member
frenck
left a comment
frenck
left a comment
There was a problem hiding this comment.
Please update the examples as well.
frenck
added
Hacktoberfest
Member
Author
|
What examples? I did the index page here: #7152 |
Member
|
@bramkragten Thx! Could you use the PR template, please? |
ghost
added
ready-for-review
frenck
approved these changes
Oct 29, 2018
Member
|
Thanks for updating the docs @bramkragten 👍 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description:
Pull request in home-assistant (if applicable): home-assistant/core#17498
Checklist:
nextis for changes and new documentation that will go public with the next home-assistant release. Fixes, changes and adjustments for the current release should be created againstcurrent.