feat: Add "option" to available role values (facebook#35137)

Summary: As pointed out by efoken on facebook#34424 (comment) we forgot we add the `option` value to the `role` prop, so this PR adds this missing value as requested on facebook#34424. ## Changelog [General] [Added] - Add "option" to available role values Pull Request resolved: facebook#35137 Test Plan: Ensure that CI is green as there isn't much to test in this case because we're just adding a value that maps to `undefined` Reviewed By: jacdebug Differential Revision: D40849497 Pulled By: NickGerleman fbshipit-source-id: 5e3e24c0ff05c361a7a8dc1ee1f20ba3fb6988ca
OlimpiaZurek · May 22, 2023 · 78cc67f · 78cc67f
1 parent f667883
commit 78cc67f
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 0 deletions.
diff --git a/Libraries/Components/View/ViewAccessibility.d.ts b/Libraries/Components/View/ViewAccessibility.d.ts
@@ -330,6 +330,7 @@ export type Role =
   | 'navigation'
   | 'none'
   | 'note'
+  | 'option'
   | 'presentation'
   | 'progressbar'
   | 'radio'

diff --git a/Libraries/Components/View/ViewAccessibility.js b/Libraries/Components/View/ViewAccessibility.js
@@ -85,6 +85,7 @@ export type Role =
   | 'navigation'
   | 'none'
   | 'note'
+  | 'option'
   | 'presentation'
   | 'progressbar'
   | 'radio'

diff --git a/Libraries/Utilities/AcessibilityMapping.js b/Libraries/Utilities/AcessibilityMapping.js
@@ -92,6 +92,8 @@ export function getAccessibilityRoleFromRole(role: Role): ?AccessibilityRole {
       return 'none';
     case 'note':
       return undefined;
+    case 'option':
+      return undefined;
     case 'presentation':
       return 'none';
     case 'progressbar':