material-components/material-components-android

[BottomNavigationView] BottomNavigationView can have only one setOnNavigationItemSelectedListener

Open

#735 geöffnet am 5. Nov. 2019

Auf GitHub ansehen
 (3 Kommentare) (8 Reaktionen) (0 zugewiesene Personen)Java (3.023 Forks)batch import
Good First IssueWidget: BottomNavigationfeature request

Repository-Metriken

Stars
 (15.910 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

FEATURE REQUEST

As per title, BottomNavigationView can have only one setOnNavigationItemSelectedListener. This is annoying because if i use the bottomNavigationView.setupWithNavController(navController) this internally sets that listener, and i cannot use it anymore for anything else (like setting the text in a custom appBar, for example) A list of listeners would be the right thing in my opinion.

Ps. this probably applies to setOnNavigationItemReselectedListener(BottomNavigationView.OnNavigationItemReselectedListener listener) too.

For Pattern reference, see NavController addOnDestinationChangedListener and removeOnDestinationChangedListener

Contributor Guide

Research-Richtung
Das Issue fordert, mehrere Listener auf BottomNavigationView zuzulassen, anstatt nur einen. Der empfohlene Ansatz ist, dem Muster von NavControllers addOnDestinationChangedListener/removeOnDestinationChangedListener zu folgen, das eine Liste von Listenern verwaltet. Zur Implementierung muss die BottomNavigationView Klasse (wahrscheinlich im Bibliotheksquellcode) lokalisiert und so geändert werden, dass sie eine Liste von OnNavigationItemSelectedListener Instanzen speichert, über die iteriert wird, wenn ein Element ausgewählt wird. Um Abwärtskompatibilität zu gewährleisten, sollte die einzelne Setter Methode beibehalten, aber an die neue Liste angepasst werden. Verknüpfte PRs oder aktuelle Kommentare bieten keine zusätzliche Anleitung.
Tech Stack
javaandroid
Domain
mobile
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Android development
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70