Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

[Android] mapPadding is lost when the native map is recreated on screen re-attach (onMapReady re-fires; value-equal prop is never re-sent)

Offen
#5,948 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 2 Tagen

@huytdps13400 arbeitet bereits daran.

Seit 26.8.2026.

  • #5984 von @huytdps13400 — offen

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
52/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
android, react-native, typescript
Bereich
mobile

Rechercherichtung

Start at the Android native MapView handling for onMapReady and mapPadding, then trace how setPadding behaves when the native GoogleMap is recreated after screen re-attach under Fabric. Reproduce the tab-detach scenario and verify that the recreated map retains the current padding without requiring a value-changing workaround.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Summary

On Android (New Architecture / Fabric), when a MapView sits inside a react-navigation screen and that screen is detached and re-attached (e.g. the surrounding bottom-tab set changes while the user is on another screen), the native GoogleMap is recreated - onMapReady fires a second time for the same component instance. The mapPadding that was in effect is not re-applied to the recreated map: the value applied during the new view's mount is swallowed by the still-initializing map, and because the JS mapPadding prop is value-equal across renders, Fabric never re-sends it. The map is left with zero padding indefinitely - visibly, the native My Location button jumps to the raw top-right corner under the system status bar.

This looks like a residual case of the padding-application family fixed by #5151 (#5150 / #5152 / #5153): those fixes (contained in 1.28.2) cover the initial mount, but not a native map recreated after a screen re-attach.

Reproducible sample code
// Map screen registered inside a react-navigation bottom-tabs navigator
// whose tab set changes while this screen is blurred (a conditional tab).
import React from 'react';
import MapView, { PROVIDER_GOOGLE } from 'react-native-maps';

export default function MapScreen() {
  const [ready, setReady] = React.useState(false);
  return (
    <MapView
      style={{ flex: 1 }}
      provider={PROVIDER_GOOGLE}
      showsUserLocation
      showsMyLocationButton
      onMapReady={() => {
        console.log('onMapReady'); // fires AGAIN after the screen re-attaches
        setReady(true);
      }}
      mapPadding={
        ready ? { top: 120, right: 0, bottom: 0, left: 0 } : undefined
      }
      initialRegion={{
        latitude: 42.7,
        longitude: 25.3,
        latitudeDelta: 1,
        longitudeDelta: 1
      }}
    />
  );
}
Steps to reproduce
  1. Render the MapView above in a screen of a react-navigation bottom-tabs navigator, with the mapPadding top offset applied. The native My Location button renders below the offset (correct).
  2. Navigate away (push a screen over the tabs), and while the map screen is blurred change the tab set (add or remove a conditional tab), so the map screen goes through a detach/re-attach.
  3. Navigate back to the map screen.
  4. Watch the console: onMapReady fires a second time - the native map view was recreated. The My Location button now renders under the status bar; the previously applied mapPadding is gone and never comes back (the JS prop value did not change, so Fabric dedupes it and nothing reaches the native side).
Expected result

The recreated native map retains (or re-applies) the mapPadding currently set on the component - the My Location button stays below the padding offset.

Actual result

The recreated map has zero padding; the My Location button renders under the system status bar and stays there indefinitely.

Workaround we ship: nudge the padding value by 1px inside onMapReady, forcing a post-init prop update Fabric cannot dedupe. Note for anyone else affected: anchoring the nudge on a navigation focus event is NOT reliable - it races the native map recreation (we reproduced both orderings on different devices); onMapReady is the deterministic signal that the recreated map can accept setPadding.

React Native Maps Version

1.28.2

What platforms are you seeing the problem on?

Android

React Native Version

0.86.0

What version of Expo are you using?

Not using Expo

Device(s)

Pixel 9 emulator (Android 16) and physical Android device

Additional information
  • Reproduced on react-native-maps 1.28.2, React Native 0.86.0, New Architecture (Fabric), googleRenderer LATEST.
  • Not fixed by upgrading: the 1.29.0 changelog contains no padding/restore-related changes.
  • Related prior work already contained in 1.28.2: #5151 (fixes #5150/#5152/#5153, initial-mount padding), #5727 (state-restore double-initialization).
  • The user-location dot (showsUserLocation) survives the recreation, so the loss is specific to the padding state, not the whole prop set.
Vorherrschende Sprache
TypeScript
Sterne
16k
Forks
5k
Ø Merge
1 T. 4 Std.
Gemergte PRs (30 T.)
4

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus react-native-maps/react-native-maps

Alle Issues in react-native-maps/react-native-maps

Ähnliche Issues

Weitere Issues zu TypeScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.