Skip to content

Conversation

flomillot
Copy link
Contributor

@flomillot flomillot commented Oct 10, 2025

  • Introduced support for sub-equipment management in filter-based contingency lists.
  • Added FilterBasedContingencyListVisualizationPanel for enhanced data visualization and management.
  • Extended data schemas, form handling, and UI interactions to accommodate new sub-equipment feature.

To fix :

  • empty space at right of the Dialog
  • responsive height on the whole Dialog, ie tables at the same level, scroll bar on tables \ dialog when adding description or lot of filters...

- Introduced support for sub-equipment management in filter-based contingency lists.
- Added `FilterBasedContingencyListVisualizationPanel` for enhanced data visualization and management.
- Extended data schemas, form handling, and UI interactions to accommodate new sub-equipment feature.

Signed-off-by: Florent MILLOT <[email protected]>
@flomillot flomillot requested a review from basseche October 10, 2025 16:59
@EtienneLt
Copy link
Contributor

/home/lesoteti/Projects/Gridsuite/gridexplore-app/src/components/dialogs/contingency-list/filter-based/FilterBasedContingencyListVisualizationPanel.tsx
22:18 warning Insert , prettier/prettier

Copy link
Contributor

@EtienneLt EtienneLt left a comment

Choose a reason for hiding this comment

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

minor things

…ters and type usage improvements

Signed-off-by: Florent MILLOT <[email protected]>
unscrollableFullHeight
disabledSave={Boolean(!!nameError || isValidating)}
isDataFetching={isFetching}
sx={{
Copy link
Contributor

Choose a reason for hiding this comment

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

height are width are not correct. Height should be 90vh and width: 50vw.
if not possible keep unscrollableFullHeight and we'll try to fix it in another PR

</Grid>
<Grid item xs={6}>
<TableContainer component={Paper} sx={{ height: '100%', border: 0.5 }}>
<Table>
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you can reduce lineHeight by using size="small"

flomillot and others added 7 commits October 14, 2025 17:32
…e and fix casing for file import paths

Signed-off-by: Florent MILLOT <[email protected]>
…fic types in contingency list filters

Signed-off-by: Florent MILLOT <[email protected]>
…zationPanel`

- Renamed `id` to `filterId` in `selectedEquipmentTypesByFilter` for better readability.
- Adjusted grid layout properties and added padding for improved UI alignment.
- Refactored variable names and method calls for enhanced code clarity.

Signed-off-by: Florent MILLOT <[email protected]>
…sistency in `ContingencyListUtils`.

Signed-off-by: Florent MILLOT <[email protected]>
- Added full-height styling to `Grid` in `ContingencyListFilterBasedForm`.
- Set dialog height to 95% of viewport in `ContingencyListFilterBasedDialog`.

Signed-off-by: Florent MILLOT <[email protected]>
- Adjusted `Grid` styling with a minimum width to address layout issues.

Signed-off-by: Florent MILLOT <[email protected]>
…VisualizationPanel`

- Updated condition to include `isFetching` check.

Signed-off-by: Florent MILLOT <[email protected]>
- Removed redundant `xs` properties and adjusted `Grid` structure to simplify layout and maintain consistency.

Signed-off-by: Florent MILLOT <[email protected]>
- Removed redundant `xs` properties and adjusted `Grid` structure to simplify layout and maintain consistency.

Signed-off-by: Florent MILLOT <[email protected]>
- Consolidated `loadingOverlayComponent` and `loadingOverlayComponentParams` into a single `overlay` object.
- Simplified and clarified conditional logic for `shouldDisplayRefreshButton`.

Signed-off-by: Florent MILLOT <[email protected]>
…stVisualizationPanel`

- Display a warning alert if study data has missing equipments.
- Update translations to include `missingEquipmentsFromStudy` entries in English and French.

Signed-off-by: Florent MILLOT <[email protected]>
resizable: false,
sortable: false,
};

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
export function RefreshOverlay({ loading, ...buttonProps }: Readonly<{ loading: boolean } & ButtonProps>) {
return loading ? <CircularProgress /> : <RefreshButton {...buttonProps} />;
}


const overlay = shouldDisplayRefreshButton
? {
loadingOverlayComponent: RefreshButton,
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
loadingOverlayComponent: RefreshButton,
loadingOverlayComponent: RefreshOverlay,

Comment on lines +170 to +171
loadingOverlayComponentParams: { disabled: true }, // disable the button when loading
// rendering another loadingOverlayComponent or overlayLoadingTemplate does not work
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
loadingOverlayComponentParams: { disabled: true }, // disable the button when loading
// rendering another loadingOverlayComponent or overlayLoadingTemplate does not work
loadingOverlayComponentParams: { loading: true },

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants