Skip to content

Commit ac20e1d

Browse files
Merge pull request #5707 from dfe-analytical-services/EES-5960-update-publication-modal-focus-close-fix
EES-5960 update publication modal focus close fix
2 parents d1712dc + 284b239 commit ac20e1d

File tree

2 files changed

+12
-5
lines changed

2 files changed

+12
-5
lines changed

src/explore-education-statistics-admin/src/pages/publication/components/PublicationDetailsForm.tsx

+11-2
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ import LoadingSpinner from '@common/components/LoadingSpinner';
1313
import useAsyncHandledRetry from '@common/hooks/useAsyncHandledRetry';
1414
import useToggle from '@common/hooks/useToggle';
1515
import Yup from '@common/validation/yup';
16-
import React, { useMemo } from 'react';
16+
import React, { useEffect, useMemo, useRef } from 'react';
1717
import { ObjectSchema } from 'yup';
1818
import PublicationUpdateConfirmModal from '@admin/pages/publication/components/PublicationUpdateConfirmModal';
1919

@@ -46,6 +46,13 @@ export default function PublicationDetailsForm({
4646
onSubmit,
4747
}: Props) {
4848
const [showConfirmModal, toggleConfirmModal] = useToggle(false);
49+
const submitButtonRef = useRef<HTMLButtonElement>(null);
50+
51+
useEffect(() => {
52+
if (showConfirmModal === false) {
53+
submitButtonRef.current?.focus();
54+
}
55+
}, [showConfirmModal]);
4956

5057
const { value, isLoading } = useAsyncHandledRetry(async () => {
5158
const themes = await themeService.getThemes();
@@ -158,7 +165,9 @@ export default function PublicationDetailsForm({
158165
)}
159166

160167
<ButtonGroup>
161-
<Button type="submit">Update publication details</Button>
168+
<Button type="submit" ref={submitButtonRef}>
169+
Update publication details
170+
</Button>
162171
<ButtonText onClick={onCancel}>Cancel</ButtonText>
163172
</ButtonGroup>
164173
</Form>

src/explore-education-statistics-admin/src/queries/releaseDataFileQueries.ts

+1-3
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
import { createQueryKeys } from '@lukemorales/query-key-factory';
2-
import releaseDataFileService, {
3-
DataFile,
4-
} from '@admin/services/releaseDataFileService';
2+
import releaseDataFileService from '@admin/services/releaseDataFileService';
53

64
const releaseDataFileQueries = createQueryKeys('releaseDataFile', {
75
list(releaseId: string) {

0 commit comments

Comments
 (0)