Open codepathreview opened 3 years ago
Hi,
I wasn’t able to include all the features in the GIF because the size of the GIF kept being too large. Is there anything I can do to send the video to you with all the features?
Santy Mendoza On Sat, Jun 26, 2021 at 5:22 AM CodePath Reviewers @.***> wrote:
Nice work! The purpose of this project was to continue learning Objective-C and the Cocoa Touch (iOS) framework. You should be comfortable creating table views (one of the most common views in iOS) and working with array of dictionaries created from JSON (in this case from the Movies Database API).
A key part of these projects is that you work to polish the visuals and the small UI / UX touches. Developing your design sense is an important part of being a mobile engineer. You'll find that perfecting the UI / UX will often lead to interesting technical challenges as well because the libraries may not behave exactly as you want them to, so you'll have to learn how to achieve the effect that you want. A few other things to note:
- Did you use groups to organize your source files? It's pretty common to organize your source files into Groups https://developer.apple.com/library/ios/recipes/xcode_help-structure_navigator/articles/Creating_a_Group.html, especially for larger projects. You can organize by type of class (i.e. View Controllers, Views, etc) or for larger apps by functionality (i.e. Home Screen, Search Screen, etc).
- Did you minimize the number of public methods and properties in your classes? You should always try to minimize the number of properties and methods that are in the header files. Only expose what you have to, e.g., properties and methods that must be accessed by other classes.
- Did you properly set the Content Mode for your ImageViews? By default the UIContentMode https://developer.apple.com/library/ios/documentation/UIKit/Reference/UIView_Class/#//apple_ref/c/tdef/UIViewContentMode will stretch the image to match the dimensions of the image view, which is probably not what you want. The most common combination is to use Aspect Fill combined with clipping subviews.
We have graded your work, and it looks like the following features are not reflected on your GIF walkthrough:
-User sees a loading state while waiting for the movies API. -User sees an error message when there's a networking error -User can tap a tab bar button to view a grid layout of Movie Posters using a CollectionView. -User can search for a movie.
The GIF helps us to make sure we don't miss any required or optional stories you have completed. We highly suggest that you provide us with a GIF that completely shows all the features you have implemented.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/santymendoza/Flix/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANEEMSD2C2COHAMBJ2Q63YLTUXA6TANCNFSM47LJU2UQ .
-- Santy Mendoza A.B. Candidate in Computer Science Harvard College | Class of 2023 Vice-President of Tech | Harvard Latinxs in Finance & Technology (LiFT)
Nice work! The purpose of this project was to continue learning Objective-C and the Cocoa Touch (iOS) framework. You should be comfortable creating table views (one of the most common views in iOS) and working with array of dictionaries created from JSON (in this case from the Movies Database API).
A key part of these projects is that you work to polish the visuals and the small UI / UX touches. Developing your design sense is an important part of being a mobile engineer. You'll find that perfecting the UI / UX will often lead to interesting technical challenges as well because the libraries may not behave exactly as you want them to, so you'll have to learn how to achieve the effect that you want. A few other things to note:
We have graded your work, and it looks like the following features are not reflected on your GIF walkthrough:
-User sees a loading state while waiting for the movies API. -User sees an error message when there's a networking error -User can tap a tab bar button to view a grid layout of Movie Posters using a CollectionView. -User can search for a movie.
The GIF helps us to make sure we don't miss any required or optional stories you have completed. We highly suggest that you provide us with a GIF that completely shows all the features you have implemented.