Skip to content
This repository has been archived by the owner on Apr 9, 2021. It is now read-only.

feat: long press on filled forms navigate to ODK collect #344

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ajay-prabhakar
Copy link
Contributor

@ajay-prabhakar ajay-prabhakar commented Jan 22, 2020

Closes #343

What has been done to verify that this works as intended?

  • Checked whether collect launching as expected to a particular form by long pressing
  • Checked whether collect is launching as expected while reviewing the forms
  • Checked when we open the form in collect the selected forms are got unselected or not, they are working as expected
  • Checked the device by changing the orientation
  • Checked transferring forms working perfectly or not with this changes

Why is this the best possible solution? Were any other approaches considered?

I removed the existing method to launch collect and wrote new class LaunchCollect in that I gave the method openFormInCollect to launch collect as we are using the same method in both the classes and I used the dependency injection for the same

GIF

longpress

How does this change affect users?

If user wants to check the form before he starts transforming he has to go the collect and check the form by, adding this feature user can directly check/review the form by long pressing the particular form

Before submitting this PR, please make sure you have:

  • run ./gradlew checkCode and confirmed all checks still pass OR confirm CircleCI build passes
  • verified that any code or assets from external sources are properly credited in comments and/or in the about file.

@huangyz0918
Copy link
Contributor

Thanks for this new feature! @Chromicle . But everytime we introduce new features, you'd better send a post to ODK Forum and seek for feedback.

And for this feature, how can our user know long pressing can launch the Collect? If you are the product manager, how can you design and indicate your users about this?

@ajay-prabhakar
Copy link
Contributor Author

Thanks for this new feature! @Chromicle . But everytime we introduce new features, you'd better send a post to ODK Forum and seek for feedback.

As this is the small feature I feel that no need to post in the forum, I will do from the next time

And for this feature, how can our user know long pressing can launch the Collect? If you are the product manager, how can you design and indicate your users about this?

There are so many sharing platforms like ShareIt and MiDrop do the same so, I feel that user can know about this feature and we can include about this feature in about app so, user can know about this feature

@lakshyagupta21
Copy link
Contributor

As this is the small feature I feel that no need to post in the forum, I will do from the next time

@Chromicle I would suggest to not directly start working on any new feature until it is been discussed within the community. And I agree that every feature need not be discussed on forum until and unless it is impacting large set of users.

There are so many sharing platforms like ShareIt and MiDrop do the same so, I feel that user can know about this feature and we can include about this feature in about app so, user can know about this feature

I agree there are so many apps in market doing lot of things, but we should not do the same thing they're doing until it is really required. For taking inspiration it is completely fine but having each and every feature is not required I guess. And I agree with @huangyz0918 that features should be discussed first then we should go for the implementation. And I don't think including everything in About app would be good way to help user, Ideally no one really reads About app but we still keep it so that user can know what this app actually does, we can't include "how to and what to" do in the app in About app.

@ajay-prabhakar
Copy link
Contributor Author

ajay-prabhakar commented Jan 23, 2020

Yeah, @lakshyagupta21 @huangyz0918 from now I will discuss any new feature before I work on implementation I have made a thread on slack please give your opinions

@lakshyagupta21
Copy link
Contributor

Can we add a long press action on any item when list with checkbox is there. @Chromicle What do you think? Or if you read any article or some Google Material Design guidelines or anywhere else who recommended to implement a long press on list with checkbox, can you share it?

@ajay-prabhakar
Copy link
Contributor Author

Yeah, we can add the long-press action while using checkbox as if we long press on the list

  • Functionality of checkbox will not get any change
  • Selected forms will not get unchecked if we open any form
  • As we already used onclick functionality for selecting forms and I think the best approach for this is long press

I took google Gmail as a reference and I did, but in Gmail, we have star the mail and we can select mails with long press
mail

I got this idea to implement this feature if we have any better suggestions I will implement that

@lakshyagupta21
Copy link
Contributor

I took google Gmail as a reference and I did, but in Gmail, we have star the mail and we can select mails with long press

Star is used for individual items and its not grouping them into one so that it can do some operation on the selected star items, if we talk about the checkboxes then we ask user to select them and then we apply some operation on them as a group which results into something useful depending on the user-chosen option. If you're seeing it now, that when you long-press the item it starts creating a list for the checked and unchecked items so that users can at last select either to delete them or mark them as unread/read.

Copy link
Contributor

@huangyz0918 huangyz0918 left a comment

Choose a reason for hiding this comment

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

Please resolve conflicts @Chromicle , thanks.

@ajay-prabhakar
Copy link
Contributor Author

Star is used for individual items and its not grouping them into one so that it can do some operation on the selected star items

I understood I feel that long press will be the good one to open form
@lakshyagupta21 @huangyz0918 if you have any other better approach for this I will implement

for checkbox, this is one more reference, this app is shareIt(one file sharing platform)
they are following the same pattern with checkbox to choose file
shareit

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Navigate to collect when sending filled forms
3 participants