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

Low Fidelity Prototype for Home Page #548

Closed

Conversation

jtj9817
Copy link
Contributor

@jtj9817 jtj9817 commented Nov 1, 2018

Description

Prototype for the home page is made. It is still a low fidelity prototype and definitely needs changes as it doesn't use the CardView component at all and instead relied on ImageView and TextView components to design a layout. This prototype is related to the issue discussed at #511 .

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

  • ./gradlew assembleDebug assembleRelease
  • ./gradlew checkstyle

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes generate no new warnings

Screenshots

Prototype Screenshot

Starting point for project-travel-mate#534

Added the 'Offline Maps' section to the Navigation Drawer complete with the icon. 'fragment_offlinemaps.xml' and 'OfflineMapsFragment.java' also created.
Implemented the changes as requested by the discussion in project-travel-mate#545.
The created .xml file is temporary only and is meant for prototyping. Later PRs that are related to this issue should have at least a workable definition for it.
@Swati4star
Copy link
Member

Please add a screenshot also.

@@ -0,0 +1,2 @@
# Auto detect text files and perform LF normalization
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it required?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just included the .gitattributes file to fix the line endings formatting but forgot to add the file to the .gitignore.

@@ -0,0 +1,24 @@
<?xml version="1.0" encoding="UTF-8"?>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove .idea folder from commit

@@ -0,0 +1,2 @@
.idea
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we already have a gitignore file. Add these there only

@@ -233,7 +233,9 @@ private Fragment getFragmentByNavMenuItemId(int id) {
case R.id.nav_utility:
fragment = UtilitiesFragment.newInstance();
break;

//Work In Progress Offline Maps
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this change shouldnt be a part of this commit

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I was meaning to ignore this file change but it still got mixed in the commit.

@@ -1,7 +1,9 @@
<?xml version="1.0" encoding="utf-8"?>
<menu xmlns:android="http://schemas.android.com/apk/res/android">

<group android:checkableBehavior="single" android:id="@+id/nav_grp1">
<group
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these changes shouldnt be a part of your commit

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know why this file was changed in the commit when I didn't touch it.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

may be, because you created this branch from the branch that has the changes the related to offline maps. So, those were also included here.

@Swati4star
Copy link
Member

Can you add cardview to it? and make it a bit more colorful. This UI cant be reused.

@Swati4star
Copy link
Member

cant be used.

@Swati4star Swati4star closed this Nov 10, 2018
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.

2 participants