Closed imanjra closed 2 months ago
This update introduces new components and updates existing ones across various packages to enhance icon handling and panel registration. Key additions include MuiIconFont
and IconButtonView
components, which integrate Material-UI icons and schema-based configurations. The update also streamlines panel registration with new parameters for icon variations and refines several type definitions.
Files | Change Summary |
---|---|
.../MuiIconFont/index.tsx , .../components/index.ts |
Introduced MuiIconFont component and exported it for broader use. |
.../SchemaIO/components/IconButtonView.tsx , .../SchemaIO/components/index.ts |
Added IconButtonView component and its export to handle icon buttons based on schema configurations. |
.../operators/src/OperatorIcon.tsx , .../operators/src/Panel/register.tsx , .../operators/src/built-in-operators.ts , .../operators/src/types.ts |
Enhanced icon handling in operator components, introduced registerPanel function, and updated type definitions. Removed unused constants. |
fiftyone/operators/operations.py , fiftyone/operators/panel.py , fiftyone/operators/types.py |
Updated register_panel function and PanelOperatorConfig class to support different icon modes. Introduced IconButtonView class in types. |
π°β¨ In the code garden, fresh sprouts arise, Icons bloom, and panels harmonize. A rabbit hops along the digital rows, Admiring the update's new composed prose. Cheers to changes, big and small, In our code world, standing tall! πΌπ
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
I think we need to be able to build these:
and these:
Sounds good. I will add another view ActionButtonView
with these two variants.
What changes are proposed in this pull request?
How is this patch tested? If it is not, please explain why.
Using example panels and operators
https://github.com/voxel51/fiftyone/assets/25350704/7817bc35-5ef9-4a50-9203-8969880141e3
Release Notes
Is this a user-facing change that should be mentioned in the release notes?
(Details in 1-2 sentences. You can just refer to another PR with a description if this PR is part of a larger change.)
What areas of FiftyOne does this PR affect?
fiftyone
Python library changesSummary by CodeRabbit
New Features
IconButtonView
class for rendering icon buttons based on schema configurations.dark_icon
andlight_icon
parameters to enhance icon display in different modes.PanelOperatorConfig
class to include icon parameters for customization.Enhancements
Bug Fixes
Button
class for uniformity.Refactor
registerPanel
function.Removals