-
-
Notifications
You must be signed in to change notification settings - Fork 25
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
Add Incomplete forms option to Root menu as Home Screen #2827
Add Incomplete forms option to Root menu as Home Screen #2827
Conversation
@@ -229,8 +229,7 @@ private static int getItemIcon(String id) { | |||
case SAVED_FORMS_ITEM_ID: | |||
return R.drawable.ic_saved_forms_nav_drawer; | |||
case INCOMPLETE_FORMS_ITEM_ID: | |||
//TODO: Need to add an icon for this option | |||
return -1; | |||
return R.drawable.ic_incomplete_nav_drawer; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we not use the existing home_incomplete.xml ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@shubham1g5 I considered that but it would distort the look and feel
, no? In that mode all, the icons are grey and the home_incomplete
is coloured and requires a background:
2258ed3
to
02b4e46
Compare
@shubham1g5 I changed the icon to a vector drawable and made the necessary changes to the layout for it to scale as needed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
Summary
This PR is just to add a new navigation drawer item for Incomplete forms, so users can easily access forms that have been saved as incomplete.
Feature Flag
This is related to the
Incomplete forms
feature and the custom propertycc-use-root-menu-as-home-screen
.Product Description
This is how the navigation drawer looks like with the addition:
Safety Assurance
Safety story
This just adds a menu item to an existing feature.