Post

I was exploring a hard to debug memory leak in react native for at least 6 hours today. Turns out that when you don't add accessibilityLabel to RectButton or Pressable, it will fucking traverse all children to find a Text node or View with accessibilityLabel. This is insane. 1/4

Is this an actual memory leak? From the code samples, it looks like it should pile up some memory but then let that go. I agree this isn't a good default, and perhaps a dev-only warning saying "need an accessibility label on <view name>" would be best.

Generally speaking, I don’t like these automated and expensive lookups. I’d rather have a dev-only warning that forces users to handle accessibility manually. And lookups should bail out fast and have some railguards.

FYI nested touchables are inaccessible as well.

Our FeedItem is actually a GestureDetector (handles play/pause and double-tap to like), and the RectButtons are placed on top of it—not even nested in the tree. We do have nested pressables, eg in lists where the whole item opens a sheet, but the avatar inside navigates to the profile instead.

But I think I get why this isn’t accessible for screen readers. I’m definitely lacking the skill and knowledge when it comes to this whole topic; the do’s and don’ts.