Closed loganSims closed 8 years ago
Fantastic update and documentation! Nice job. I'll pull it down shortly and take a look. If everything looks good we can stage it for a Monday morning deployment.
Here's what I've observed:
Could you explain your comment about the "Day of Week" drop down being moved back into the AppBar? Do you want it in the AppBar similar to how the twitter activity has it?
Yes, however, I'm not sure if that's possible. Under the old code you could only set one type of navigation mode for the action bar. It could either be tabs or a drop down list, but not both. For example
getSupportActionBar().setNavigationMode(ActionBar.NAVIGATION_MODE_TABS);
or
getSupportActionBar().setNavigationMode(ActionBar.NAVIGATION_MODE_LIST);
Not sure if that is still the case with the new code.
Sounds good. I don't think it'll be a problem. I'd like to think we switched to the toolbar to pull off stuff like that. I'll look into it once I'm finished with the new favorites fragment.
Let me know what you think about the toolbar in the favorites. It has a few weird behaviors. The most noticeable being:
EDIT Here's a link to the library I pulled in to get the toolbar collapsing in favorites https://github.com/ksoichiro/Android-ObservableScrollView
Basically it turns the favorites fragment into a scroll listener, which tells to toolbar to collapse at the appropriate times.
I'll see if I can pull this down today and take a look at it.
I just noticed something weird. I added a spinner to the activity_with_tabs.xml file for the Day of the Week drop down in the ferries activity. An empty spinner now shows up in all other activities that use that layout. I will either set spinner visibility to GONE or make separate layout for the ferries activity. Should be an easy fix.
Awesome job!
Changes
Updated most list views throughout app to recycler views.
The recycler view is a slimed down more powerful version of the list view. The most noticeable change to the app with this update is the new material look. Using a recycler view allows for fun toolbar interactions with our lists. (things like scrolling toolbars and collapsing toolbars with banners) Some readings for the recycler view:
The take away is the recycler view offers us more flexibility with our lists.
Removed deprecated tab calls in activities with tabs.
We now use the TabLayout class. Some readings on that:
Use the CursorRecyclerAdapter Class
As of this update there isn't a standard way to use recycler views with cursors. I pulled in a class that works like a wrapper for a cursor adapter and recycler view adapter. This way we can use the recycler view will data pulled from our Databases.
Other changes