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

[FEATURE] Protection from accidental editing #99

Closed
opk12 opened this issue Sep 7, 2023 · 3 comments · Fixed by #120
Closed

[FEATURE] Protection from accidental editing #99

opk12 opened this issue Sep 7, 2023 · 3 comments · Fixed by #120
Assignees
Labels
enhancement New feature or request

Comments

@opk12
Copy link

opk12 commented Sep 7, 2023

Is your feature request related to a problem? Please describe.
What about a protection in the table view from accidental modification of the wrong column? I'm particularly afraid when scrolling - the finger is on the old columns, and the phone screen is short enough that the finger must attach and detach from the screen multiple times. So a spurious movement could be seen as a tap.

Describe the solution you'd like
Throwing ideas up in the air: a Save button; highlight the modified cell, and the row and column headers, for a few seconds; have a switch or confirmation dialog to modify days that are not today.

Describe alternatives you've considered

Additional context

@opk12 opk12 added the suggestion Make decision later label Sep 7, 2023
@FriesI23
Copy link
Owner

A possible solution is to extend the 'tap-long_press' combination to 'double_tap-long_press,' or just allow users to choose which two tap actions (press, long press, or double press) to use. That's a great suggestion, I plan to add similar feature in future.

However, there might be a need to refactor some code in the near term, so this feature might be added in a slightly later version."

@FriesI23 FriesI23 added enhancement New feature or request and removed suggestion Make decision later labels Sep 10, 2023
@FriesI23
Copy link
Owner

@opk12
Copy link
Author

opk12 commented Sep 10, 2023

Thank you. But what is the tap + long press combination? I open the history with one tap on < and scroll the history with a swipe(?)

@FriesI23 FriesI23 linked a pull request Dec 16, 2023 that will close this issue
FriesI23 added a commit that referenced this issue Dec 17, 2023
feat: Introduce custom tap actions for habit records (#120)

- organize tiles related to operations into a separate group
- add configuration tile
- add translations

see #99
konstantintutsch added a commit to konstantintutsch/mhabit that referenced this issue Dec 21, 2023
konstantintutsch added a commit to konstantintutsch/mhabit that referenced this issue Dec 21, 2023
FriesI23 pushed a commit to konstantintutsch/mhabit that referenced this issue Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants