Skip to content

[Playground] Feature: Headless components #5535

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

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

kien-ngo
Copy link
Contributor

@kien-ngo kien-ngo commented Nov 28, 2024

CNCT-2588


PR-Codex overview

This PR focuses on enhancing the playground-web application by adding new components for NFT and account functionalities, improving user interface elements, and integrating headless UI components for seamless wallet interactions.

Detailed summary

  • Added ArticleCardIndex for displaying information about Headless UI.
  • Introduced new navigation links for Headless UI.
  • Expanded Account and NFT components with additional features and examples.
  • Created new pages for NFT Components and Account Components.
  • Enhanced query parameters for useQuery in account-related components.
  • Developed multiple examples for NFT and Account components, showcasing usage and customization options.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Copy link

changeset-bot bot commented Nov 28, 2024

⚠️ No Changeset found

Latest commit: e1fd832

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@vercel vercel bot temporarily deployed to Preview – docs-v2 November 28, 2024 13:10 Inactive
Copy link

vercel bot commented Nov 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 10, 2024 7:34am
thirdweb_playground ✅ Ready (Inspect) Visit Preview 1 resolved Dec 10, 2024 7:34am
thirdweb-www ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 10, 2024 7:34am
wallet-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 10, 2024 7:34am

Copy link
Contributor

graphite-app bot commented Nov 28, 2024

Your org has enabled the Graphite merge queue for merging into main

Add the label “merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

@vercel vercel bot temporarily deployed to Preview – wallet-ui November 28, 2024 13:10 Inactive
@vercel vercel bot temporarily deployed to Preview – thirdweb-www November 28, 2024 13:10 Inactive
Copy link
Contributor Author

kien-ngo commented Nov 28, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

@github-actions github-actions bot added the Playground Changes involving the Playground codebase. label Nov 28, 2024
@kien-ngo kien-ngo changed the title update Playground: Headless components Nov 28, 2024
@kien-ngo kien-ngo marked this pull request as ready for review November 28, 2024 13:11
Copy link
Contributor

github-actions bot commented Nov 28, 2024

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
thirdweb (esm) 44.93 KB (0%) 899 ms (0%) 399 ms (+13.29% 🔺) 1.3 s
thirdweb (cjs) 110.71 KB (0%) 2.3 s (0%) 1.2 s (+32.94% 🔺) 3.4 s
thirdweb (minimal + tree-shaking) 5.58 KB (0%) 112 ms (0%) 141 ms (+360.97% 🔺) 253 ms
thirdweb/chains (tree-shaking) 506 B (0%) 10 ms (0%) 19 ms (+119.94% 🔺) 29 ms
thirdweb/react (minimal + tree-shaking) 19.01 KB (0%) 381 ms (0%) 187 ms (+182.61% 🔺) 568 ms

@kien-ngo kien-ngo force-pushed the kien/headless-ui-playground branch from f0b150e to 1c90623 Compare November 28, 2024 13:17
@github-actions github-actions bot added packages SDK Involves changes to the thirdweb SDK labels Nov 28, 2024
Copy link

codecov bot commented Nov 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 51.23%. Comparing base (d35eaa6) to head (f25d1d8).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5535      +/-   ##
==========================================
+ Coverage   51.20%   51.23%   +0.03%     
==========================================
  Files        1093     1092       -1     
  Lines       57364    57346      -18     
  Branches     4677     4677              
==========================================
+ Hits        29374    29383       +9     
+ Misses      27274    27247      -27     
  Partials      716      716              
Flag Coverage Δ *Carryforward flag
legacy_packages 65.68% <ø> (ø) Carriedforward from d35eaa6
packages 47.85% <100.00%> (+0.03%) ⬆️

*This pull request uses carry forward flags. Click here to find out more.

Files with missing lines Coverage Δ
...rdweb/src/react/web/ui/prebuilt/Account/avatar.tsx 66.12% <100.00%> (+2.97%) ⬆️
...hirdweb/src/react/web/ui/prebuilt/Account/name.tsx 81.63% <100.00%> (+2.08%) ⬆️

... and 1 file with indirect coverage changes

@kien-ngo kien-ngo changed the base branch from kien/nft-components-resolver-fn to graphite-base/5535 December 9, 2024 11:39
@kien-ngo kien-ngo force-pushed the kien/headless-ui-playground branch from a8f76dc to 5823b80 Compare December 9, 2024 11:46
@kien-ngo kien-ngo changed the base branch from graphite-base/5535 to main December 9, 2024 11:46
@kien-ngo kien-ngo force-pushed the kien/headless-ui-playground branch from 5823b80 to 29f4630 Compare December 9, 2024 11:46
@kien-ngo kien-ngo force-pushed the kien/headless-ui-playground branch from 29f4630 to e1fd832 Compare December 9, 2024 15:11
Copy link
Member

@joaquim-verges joaquim-verges left a comment

Choose a reason for hiding this comment

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

looking good ser!

Copy link
Contributor

graphite-app bot commented Dec 10, 2024

Merge activity

  • Dec 10, 2:14 AM EST: A user added this pull request to the Graphite merge queue.
  • Dec 10, 2:22 AM EST: The Graphite merge queue couldn't merge this PR because it was not satisfying all requirements (Failed CI: 'Vercel – thirdweb-www', 'E2E Tests (npm, webpack)', 'E2E Tests (yarn, vite)', 'E2E Tests (yarn, webpack)', 'E2E Tests (yarn, esbuild)', 'E2E Tests (bun, webpack)').
  • Dec 10, 2:34 AM EST: A user added this pull request to the Graphite merge queue.
  • Dec 10, 2:34 AM EST: A user merged this pull request with the Graphite merge queue.

CNCT-2588

<!-- start pr-codex -->

---

## PR-Codex overview
This PR focuses on enhancing the `playground-web` application by adding new components and improving existing ones related to NFT and account functionalities, showcasing headless UI components for better Web3 integration.

### Detailed summary
- Added `ArticleCardIndex` in `page.tsx`.
- Introduced new navigation link for "Headless UI".
- Expanded `Account` and `NFT` components with detailed examples.
- Improved `useQuery` parameters in `avatar.tsx` and `name.tsx`.
- Created several new components in `nft-examples.tsx` and `account-examples.tsx` for displaying NFT and account information.

> ✨ Ask PR-Codex anything about this PR by commenting with `/codex {your question}`

<!-- end pr-codex -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages Playground Changes involving the Playground codebase. SDK Involves changes to the thirdweb SDK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants