-
-
Notifications
You must be signed in to change notification settings - Fork 403
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
feat: embed-linked-page card views and toolbar #5955
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
|
60c9747
to
4604756
Compare
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.
ff021ae
to
155b710
Compare
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.
Functionally, it meets expectations, and there are some UI details worth optimizing.
6f46770
to
02f32d1
Compare
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.
I think we need to rename the all embed-linked-page
block type to embed-linked-doc
due to product naming change from @houjoe0829.
We can land this PR and have the naming change in further PR. The naming update won't be considered as breaking change before there is visible entry for users. So we can make this happen without writing migration rules.
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.
🎉
c936402
to
3a43f93
Compare
closes #5713
Linked Page Playground Demo
Changes: