Skip to content

Conversation

@divyanshu013
Copy link
Member

@divyanshu013 divyanshu013 commented Oct 29, 2025

Details

Adds a demo for building comments functionality using the UI customization (slots) API. Uses React and BaselineUI.

Note for review:

  • Main changes to review are in App.tsx and CommentThread.tsx
  • Other code is mostly generated from vite and follows our getting started guide for the setup

Demo

2025-11-07.at.17.30.06.-.Tan.Tern.mp4

Todo

  • Revert nightly version to stable
@divyanshu013 divyanshu013 self-assigned this Nov 7, 2025
@divyanshu013 divyanshu013 marked this pull request as ready for review November 7, 2025 12:02
@divyanshu013 divyanshu013 requested a review from a team November 7, 2025 12:02
Copy link
Contributor

@miguelcalderon miguelcalderon left a comment

Choose a reason for hiding this comment

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

Works great! Just shared some comments for your consideration.

I'm getting a Node update warning from Vite. Maybe we can bump the NodeJS version in .tool-versions a bit up, I think 22.12.0?

"@baseline-ui/core": "^0.45.8",
"@baseline-ui/icons": "^0.45.8",
"@baseline-ui/tokens": "^0.45.8",
"@nutrient-sdk/viewer": "1.9.0-nightly.20251106",
Copy link
Contributor

Choose a reason for hiding this comment

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

Can't this wait until 1.9.0? :D

Copy link
Member Author

Choose a reason for hiding this comment

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

Yup, definitely. Waiting for 1.9.0 to go out first.

Copy link
Member Author

Choose a reason for hiding this comment

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

Bumped to latest in bb1c082

});
}

cleanup = () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

By the time we get here, the initial cleanup function has already been returned. I don't think this would ever be called, would it?

Copy link
Member Author

Choose a reason for hiding this comment

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

I checked by unmounting App with a timer and it's being called. By the time cleanup of the effect runs the cleanup variable is updated to the function which runs unload on NutrientViewer.

@divyanshu013 divyanshu013 merged commit e14c26e into master Nov 17, 2025
2 checks passed
@divyanshu013 divyanshu013 deleted the div/comments-customization-next branch November 17, 2025 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants