-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
WIP Copy and edit forms inside "Edit saved form" #886
Conversation
@yanokwa @lognaturel @nribeka @rafres What do you think? Maybe I have one another idea just let me now what you think about this current and let me rethink other possible solutions. |
This is looking really nice, @grzesiek2010! Sorry about the delay getting back to you on it. Here are a few things I thought about looking at it:
It would be helpful to get a quick summary of your other ideas for approaching this @grzesiek2010 to make sure we're not missing anything. I think the copy pasta 🍝 is inevitable since the whole point is to copy! |
@lognaturel
I tested this approach and first I thought it's better but I ran into some disadvantages like: so I decided to continue my first approach and here is the result. It's not tested well but you can look at it and let me know what you think.
I've decided to keep long-pressing. I don't have any better idea now and I think we can add something after: #464 then we can an icon for example
It's fixed now.
It's fixed as well. |
ERROR: This issue is in progress, but has no assignee. |
Heads up @grzesiek2010, we just merged some commits that conflict with the changes your made in this pull request! You can review this repository's recent commits to see where the conflicts occur. Please rebase your feature branch against the |
ERROR: This issue is in progress, but has no assignee. |
1 similar comment
ERROR: This issue is in progress, but has no assignee. |
@grzesiek2010 thanks for your hard work on this. This general concept of reusing values is a really powerful one and we want to make sure that we get the feature right. There was some conversation at https://forum.opendatakit.org/t/remembering-previously-entered-value-in-odk-collect/9116/13 and no clear decisions made. I hate to do this but I'm going to close the PR for now. Please keep the branch. I have also saved it both locally and on my remote. Hopefully we can fully specify the feature soon and bring this back. |
No description provided.