Skip to content

Commit 1bf5e60

Browse files
authored
Merge pull request #506 from Lemoncode/fixaccessibilitybug/expand-button-and-css-dev-canvas
Change the labels of the expand nested fields button and add styles f…
2 parents b2889e2 + 8c51e6a commit 1bf5e60

File tree

3 files changed

+5
-6
lines changed

3 files changed

+5
-6
lines changed

src/pods/canvas/components/canvas-accessible/canvas-accessible.pod.module.css

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,9 @@
55
/* only for see the content while developing */
66
/*
77
z-index: 1;
8-
background-color: rgba(0, 0, 0, 0.908); */
8+
background-color: rgba(0, 0, 0, 0.908);
9+
text-align: left;
10+
padding-left: 2rem; */
911
/* comment for show accessibility canvas*/
1012
width: 1px;
1113
height: 1px;

src/pods/canvas/components/canvas-accessible/canvas-accessible.pod.tsx

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,6 @@ export const CanvasAccessible: React.FC<CanvasAccessibleProps> = props => {
3333
}, []);
3434

3535
return (
36-
//Maybe it needs something for better accessibility.
37-
//Before, the screen reader read all the content and displayed
38-
// the levels of the titles and the list of items, now read all like plain text.
3936
<section
4037
className={classes.screenReaderOnly}
4138
tabIndex={-1}

src/pods/edit-table/components/field.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -134,8 +134,8 @@ export const Field: React.FC<Props> = props => {
134134
onClick={() => toggleExpand(field.id)}
135135
aria-label={
136136
expandedFields.has(field.id)
137-
? 'contract nested fields'
138-
: 'expand nested fields'
137+
? `collapse nested fields of "${field.name}"`
138+
: `expand nested fields of "${field.name}"`
139139
}
140140
>
141141
{expandedFields.has(field.id) ? (

0 commit comments

Comments
 (0)