Skip to content

[react-native-web] Update accessibilityRole values and add userSelect style for RN 0.83#74673

Closed
war-in wants to merge 3 commits intoDefinitelyTyped:masterfrom
war-in:war-in/fix-types-RN-83
Closed

[react-native-web] Update accessibilityRole values and add userSelect style for RN 0.83#74673
war-in wants to merge 3 commits intoDefinitelyTyped:masterfrom
war-in:war-in/fix-types-RN-83

Conversation

@war-in
Copy link
Copy Markdown

@war-in war-in commented Mar 11, 2026

Please fill in this template.

If changing an existing definition:

@war-in war-in changed the title fix: adjust types to React Native 0.83 [react-native-web] Update accessibilityRole values and add userSelect style for RN 0.83 Mar 17, 2026
@war-in war-in changed the title [react-native-web] Update accessibilityRole values and add userSelect style for RN 0.83 [react-native-web] Update accessibilityRole values and add userSelect style for RN 0.83 Mar 17, 2026
@war-in war-in marked this pull request as ready for review March 17, 2026 11:06
@typescript-bot
Copy link
Copy Markdown
Contributor

typescript-bot commented Mar 17, 2026

@war-in Thank you for submitting this PR! I see this is your first time submitting to DefinitelyTyped 👋 — I'm the local bot who will help you through the process of getting things through.

This is a live comment that I will keep updated.

1 package in this PR

Code Reviews

Because you edited one package and updated the tests (👏), I can help you merge this PR once someone else signs off on it.

You can test the changes of this PR in the Playground.

Status

  • ✅ No merge conflicts
  • ✅ Continuous integration tests have passed
  • 🕐 Most recent commit is approved by type definition owners or DT maintainers

Once every item on this list is checked, I'll ask you for permission to merge and publish the changes.

Inactive

This PR has been inactive for 31 days — it is considered abandoned, and therefore closed!


Diagnostic Information: What the bot saw about this PR
{
  "type": "info",
  "now": "-",
  "pr_number": 74673,
  "author": "war-in",
  "headCommitOid": "72eda41ea0ecd40ac3fd0b93f1acf78460689c7c",
  "mergeBaseOid": "1e61bc5e872ac817652ae8625a8d21abe8ba94fc",
  "lastPushDate": "2026-03-11T15:14:47.000Z",
  "lastActivityDate": "2026-03-17T16:20:26.000Z",
  "hasMergeConflict": false,
  "isFirstContribution": true,
  "tooManyFiles": false,
  "hugeChange": false,
  "popularityLevel": "Popular",
  "pkgInfo": [
    {
      "name": "react-native-web",
      "kind": "edit",
      "files": [
        {
          "path": "types/react-native-web/index.d.ts",
          "kind": "definition"
        },
        {
          "path": "types/react-native-web/react-native-web-tests.tsx",
          "kind": "test"
        }
      ],
      "owners": [
        "fabioh8010",
        "blazejkustra"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Popular"
    }
  ],
  "reviews": [
    {
      "type": "changereq",
      "reviewer": "blazejkustra",
      "date": "2026-03-17T16:20:26.000Z"
    }
  ],
  "mainBotCommentID": 4074153334,
  "ciResult": "pass"
}

@typescript-bot typescript-bot added the Popular package This PR affects a popular package (as counted by NPM download counts). label Mar 17, 2026
@typescript-bot
Copy link
Copy Markdown
Contributor

🔔 @fabioh8010 @blazejkustra — please review this PR in the next few days. Be sure to explicitly select Approve or Request Changes in the GitHub UI so I know what's going on.

@typescript-bot typescript-bot moved this from Needs Author Action to Waiting for Code Reviews in Pull Request Status Board Mar 17, 2026
Copy link
Copy Markdown
Contributor

@blazejkustra blazejkustra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall looks good! Could you change the version of the package so it is compatible with the newest RNW release?

@typescript-bot typescript-bot added the Revision needed This PR needs code changes before it can be merged. label Mar 17, 2026
@typescript-bot typescript-bot moved this from Waiting for Code Reviews to Needs Author Action in Pull Request Status Board Mar 17, 2026
@typescript-bot
Copy link
Copy Markdown
Contributor

@war-in One or more reviewers has requested changes. Please address their comments. I'll be back once they sign off or you've pushed new commits. Thank you!

@typescript-bot typescript-bot added the Abandoned This PR had no activity for a long time, and is considered abandoned label Apr 9, 2026
@typescript-bot
Copy link
Copy Markdown
Contributor

@war-in I haven't seen any activity on this PR in more than three weeks, and it still has problems that prevent it from being merged. The PR will be closed on Apr 16th (in a week) if the issues aren't addressed.

@typescript-bot
Copy link
Copy Markdown
Contributor

@war-in To keep things tidy, we have to close PRs that aren't mergeable and don't have activity in the last month. No worries, though — please open a new PR if you'd like to continue with this change. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Abandoned This PR had no activity for a long time, and is considered abandoned Popular package This PR affects a popular package (as counted by NPM download counts). Revision needed This PR needs code changes before it can be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants