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

Buttons are blocked a bit too long after e.g. going to the end page #6061

Closed
dbemke opened this issue Mar 29, 2024 · 4 comments · Fixed by #6098
Closed

Buttons are blocked a bit too long after e.g. going to the end page #6061

dbemke opened this issue Mar 29, 2024 · 4 comments · Fixed by #6098
Assignees
Milestone

Comments

@dbemke
Copy link

dbemke commented Mar 29, 2024

ODK Collect version

the master version 031183e

Android version

10, 14

Device used

Redmi 9T, Pixel 7a

Problem description

Buttons are blocked a bit too long after e.g. going to the end page in a form and tapping a button quickly. If a user taps the button at the end page quickly (after coming to the end page), it's necessary to tap the button twice.
The issue doesn't occur in the store version.

XRecorder_29032024_135912.mp4

Steps to reproduce the problem

  1. Open a form.
  2. Go to the hierarchy view.
  3. Tap "Go to end”.
  4. Tap "save as draft”/ "send” button quickly.

Expected behavior

The user shouldn't notice the time when the buttons are blocked.

@dbemke
Copy link
Author

dbemke commented Mar 29, 2024

Another way to reproduce it:

  1. Save some drafts or send forms.
  2. Go to "Delete form - Saved Forms".
  3. Tap "Select all" and quickly after that "Delete Selected".

@srujner
Copy link

srujner commented Mar 29, 2024

I also found some other way to reproduce it in a different way:

  1. Have at least one form in Ready to Send tab;
  2. Quickly click on the Select All button and then Send Selected button;

The Send Selected buttons needs 2-3 taps to become active and send selected forms

@seadowg
Copy link
Member

seadowg commented Apr 10, 2024

@grzesiek2010 I'm thinking that a solve for this is to just ditch the javaClass.name default for the screenName param (for MultiClickGuard#allowClick) and just pass in a screen name constant (but make screenName optional). This would let us target the cases the places where users are more likely to quick through quickly (I ran into the form end screen one a couple of times today for example).

@seadowg
Copy link
Member

seadowg commented Apr 16, 2024

We chatted about this in a meeting and it sounds like we're in agreement to fix the cases mentioned in the issue as well as allowing faster clicks for the "next" and "back" button in form entry. For some of these, the screenName approached above will be the most appropriate, but for others it might just make sense to opt out of the MultiClickButton approach.

@seadowg seadowg added this to the v2024.2 milestone Apr 16, 2024
@seadowg seadowg moved this from not ready to ready in ODK Collect Apr 16, 2024
@grzesiek2010 grzesiek2010 moved this from ready to in progress in ODK Collect Apr 18, 2024
@grzesiek2010 grzesiek2010 self-assigned this Apr 18, 2024
@github-project-automation github-project-automation bot moved this from in progress to done in ODK Collect Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: done
Development

Successfully merging a pull request may close this issue.

4 participants