Skip to content
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

docs: add grid empty state documentation #3493

Merged
merged 4 commits into from
Jul 12, 2024
Merged

Conversation

tomivirkki
Copy link
Member

Add documentation for the Grid empty state feature

Part of vaadin/platform#5719

Screenshot 2024-06-17 at 10 35 50

@github-actions github-actions bot added the Language unchecked English language check has not been done label Jun 17, 2024
Comment on lines 14 to 16
{/* tag::snippet[] */}
<span slot="empty-state">No employees found.</span>
{/* end::snippet[] */}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Snippets for React / Lit should include the grid definition IMO, otherwise it may be confusing what to do with that single line.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@jouni jouni changed the base branch from next to latest June 17, 2024 13:39
@tomivirkki
Copy link
Member Author

@jouni why change the base to latest? Grid empty state is a 24.5 feature.

@tomivirkki tomivirkki changed the base branch from latest to next June 18, 2024 10:30
@russelljtdyer russelljtdyer added Language checking English language checking is in progress Language checked English language check done and removed Language unchecked English language check has not been done Language checking English language checking is in progress labels Jun 19, 2024
@jouni
Copy link
Member

jouni commented Jun 19, 2024

@tomivirkki because new, non-breaking, features are always merged to latest, and indicated with the versioning badge.

vaadin.com/docs/next should redirect to vaadin.com/docs/latest until we need a breaking change in the docs again.

@tomivirkki
Copy link
Member Author

tomivirkki commented Jun 19, 2024

@jouni Since this PR includes code examples that require API from 24.5, merging to latest would make it not compile. Need to wait until latest uses 24.5 then.

@jouni
Copy link
Member

jouni commented Jun 19, 2024

Yes, we do need to update the version to be able to use new APIs.

@tomivirkki tomivirkki changed the base branch from next to latest June 19, 2024 13:38
@jouni jouni merged commit c6ab869 into latest Jul 12, 2024
3 checks passed
@jouni jouni deleted the docs/grid-empty-state branch July 12, 2024 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language checked English language check done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants