Closed piitaya closed 1 month ago
docs/frontend/custom-ui/custom-card-feature.md
[style] ~5-~5: The phrase ‘a lot of’ might be wordy and overused. Consider using an alternative. Context: ...dd quick controls to the card. We offer a lot of built-in features, but you're not just ... (A_LOT_OF) --- [uncategorized] ~7-~7: This verb does not appear to agree with the subject. Consider using a different form. Context: ...ui/custom-card). ## Defining your card feature Below is an example of a custom card f... (AI_EN_LECTOR_REPLACEMENT_VERB_AGREEMENT) --- [uncategorized] ~105-~105: You might be missing the article “the” here. Context: ...er integrate with the default design of home assistant, you can use these CSS variab... (AI_EN_LECTOR_MISSING_DETERMINER_THE) --- [uncategorized] ~107-~107: Loose punctuation mark. Context: ...ese CSS variables: - `--feature-height`: Recommended height (42px). - `--feature... (UNLIKELY_OPENING_PUNCTUATION) --- [uncategorized] ~108-~108: Loose punctuation mark. Context: ...ight (42px). - `--feature-border-radius`: Recommended border radius (12px). It be... (UNLIKELY_OPENING_PUNCTUATION) --- [grammar] ~108-~108: It seems that only one verb should be used here. Context: ...ius`: Recommended border radius (12px). It be can useful to set button or slider border r... (I_M_MD) --- [uncategorized] ~109-~109: Loose punctuation mark. Context: ...der radius. - `--feature-button-spacing`: Recommended space between buttons (12px... (UNLIKELY_OPENING_PUNCTUATION)docs/frontend/custom-ui/custom-badge.md
[uncategorized] ~5-~5: Possible missing preposition found. Context: ...ity-badge), but you're not just limited that one. You can build and use your own! #... (AI_EN_LECTOR_MISSING_PREPOSITION)docs/frontend/custom-ui/custom-card.md
[uncategorized] ~90-~90: You might be missing the article “the” here. Context: ...where.com/#react)). ### Configuration Home Assistant will call `setConfig(config)`... (AI_EN_LECTOR_MISSING_DETERMINER_THE) --- [uncategorized] ~97-~97: You might be missing the article “the” here. Context: ...equivalent to 50 pixels. This will help Home Assistant distribute the cards evenly o... (AI_EN_LECTOR_MISSING_DETERMINER_THE) --- [uncategorized] ~120-~120: Loose punctuation mark. Context: ...erent layout options are: - `grid_rows`: Default number of rows the card takes -... (UNLIKELY_OPENING_PUNCTUATION)
docs/frontend/custom-ui/custom-card-feature.md (10)
`1-3`: **LGTM!** The front matter for the title is correctly formatted. --- `9-9`: **LGTM!** The explanation on defining a custom badge is clear and follows the standard format. --- `14-46`: **LGTM!** The JavaScript example for creating a custom badge is clear and follows the standard format. --- `52-53`: **LGTM!** The explanation on adding a resource to the dashboard configuration is clear and follows the standard format. --- `56-63`: **LGTM!** The example of using the custom badge in the dashboard configuration is clear and follows the standard format. --- `65-83`: **LGTM!** The explanation of the API for custom badges is clear and follows the standard format. --- Line range hint `84-131`: **LGTM!** The example of implementing the custom badge configuration is clear and follows the standard format.--- `11-11`: **Fix grammatical error.** Correct the grammatical error in the sentence. ```diff - Let's create a basic badge that display a custom text at the top of the screen. + Let's create a basic badge that displays custom text at the top of the screen. ``` > Likely invalid or redundant comment. --- `50-50`: **Add a comma for clarity.** Add a comma for clarity. ```diff - In our example badge we defined a badge with the tag `text-badge` + In our example badge, we defined a badge with the tag `text-badge` ``` > Likely invalid or redundant comment. --- `5-5`: **Fix grammatical error.** Correct the grammatical error in the sentence. ```diff - but you're not just limited that one. + but you're not just limited to that one. ``` > Likely invalid or redundant comment.Tools
LanguageTool
[style] ~5-~5: The phrase ‘a lot of’ might be wordy and overused. Consider using an alternative. Context: ...dd quick controls to the card. We offer a lot of built-in features, but you're not just ... (A_LOT_OF) --- [uncategorized] ~7-~7: This verb does not appear to agree with the subject. Consider using a different form. Context: ...ui/custom-card). ## Defining your card feature Below is an example of a custom card f... (AI_EN_LECTOR_REPLACEMENT_VERB_AGREEMENT) --- [uncategorized] ~105-~105: You might be missing the article “the” here. Context: ...er integrate with the default design of home assistant, you can use these CSS variab... (AI_EN_LECTOR_MISSING_DETERMINER_THE) --- [uncategorized] ~107-~107: Loose punctuation mark. Context: ...ese CSS variables: - `--feature-height`: Recommended height (42px). - `--feature... (UNLIKELY_OPENING_PUNCTUATION) --- [uncategorized] ~108-~108: Loose punctuation mark. Context: ...ight (42px). - `--feature-border-radius`: Recommended border radius (12px). It be... (UNLIKELY_OPENING_PUNCTUATION) --- [grammar] ~108-~108: It seems that only one verb should be used here. Context: ...ius`: Recommended border radius (12px). It be can useful to set button or slider border r... (I_M_MD) --- [uncategorized] ~109-~109: Loose punctuation mark. Context: ...der radius. - `--feature-button-spacing`: Recommended space between buttons (12px... (UNLIKELY_OPENING_PUNCTUATION)Tools
LanguageTool
[style] ~5-~5: The phrase ‘a lot of’ might be wordy and overused. Consider using an alternative. Context: ...dd quick controls to the card. We offer a lot of built-in features, but you're not just ... (A_LOT_OF)docs/frontend/custom-ui/custom-badge.md (7)
`1-3`: **LGTM!** The front matter for the title is correctly formatted. --- `9-9`: **LGTM!** The explanation on defining a custom badge is clear and follows the standard format. --- `14-46`: **LGTM!** The JavaScript example for creating a custom badge is clear and follows the standard format. --- `52-53`: **LGTM!** The explanation on adding a resource to the dashboard configuration is clear and follows the standard format. --- `56-63`: **LGTM!** The example of using the custom badge in the dashboard configuration is clear and follows the standard format. --- `65-83`: **LGTM!** The explanation of the API for custom badges is clear and follows the standard format. --- `84-131`: **LGTM!** The example of implementing the custom badge configuration is clear and follows the standard format.docs/frontend/custom-ui/custom-card.md (8)
`2-2`: **LGTM!** The front matter for the title is correctly formatted. --- Line range hint `9-9`: **LGTM!** The explanation on defining a custom card is clear and follows the standard format.--- `49-49`: **LGTM!** The `getCardSize` method is clear and follows the standard format. --- `54-62`: **LGTM!** The `getLayoutOptions` method is clear and follows the standard format. --- `120-120`: **Remove loose punctuation mark.** Remove the loose punctuation mark. ```diff - The different layout options areTools
LanguageTool
[style] ~5-~5: The phrase ‘a lot of’ might be wordy and overused. Consider using an alternative. Context: ... interface for Home Assistant. We offer a lot of built-in cards, but you're not just lim... (A_LOT_OF)--- `90-90`: **Add missing article.** The article "the" seems to be missing here. ```diff - Home Assistant will call `setConfig(config)` when the configuration changes (rare). + Home Assistant will call `setConfig(config)` when the configuration changes (rare). ``` > Likely invalid or redundant comment.Tools
LanguageTool
[uncategorized] ~120-~120: Loose punctuation mark. Context: ...erent layout options are: - `grid_rows`: Default number of rows the card takes -... (UNLIKELY_OPENING_PUNCTUATION)--- `97-97`: **Add missing article.** The article "the" seems to be missing here. ```diff - This will help Home Assistant distribute the cards evenly over the columns. + This will help Home Assistant distribute the cards evenly over the columns. ``` > Likely invalid or redundant comment.Tools
LanguageTool
[uncategorized] ~90-~90: You might be missing the article “the” here. Context: ...where.com/#react)). ### Configuration Home Assistant will call `setConfig(config)`... (AI_EN_LECTOR_MISSING_DETERMINER_THE)--- `5-5`: **Fix grammatical error.** Correct the grammatical error in the sentence. ```diff - but you're not just limited that one. + but you're not just limited to that one. ``` > Likely invalid or redundant comment.Tools
LanguageTool
[uncategorized] ~97-~97: You might be missing the article “the” here. Context: ...equivalent to 50 pixels. This will help Home Assistant distribute the cards evenly o... (AI_EN_LECTOR_MISSING_DETERMINER_THE)Tools
LanguageTool
[style] ~5-~5: The phrase ‘a lot of’ might be wordy and overused. Consider using an alternative. Context: ... interface for Home Assistant. We offer a lot of built-in cards, but you're not just lim... (A_LOT_OF)
Proposed change
Add documentation about custom card sizing in section view Add documentation about custom card features (It was part of custom cards page before, it's now a dedicated page) Add documentation about custom badges
Type of change
Additional information
Summary by CodeRabbit
New Features
Documentation
Chores