Christian Byrne
|
2d22c9f7f0
|
Update diffusion_models display to 'Diffusion' in asset browser (#6533)
## Summary
- Update the display label for `diffusion_models` category from
"Diffusion Models" to "Diffusion" in the asset browser/gallery UI
- Added special case handling in `formatCategoryLabel` function for
cleaner display
- This is a display-only change that does not affect underlying data
structures or API calls
## Test plan
- [x] Unit tests pass
- [x] Linter passes
- [x] Formatter passes
- [x] Type checking passes
🤖 Generated with [Claude Code](https://claude.com/claude-code)
┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-6533-Update-diffusion_models-display-to-Diffusion-in-asset-browser-29f6d73d365081009fa5cb396861570b)
by [Unito](https://www.unito.io)
Co-authored-by: Christian Byrne <chrbyrne96@gmail.com>
Co-authored-by: Claude <noreply@anthropic.com>
|
2025-11-01 23:26:02 -07:00 |
|
Christian Byrne
|
8120ed9dfa
|
load assets browser before fetch completes and show loading state (#6189)
## Summary
Moves the fetch and post-fetch logic associated with the asset browser
into the component and shows a loading state while fetching.
To test, use this branch:
https://github.com/comfyanonymous/ComfyUI/pull/10045
https://github.com/user-attachments/assets/718974d5-efc7-46a0-bcd6-e82596d4c389
┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-6189-load-assets-browser-before-fetch-completes-and-show-loading-state-2946d73d365081879d1bd05d86e8c036)
by [Unito](https://www.unito.io)
---------
Co-authored-by: GitHub Action <action@github.com>
|
2025-10-23 13:43:34 -07:00 |
|
Alexander Brown
|
874ef3ba0c
|
Lint: Add eslint import plugin (#5955)
## Summary
Adds the linter, turns on the recommended and a few extra rules, fixes
existing violations.
Doesn't prohibit `../../...` imports yet, that'll be it's own PR.
## Changes
- **What**: Consistent and fixable imports
- **Dependencies**: The plugin and parser
## Review Focus
How do you feel about the recommended rules?
What about the extra ones?
[Any
more](https://github.com/un-ts/eslint-plugin-import-x?tab=readme-ov-file#rules)
you'd want to turn on?
┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-5955-Lint-Add-eslint-import-plugin-2856d73d3650819985c0fb9ca3fa94b0)
by [Unito](https://www.unito.io)
|
2025-10-07 20:31:00 -07:00 |
|
Arjan Singh
|
abf2b3b980
|
Full Asset Selection Experience (Assets API) (#5900)
## Summary
Full Integration of Asset Browsing and Selection when Assets API is
enabled.
## Changes
1. Replace Model Left Side Tab with experience
2. Configurable titles for the Asset Browser Modal
3. Refactors to simplify callback code
4. Refactor to make modal filters reactive (they change their values
based on assets displayed)
5. Add `browse()` mode with ability to create node directly from the
Asset Browser Modal (in `browse()` mode)
## Screenshots
Demo of many different types of Nodes getting configured by the Modal
https://github.com/user-attachments/assets/34f9c964-cdf2-4c5d-86a9-a8e7126a7de9
┆Issue is synchronized with this [Notion
page](https://www.notion.so/PR-5900-Feat-asset-selection-cloud-integration-2816d73d365081ccb4aeecdc14b0e5d3)
by [Unito](https://www.unito.io)
|
2025-10-03 20:34:59 -07:00 |
|
Arjan Singh
|
dfa1cbba4f
|
Asset Browser Modal Component (#5607)
* [ci] ignore playwright mcp directory
* [feat] add AssetBrowserModal
And all related sub components
* [feat] reactive filter functions
* [ci] clean up storybook config
* [feat] add sematic AssetCard
* [fix] i love lucide
* [fix] AssetCard layout issues
* [fix] add AssetBadge type
* [fix] simplify useAssetBrowser
* [fix] modal layout
* [fix] simplify useAssetBrowserDialog
* [fix] add tailwind back to storybook
* [fix] better reponsive layout
* [fix] missed i18n string
* [fix] missing i18n translations
* [fix] remove erroneous prevent on keyboard.space
* [feat] add asset metadata validation utilities
* [fix] remove erroneous test code
* [fix] remove forced min and max width on AssetCard
* [fix] import statement nits
|
2025-09-17 16:17:09 -07:00 |
|