williamlian / sweetter

0 stars 0 forks source link

[Android Bootcamp] Project 3: Sweeter - Ready for Review #1

Open williamlian opened 8 years ago

williamlian commented 8 years ago

My app is complete, please review. /cc @codepathreview @codepath

Issue for this assignment: I want to dynamically set a max height of a dialog. The detail view dialog, when there are no media it is good to stay as the height to wrap the content, but when there are media it will stretch to the top and bottom of the screen which is not good looking. So basically I want to set the height of the dialog at runtime by checking if the measured height of the dialog is larger than a threshold, if so then set it to be fixed at the threshold. I searched around and tried the getDecorView(), getMeasureHeight() but all of the methods give me 0 as the runtime height of the dialog.

codepathreview commented 8 years ago

:+1: Excellent work. Great to see you put in the effort to turn in a polished assignment.. A few notes after checking out the code:

Here's a detailed Project 3 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 could improve your submission.

Let us know if you have any other thoughts or questions about this assignment. Hopefully by now you feel pretty comfortable with all the major pieces to basic Android apps (Views, Controllers, ActionBar, Navigation, Models, Authentication, API Communication, Persistence, et al) and see how they all fit together. We are close now to a turning point in the course where you should be hitting a "critical mass" towards your knowledge of Android.