Skip to content

Commit e7241b4

Browse files
antonisantonis
authored
Merge 85d4f9e into 4d4400c 
2 parents 4d4400c + 85d4f9e commit e7241b4

File tree

4 files changed

+32
-14
lines changed

4 files changed

+32
-14
lines changed

‎CHANGELOG.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
### Fixes
1818

1919
- Avoid silent failure when JS bundle was not created due to Sentry Xcode scripts failure ([#4690](https://github.com/getsentry/sentry-react-native/pull/4690))
20+
- Fixes Feedback Widget accessibility issue on iOS ([#4739](https://github.com/getsentry/sentry-react-native/pull/4739))
2021

2122
### Dependencies
2223

‎packages/core/src/js/feedback/FeedbackWidget.tsx‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -262,7 +262,11 @@ export class FeedbackWidget extends React.Component<FeedbackWidgetProps, Feedbac
262262
}
263263

264264
return (
265-
<TouchableWithoutFeedback onPress={notWeb() ? Keyboard.dismiss: undefined}>
265+
<TouchableWithoutFeedback
266+
onPress={notWeb() ? Keyboard.dismiss : undefined}
267+
accessible={false}
268+
accessibilityElementsHidden={false}
269+
>
266270
<View style={styles.container}>
267271
<View style={styles.titleContainer}>
268272
<Text style={styles.title}>{text.formTitle}</Text>

‎packages/core/test/feedback/__snapshots__/FeedbackWidget.test.tsx.snap‎

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
exports[`FeedbackWidget matches the snapshot with custom styles 1`] = `
44
<View
5+
accessibilityElementsHidden={false}
56
accessibilityState={
67
{
78
"busy": undefined,
@@ -11,7 +12,7 @@ exports[`FeedbackWidget matches the snapshot with custom styles 1`] = `
1112
"selected": undefined,
1213
}
1314
}
14-
accessible={true}
15+
accessible={false}
1516
focusable={true}
1617
onClick={[Function]}
1718
onResponderGrant={[Function]}
@@ -235,6 +236,7 @@ exports[`FeedbackWidget matches the snapshot with custom styles 1`] = `
235236

236237
exports[`FeedbackWidget matches the snapshot with custom styles and screenshot button 1`] = `
237238
<View
239+
accessibilityElementsHidden={false}
238240
accessibilityState={
239241
{
240242
"busy": undefined,
@@ -244,7 +246,7 @@ exports[`FeedbackWidget matches the snapshot with custom styles and screenshot b
244246
"selected": undefined,
245247
}
246248
}
247-
accessible={true}
249+
accessible={false}
248250
focusable={true}
249251
onClick={[Function]}
250252
onResponderGrant={[Function]}
@@ -525,6 +527,7 @@ exports[`FeedbackWidget matches the snapshot with custom styles and screenshot b
525527

526528
exports[`FeedbackWidget matches the snapshot with custom texts 1`] = `
527529
<View
530+
accessibilityElementsHidden={false}
528531
accessibilityState={
529532
{
530533
"busy": undefined,
@@ -534,7 +537,7 @@ exports[`FeedbackWidget matches the snapshot with custom texts 1`] = `
534537
"selected": undefined,
535538
}
536539
}
537-
accessible={true}
540+
accessible={false}
538541
focusable={true}
539542
onClick={[Function]}
540543
onResponderGrant={[Function]}
@@ -789,6 +792,7 @@ exports[`FeedbackWidget matches the snapshot with custom texts 1`] = `
789792

790793
exports[`FeedbackWidget matches the snapshot with custom texts and screenshot button 1`] = `
791794
<View
795+
accessibilityElementsHidden={false}
792796
accessibilityState={
793797
{
794798
"busy": undefined,
@@ -798,7 +802,7 @@ exports[`FeedbackWidget matches the snapshot with custom texts and screenshot bu
798802
"selected": undefined,
799803
}
800804
}
801-
accessible={true}
805+
accessible={false}
802806
focusable={true}
803807
onClick={[Function]}
804808
onResponderGrant={[Function]}
@@ -1116,6 +1120,7 @@ exports[`FeedbackWidget matches the snapshot with custom texts and screenshot bu
11161120

11171121
exports[`FeedbackWidget matches the snapshot with default configuration 1`] = `
11181122
<View
1123+
accessibilityElementsHidden={false}
11191124
accessibilityState={
11201125
{
11211126
"busy": undefined,
@@ -1125,7 +1130,7 @@ exports[`FeedbackWidget matches the snapshot with default configuration 1`] = `
11251130
"selected": undefined,
11261131
}
11271132
}
1128-
accessible={true}
1133+
accessible={false}
11291134
focusable={true}
11301135
onClick={[Function]}
11311136
onResponderGrant={[Function]}
@@ -1380,6 +1385,7 @@ exports[`FeedbackWidget matches the snapshot with default configuration 1`] = `
13801385

13811386
exports[`FeedbackWidget matches the snapshot with default configuration and screenshot button 1`] = `
13821387
<View
1388+
accessibilityElementsHidden={false}
13831389
accessibilityState={
13841390
{
13851391
"busy": undefined,
@@ -1389,7 +1395,7 @@ exports[`FeedbackWidget matches the snapshot with default configuration and scre
13891395
"selected": undefined,
13901396
}
13911397
}
1392-
accessible={true}
1398+
accessible={false}
13931399
focusable={true}
13941400
onClick={[Function]}
13951401
onResponderGrant={[Function]}

‎packages/core/test/feedback/__snapshots__/FeedbackWidgetManager.test.tsx.snap‎

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -702,6 +702,7 @@ exports[`FeedbackButtonManager the Feedback Widget matches the snapshot with cus
702702
>
703703
<View>
704704
<View
705+
accessibilityElementsHidden={false}
705706
accessibilityState={
706707
{
707708
"busy": undefined,
@@ -711,7 +712,7 @@ exports[`FeedbackButtonManager the Feedback Widget matches the snapshot with cus
711712
"selected": undefined,
712713
}
713714
}
714-
accessible={true}
715+
accessible={false}
715716
focusable={true}
716717
onClick={[Function]}
717718
onResponderGrant={[Function]}
@@ -1049,6 +1050,7 @@ exports[`FeedbackButtonManager the Feedback Widget matches the snapshot with cus
10491050
>
10501051
<View>
10511052
<View
1053+
accessibilityElementsHidden={false}
10521054
accessibilityState={
10531055
{
10541056
"busy": undefined,
@@ -1058,7 +1060,7 @@ exports[`FeedbackButtonManager the Feedback Widget matches the snapshot with cus
10581060
"selected": undefined,
10591061
}
10601062
}
1061-
accessible={true}
1063+
accessible={false}
10621064
focusable={true}
10631065
onClick={[Function]}
10641066
onResponderGrant={[Function]}
@@ -1396,6 +1398,7 @@ exports[`FeedbackButtonManager the Feedback Widget matches the snapshot with def
13961398
>
13971399
<View>
13981400
<View
1401+
accessibilityElementsHidden={false}
13991402
accessibilityState={
14001403
{
14011404
"busy": undefined,
@@ -1405,7 +1408,7 @@ exports[`FeedbackButtonManager the Feedback Widget matches the snapshot with def
14051408
"selected": undefined,
14061409
}
14071410
}
1408-
accessible={true}
1411+
accessible={false}
14091412
focusable={true}
14101413
onClick={[Function]}
14111414
onResponderGrant={[Function]}
@@ -1743,6 +1746,7 @@ exports[`FeedbackButtonManager the Feedback Widget matches the snapshot with def
17431746
>
17441747
<View>
17451748
<View
1749+
accessibilityElementsHidden={false}
17461750
accessibilityState={
17471751
{
17481752
"busy": undefined,
@@ -1752,7 +1756,7 @@ exports[`FeedbackButtonManager the Feedback Widget matches the snapshot with def
17521756
"selected": undefined,
17531757
}
17541758
}
1755-
accessible={true}
1759+
accessible={false}
17561760
focusable={true}
17571761
onClick={[Function]}
17581762
onResponderGrant={[Function]}
@@ -2090,6 +2094,7 @@ exports[`FeedbackButtonManager the Feedback Widget matches the snapshot with def
20902094
>
20912095
<View>
20922096
<View
2097+
accessibilityElementsHidden={false}
20932098
accessibilityState={
20942099
{
20952100
"busy": undefined,
@@ -2099,7 +2104,7 @@ exports[`FeedbackButtonManager the Feedback Widget matches the snapshot with def
20992104
"selected": undefined,
21002105
}
21012106
}
2102-
accessible={true}
2107+
accessible={false}
21032108
focusable={true}
21042109
onClick={[Function]}
21052110
onResponderGrant={[Function]}
@@ -2437,6 +2442,7 @@ exports[`FeedbackButtonManager the Feedback Widget matches the snapshot with sys
24372442
>
24382443
<View>
24392444
<View
2445+
accessibilityElementsHidden={false}
24402446
accessibilityState={
24412447
{
24422448
"busy": undefined,
@@ -2446,7 +2452,7 @@ exports[`FeedbackButtonManager the Feedback Widget matches the snapshot with sys
24462452
"selected": undefined,
24472453
}
24482454
}
2449-
accessible={true}
2455+
accessible={false}
24502456
focusable={true}
24512457
onClick={[Function]}
24522458
onResponderGrant={[Function]}
@@ -2784,6 +2790,7 @@ exports[`FeedbackButtonManager the Feedback Widget matches the snapshot with sys
27842790
>
27852791
<View>
27862792
<View
2793+
accessibilityElementsHidden={false}
27872794
accessibilityState={
27882795
{
27892796
"busy": undefined,
@@ -2793,7 +2800,7 @@ exports[`FeedbackButtonManager the Feedback Widget matches the snapshot with sys
27932800
"selected": undefined,
27942801
}
27952802
}
2796-
accessible={true}
2803+
accessible={false}
27972804
focusable={true}
27982805
onClick={[Function]}
27992806
onResponderGrant={[Function]}

0 commit comments

Comments
 (0)