Skip to content

Commit 1b64953

Browse files
committed
Fixed broken aria label
1 parent 518b8fb commit 1b64953

1 file changed

Lines changed: 11 additions & 11 deletions

File tree

src/groups/components/GroupSessions.tsx

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
1-
import React, { useCallback } from "react";
2-
import { ApiHelper, ArrayHelper, GroupInterface, DisplayBox, SessionInterface, VisitSessionInterface, PersonInterface, PersonHelper, VisitInterface, UserHelper, ExportLink, Permissions, Loading, SmallButton, Locale } from "@churchapps/apphelper";
1+
import React, { useCallback, type JSX } from "react";
2+
import { ApiHelper, ArrayHelper, type GroupInterface, DisplayBox, type SessionInterface, type VisitSessionInterface, type PersonInterface, PersonHelper, type VisitInterface, UserHelper, ExportLink, Permissions, Loading, SmallButton, Locale } from "@churchapps/apphelper";
33
import { Table, TableBody, TableRow, TableCell, TableHead, Icon, FormControl, InputLabel, Select, Button, Grid, MenuItem, type SelectChangeEvent } from "@mui/material"
44

55
interface Props {
@@ -62,8 +62,8 @@ export const GroupSessions: React.FC<Props> = (props) => {
6262
for (let i = 0; i < visitSessions.length; i++) {
6363
const vs = visitSessions[i];
6464
//let editLink = (canEdit) ? (<a href="about:blank" onClick={handleRemove} className="text-danger" data-personid={vs.visit.personId}><Icon>person_remove</Icon> Remove</a>) : null;
65-
const editLink = (canEdit) ? <SmallButton icon="person_remove" text="Remove" onClick={() => handleRemove(vs)} color="error" data-testid={`remove-session-visitor-button-${vs.id}`} ariaLabel={`Remove ${person?.name?.display || 'visitor'} from session`} /> : <></>
6665
const person = ArrayHelper.getOne(people, "id", vs.visit.personId);
66+
const editLink = (canEdit) ? <SmallButton icon="person_remove" text="Remove" onClick={() => handleRemove(vs)} color="error" data-testid={`remove-session-visitor-button-${vs.id}`} ariaLabel={`Remove ${person?.name?.display || 'visitor'} from session`} /> : <></>
6767
if (person) {
6868
result.push(
6969
<TableRow key={vs.id}>
@@ -121,17 +121,17 @@ export const GroupSessions: React.FC<Props> = (props) => {
121121
}
122122
}, [addedPerson?.id, session?.id, loadAttendance, addedCallback]);
123123

124-
React.useEffect(() => {
125-
if (group.id !== undefined) {
126-
loadSessions();
127-
addedCallback("");
124+
React.useEffect(() => {
125+
if (group.id !== undefined) {
126+
loadSessions();
127+
addedCallback("");
128128
}
129129
}, [group.id, addedSession?.id, addedCallback, loadSessions]);
130130

131-
React.useEffect(() => {
132-
if (addedPerson?.id !== undefined) {
133-
handlePersonAdd();
134-
}
131+
React.useEffect(() => {
132+
if (addedPerson?.id !== undefined) {
133+
handlePersonAdd();
134+
}
135135
}, [addedPerson?.id, handlePersonAdd]);
136136

137137
React.useEffect(() => { handleSessionSelected(); }, [handleSessionSelected]);

0 commit comments

Comments
 (0)