-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Use grid options instead of layout options #2440
Conversation
📝 WalkthroughWalkthroughThe documentation for the custom card in Home Assistant has been updated to reflect changes in terminology and structure regarding layout options. The method Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant CustomCard
participant GridOptions
User->>CustomCard: Request layout options
CustomCard->>GridOptions: Call getGridOptions()
GridOptions-->>CustomCard: Return updated grid options
CustomCard-->>User: Display card with new grid layout
🪧 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: 0
🧹 Outside diff range and nitpick comments (3)
docs/frontend/custom-ui/custom-card.md (3)
55-62
: Consider making row configuration more flexible.The current implementation sets
rows
,min_rows
, andmax_rows
all to 3, which effectively locks the card height. Consider allowing more flexibility by either:
- Removing
min_rows
andmax_rows
to allow dynamic resizing, or- Setting different values for min/max to provide a range.
getGridOptions() { return { rows: 3, columns: 6, - min_rows: 3, - max_rows: 3, + min_rows: 2, // Allow slightly smaller + max_rows: 4, // Allow slightly larger }; }
118-126
: Fix formatting of grid options list.The list formatting needs improvement for better readability.
The different grid options are: - -- `rows`: Default number of rows the card takes -- `min_rows`: Minimal number of rows the card takes -- `max_rows`: Maximal number of rows the card takes -- `columns`: Default number of columns the card takes (set it to `full` to enforce your card to be full width) -- `min_columns`: Minimal number of columns the card takes -- `max_columns`: Maximal number of columns the card takes + + - `rows`: Default number of rows the card takes + - `min_rows`: Minimal number of rows the card takes + - `max_rows`: Maximal number of rows the card takes + - `columns`: Default number of columns the card takes (set it to `full` to enforce your card to be full width) + - `min_columns`: Minimal number of columns the card takes + - `max_columns`: Maximal number of columns the card takes🧰 Tools
🪛 LanguageTool
[uncategorized] ~120-~120: Loose punctuation mark.
Context: ...he different grid options are: -rows
: Default number of rows the card takes -...(UNLIKELY_OPENING_PUNCTUATION)
132-141
: Consider enhancing the example implementation.While the current example is good, it could be more comprehensive by:
- Demonstrating the recommended column values (multiples of 3)
- Showing how to use
min_columns
andmax_columns
- Including the
full
width option mentioned in the documentationpublic getGridOptions() { return { rows: 2, - columns: 6, + columns: 6, // Multiple of 3 for better alignment min_rows: 2, + min_columns: 3, // Example of column constraints + max_columns: 9, }; }Add another example demonstrating full-width usage:
// Example of full-width card public getGridOptions() { return { rows: 1, columns: "full", // Forces card to full width min_rows: 1, max_rows: 2, }; }
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (1)
docs/frontend/custom-ui/custom-card.md
(2 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/frontend/custom-ui/custom-card.md
[uncategorized] ~120-~120: Loose punctuation mark.
Context: ...he different grid options are: - rows
: Default number of rows the card takes -...
(UNLIKELY_OPENING_PUNCTUATION)
🔇 Additional comments (2)
docs/frontend/custom-ui/custom-card.md (2)
109-110
: LGTM! Clear introduction to grid options.
The documentation clearly explains the purpose and context of the grid options method.
114-116
: LGTM! Well-defined cell dimensions.
The cell dimensions are clearly specified with exact measurements.
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
🧹 Nitpick comments (3)
docs/frontend/custom-ui/custom-card.md (3)
55-61
: Verify UpdatedgetGridOptions()
Implementation
The updatedgetGridOptions()
method now returns the keysrows
,columns
,min_rows
, andmax_rows
with the new default (columns: 6
). Consider whether additional keys such asmin_columns
andmax_columns
should be included to fully document the new grid options. If these keys are optional, a brief note could help clarify the intended usage.getGridOptions() { return { - rows: 3, - columns: 6, - min_rows: 3, - max_rows: 3, + rows: 3, + columns: 6, + min_rows: 3, + max_rows: 3, + // Optionally add: + // min_columns: <default_value>, + // max_columns: <default_value or undefined>, }; }
109-110
: Refine Preposition in Grid Option Description
In the explanatory text describing grid behavior, consider changing the phrase "take in the grid" to "take into the grid" for improved clarity and consistency with standard English usage.-You can define a `getGridOptions` method that returns the min, max and default number of cells your card will take in the grid if your card is used in the [sections view](https://www.home-assistant.io/dashboards/sections/). +You can define a `getGridOptions` method that returns the min, max, and default number of cells your card will take into the grid if your card is used in the [sections view](https://www.home-assistant.io/dashboards/sections/).🧰 Tools
🪛 LanguageTool
[uncategorized] ~109-~109: The preposition “into” seems more likely in this position than the preposition “in”.
Context: ...rds/sections/). Each section is divided in 12 columns. If you don't define this me...(AI_EN_LECTOR_REPLACEMENT_PREPOSITION_IN_INTO)
126-127
: Improve Grammar in Grid Options List
For the recommendation on column defaults, consider adjusting the phrasing to include the missing article and improve readability. For example, change "use multiple of 3" to "use a multiple of 3" and rephrase the ending to enhance clarity.-For the number of columns, it's `highly` recommended to use multiple of 3 for the default value (`3`, `6`, `9` or `12`) so your card will have better looking on the dashboard by default. +For the number of columns, it's highly recommended to use a multiple of 3 for the default value (e.g., `3`, `6`, `9`, or `12`) so that your card displays better on the dashboard by default.🧰 Tools
🪛 LanguageTool
[uncategorized] ~127-~127: You might be missing the article “a” here.
Context: ...lumns, it'shighly
recommended to use multiple of 3 for the default value (3
,6
, `...(AI_EN_LECTOR_MISSING_DETERMINER_A)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
docs/frontend/custom-ui/custom-card.md
(2 hunks)
🧰 Additional context used
🪛 LanguageTool
docs/frontend/custom-ui/custom-card.md
[uncategorized] ~109-~109: The preposition “into” seems more likely in this position than the preposition “in”.
Context: ...rds/sections/). Each section is divided in 12 columns. If you don't define this me...
(AI_EN_LECTOR_REPLACEMENT_PREPOSITION_IN_INTO)
[uncategorized] ~120-~120: Loose punctuation mark.
Context: ...he different grid options are: - rows
: Default number of rows the card takes. ...
(UNLIKELY_OPENING_PUNCTUATION)
[uncategorized] ~127-~127: You might be missing the article “a” here.
Context: ...lumns, it's highly
recommended to use multiple of 3 for the default value (3
, 6
, `...
(AI_EN_LECTOR_MISSING_DETERMINER_A)
🔇 Additional comments (1)
docs/frontend/custom-ui/custom-card.md (1)
132-138
: Example Implementation Clarity
The provided example for thegetGridOptions()
method correctly demonstrates the new method signature and default grid sizing. This example is clear and consistent with the prior implementation.
Proposed change
Grid options is replacing layout options in
2024.11
Type of change
Additional information
Summary by CodeRabbit
getLayoutOptions
togetGridOptions
and updated its return structure for clarity.