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

fix: update Headless UI link #3563

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

tomquirk
Copy link

Per title, updates broken Headless UI link to https://tanstack.com/table/v8/docs/introduction#what-is-headless-ui

Copy link

vercel bot commented Apr 21, 2024

@tomquirk is attempting to deploy a commit to the shadcn-pro Team on Vercel.

A member of the Team first needs to authorize it.

@@ -12,7 +12,7 @@ links:

Every data table or datagrid I've created has been unique. They all behave differently, have specific sorting and filtering requirements, and work with different data sources.

It doesn't make sense to combine all of these variations into a single component. If we do that, we'll lose the flexibility that [headless UI](https://tanstack.com/table/v8/docs/guide/introduction#what-is-headless-ui) provides.
It doesn't make sense to combine all of these variations into a single component. If we do that, we'll lose the flexibility that [Headless UI](https://tanstack.com/table/v8/docs/introduction#what-is-headless-ui) provides.

Choose a reason for hiding this comment

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

- [headless UI]
+ [Headless UI]

this is required?

@zk1st
Copy link

zk1st commented Apr 23, 2024

This seems similar to pull request #3561. Perhaps we can combine these changes?

@ruru-m07
Copy link

I think there is 3 PRs for the same issue

@zk1st
Copy link

zk1st commented Apr 23, 2024

I think there is 3 PRs for the same issue

Yeah, #3421 too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants