Skip to content

Commit

Permalink
Préavis - Les lignes de détail des préavis doivent se refermer en cas…
Browse files Browse the repository at this point in the history
… de tri ou de filtrage du tableau (#3533)

## Linked issues

- Resolve #3385 

----

- [ ] Tests E2E (Cypress)
  • Loading branch information
ivangabriele authored Aug 14, 2024
2 parents 206cb9d + 030fa9b commit 58aacf1
Showing 1 changed file with 9 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -15,12 +15,12 @@ import { useLoadingState } from '@hooks/useLoadingState'
import { useMainAppDispatch } from '@hooks/useMainAppDispatch'
import { useMainAppSelector } from '@hooks/useMainAppSelector'
import { DisplayedErrorKey } from '@libs/DisplayedError/constants'
import { Accent, Button, Icon, Size, TableWithSelectableRows } from '@mtes-mct/monitor-ui'
import { Accent, Button, Icon, Size, TableWithSelectableRows, usePrevious } from '@mtes-mct/monitor-ui'
import { skipToken } from '@reduxjs/toolkit/query'
import { flexRender, getCoreRowModel, getExpandedRowModel, useReactTable } from '@tanstack/react-table'
import { isLegacyFirefox } from '@utils/isLegacyFirefox'
import { useIsSuperUser } from 'auth/hooks/useIsSuperUser'
import { useCallback, useState } from 'react'
import { useCallback, useEffect, useState } from 'react'
import styled, { css } from 'styled-components'

import { getTableColumns } from './columns'
Expand Down Expand Up @@ -92,6 +92,7 @@ export function PriorNotificationList({ isFromUrl }: PriorNotificationListProps)
const loadingState = useLoadingState(isFetching, { apiSortingParams, listFilter }, apiPaginationParams)
const isBodyLoaderVisible = loadingState.isLoadingNewPage || (loadingState.isReloading && !!error)
const isBodyEmptyDataVisible = !isBodyLoaderVisible && !!priorNotifications && priorNotifications.length === 0
const previousListFilter = usePrevious(listFilter)
const title = getTitle(listFilter.seafrontGroup)

const handleSubMenuChange = useCallback(
Expand Down Expand Up @@ -137,6 +138,12 @@ export function PriorNotificationList({ isFromUrl }: PriorNotificationListProps)

const { rows } = table.getRowModel()

useEffect(() => {
if (previousListFilter !== listFilter) {
table.resetExpanded()
}
}, [previousListFilter, listFilter, table])

return (
<>
<SubMenu
Expand Down

0 comments on commit 58aacf1

Please sign in to comment.