Na ten moment zostawilam NavigationDrawer w spokoju, poniewaz Activities i
tak beda zmienione na fragmenty. Poprawilam uwagi, ktore byly do samej
listy zwierzat, i przenioslam kod odpowiedzialny za te liste do fragmentu.
Na razie jeszcze zostawilam AnimalListActivity, ale znajduje się w niej
jedynie Search i fragment.


W dniu 31 marca 2014 17:51 użytkownik maciej-malecki <
notifications@github.com> napisał:

> In src/com/blstream/myguide/SightseeingActivity.java:
>
> > @@ -386,6 +380,35 @@ public boolean onTouch(View view, MotionEvent motionEvent) {
> >  				return false;
> >  			}
> >  		});
> > +		
> > +		mDrawerList.setOnItemClickListener(new OnItemClickListener() {
> > +
> > +			@Override
> > +			public void onItemClick(AdapterView<?> parent, View view, int position,
> > +					long id) {
> > +				
> > +				Class<?> goToActivity = null;
>
> Zapisywanie typu activity nie jest najlepszym pomysłem. Myślę, że zamiast
> tego mogłabyś dodać metodę włączającą odpowiednie activity (docelowo pewnie
> fragment) której byłyby przekazywany Intent tworzony w case'ach.
> Jeśli chodzi o switch(position) to sugeruję zdefiniować stałe określające
> na którym miejscu jest który element menu, żeby nie było "case 0:" ale np.
> "case SIGHTSEEING_ITEM_POSITION:"
>
> Wydaje mi się też, że warto się zastanowić nad lekką modyfikacją działania
> naszego drawera w taki sposób, aby ustawiać tagi albo id na elementach
> wyświetlanych w drawerze (nie jest to wymóg, ale tylko poddaję to pod
> rozwagę). Wtedy tutaj w obiekcie view można odczytać wartość taga/id dla
> view i na tej podstawie podjąć akcję. Zaletą byłoby uniezależnienie akcji
> elementu od jego pozycji na drawerze - bo nie sprawdzalibyśmy position i na
> przyszłość wszelkie zmiany ux nie wymagałyby zmian kodu java.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/blstream/MyGuide/pull/60/files#r11121896>
> .
>