fiftyone icon indicating copy to clipboard operation
fiftyone copied to clipboard

fix non-editable views state issue and conditional row actions for TableView

Open imanjra opened this issue 4 months ago • 2 comments

What changes are proposed in this pull request?

fix non-editable views state issue and conditional row actions for TableView

How is this patch tested? If it is not, please explain why.

Using a python panel

Release Notes

Is this a user-facing change that should be mentioned in the release notes?

  • [x] No. You can skip the rest of this section.
  • [ ] Yes. Give a description of this change to be included in the release notes for FiftyOne users.

(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?

  • [ ] App: FiftyOne application changes
  • [ ] Build: Build and test infrastructure changes
  • [ ] Core: Core fiftyone Python library changes
  • [ ] Documentation: FiftyOne documentation changes
  • [ ] Other

Summary by CodeRabbit

  • New Features
    • Introduced a new utility function to enhance schema handling.
    • Added a constant to define non-editable view names.
    • New property added to customize the selected cell's background color in the table component.
  • Improvements
    • Refactored row actions in the TableView component for dynamic retrieval based on the current row.
    • Enhanced logic for state initialization in the DynamicIO component, ensuring it only initializes for editable schemas.
  • Style
    • Minor adjustments to the formatting and alignment of table cells.

imanjra avatar Oct 23 '24 12:10 imanjra