shivagss / AndroidTwitterClient

A simple Twitter client that supports viewing a Twitter timeline and composing a new tweet.
0 stars 1 forks source link

Please review my app #2

Open shivagss opened 9 years ago

shivagss commented 9 years ago

Please review my app. @nesquena and @thecodepath

codepathreview commented 9 years ago

:+1: Decent work Shiva. A few notes after checking out the code:

Here's a detailed Project 4 Feedback Guide here which covers the most common issues with this submitted project. Read through the feedback guide point-by-point to determine how you might be able to improve your submission.

This week (Week 5), we are going to cover the last major piece to the Android puzzle and that is using the hardware and SDK components such as the camera, photo gallery, location, maps, etc. After that, Week 6 and week 7 we will be covering a few important intermediate topics such as more about styling and animation as well as testing.

Following the bootcamp, we are going to have a demo day to celebrate the progress you've all made with our next batch of Android students and multiple companies attending to see the group projects that you all have built. We are going to help however we can over the next few weeks to get the team project apps in shape for that.

shivagss commented 9 years ago

@nesquena @codepathreview I thought it was important to make the app look closely like Twitter app itself. The profile activity resembles very much like the Twitter app which shows the top 3 tweets and latest photos along with lists of Friends, Followers and Favorites. The assignment goal of re-using the fragment was achieved when you click "View all Tweets" from profile, which calls the newInstance() of the re-usable fragment via the DisplayTweetsListActivity. I originally implemented manually adding the tabs, but the ViewPager seems to be a better approach and once again its what the original Twitter app does.