Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Material #55

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Material #55

wants to merge 3 commits into from

Conversation

naishadhparmar
Copy link

Applied Material Design to many layout elements. Removed background pictures from Spinners, EditText and Button in MainActivity. Clicking the Button elevates it slightly (material). In SearchResultActivity, changed the font to the default Roboto and applied Android defined styles to headers and other information. Clicking on a card animates it into the DetailsActivity which gives a kind of flow of information (material). In DetailsActivity, changed the layout to make it look better, removed the Facebook button and removed the card slider. In AboutAppActivity, arranged the information in CardViews with text buttons which allow users to directly contact the developers. Reduced the SplashActivity wait time, as 1000ms may get annoying for users. Implemented the Lazy Singleton design pattern in MappingUtils. This optimizes in terms of memory. Optimized a few lines of code here and there as well.

Removed the background pictures of the MainActivity's EditText and Spinners. Slimmed the button down a bit. This became material. In AboutAppActivity, displaying the data in card format and exposed buttons to visit the links instead of just displaying the links.
…ing so many maps multiple times is a memory-heavy operation. Changed the SearchResultActivity to Material design
List of students displayed in CardView form. Clicking on a card animates the student's photo into the DetailActivity which gives a kind of flow to the information (follows Material Design). Removed card slider. Removed Facebook button.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant