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

Unable to edit input field when recurrence is set #763

Open
TriggerDingus opened this issue Jan 19, 2025 · 12 comments
Open

Unable to edit input field when recurrence is set #763

TriggerDingus opened this issue Jan 19, 2025 · 12 comments
Labels
bug Something isn't working

Comments

@TriggerDingus
Copy link

TriggerDingus commented Jan 19, 2025

Bug Report

Important

Please follow this template strictly when reporting bugs.

Reports that do not adhere to this template will not be addressed and will be closed.

App Version:
2.0.16

Platform: Windows / macOS / Linux
Linux LMDE 6
Cinnamon 6.4.6

Installation Method: App Store / Direct Download / Other
Flatpak

Expected Behavior:
Edits to existing ToDo's will be captured and saved.

Actual Behavior:
Edits to existing ToDo's are being ignored.
However, I'm able to add and update new ToDo's.

Steps to Reproduce:
Attempt to make any update to an existing ToDo item and it will be rejected.
Creating a new ToDo is accepted and making edits to that item is also accepted.

I've been using Sleek a few years now and this is a very new issue that appeared just today. I've tried rebooting my system (a.k.a. "punting") to see if that resolves it, but it doesn't. I can edit the ToDo file directly through a text editor and the updates will show up in Sleek, so it doesn't appear to be a file permissions issue. I also archived completed ToDo's in Sleek thinking maybe reducing my file size would help, but it did not.

Screenshots:
I've attached a video of the issue, with an explanation of the steps I'm attempting.

Kazam_screencast_00000.mp4
@TriggerDingus TriggerDingus added the bug Something isn't working label Jan 19, 2025
@github-project-automation github-project-automation bot moved this to Backlog in sleek 2.x Jan 19, 2025
@TriggerDingus
Copy link
Author

TriggerDingus commented Jan 19, 2025

Closing - I determined the issue was specifically with ToDo items that have parenthesis in the ToDo text. Not sure why that triggers the issue, but that was it.

@github-project-automation github-project-automation bot moved this from Backlog to Done in sleek 2.x Jan 19, 2025
@ransome1
Copy link
Owner

@TriggerDingus Can you sure an example? I mean, this might be a bug after all.

@TriggerDingus
Copy link
Author

TriggerDingus commented Jan 19, 2025

Within the bug report, I attached a video showing the issue. I suspect you should be able to recreate the issue if you create a similar ToDo item that includes parens.

@ransome1
Copy link
Owner

Unfortunately it doesn't work on my end :(

Image

@TriggerDingus
Copy link
Author

I'll embed a link in a few minutes.
I just determined the issue isn't only with items that include parens. I'm also experiencing the issue with this ToDo item, which you may be able to use to reproduce it:
(C) 2025-01-01 Build into blog going forward: long-tail keywords listed in your 12/5/24 journal notes due:2025-01-19 rec:1w

@ransome1
Copy link
Owner

Thanks, I can confirm there is some sort of bug here. Do you mind if I still use this bug report?

@TriggerDingus
Copy link
Author

TriggerDingus commented Jan 19, 2025 via email

@ransome1 ransome1 moved this from Done to 2.0.17 in sleek 2.x Jan 19, 2025
@ransome1 ransome1 changed the title Sleek unable to update existing ToDo's Unable to edit input field when recurrence is set Jan 19, 2025
@ransome1 ransome1 reopened this Jan 19, 2025
@github-project-automation github-project-automation bot moved this from 2.0.17 to Bugs in sleek 2.x Jan 19, 2025
@TriggerDingus
Copy link
Author

TriggerDingus commented Jan 19, 2025

Here is an example of an item with parenthesis that triggers the issue:

(A) Test to-do item that includes paren (like this) triggers the issue. due:2025-01-21

Note: I see you changed the issue title to imply Recurrence is a factor. Please note that this example doesn't have recurrence enabled.

@ransome1 ransome1 moved this from Bugs to In Progress in sleek 2.x Jan 20, 2025
@ransome1
Copy link
Owner

@TriggerDingus could you please take a quick glance at this pre-release? https://github.com/ransome1/sleek/releases/tag/v2.0.17-rc.1. I refactored the recurrence picker and I believe this (partly?) solves the issue, we were seeing here.

The AppImage should be a good choice, it won't interfere with your Flathub installation.

@TriggerDingus
Copy link
Author

I tried the appImage against a copy of my ToDo file and it appears to be functioning. I tested numerous ToDo items, regardless of recurrence and included punctuation, and couldn't get it to fail. Appears to be working.

@ransome1
Copy link
Owner

Thanks for the feedback. I'm going to test this some more and will then add this to 2.0.17.

@mshuflin
Copy link

I had this issue in 2.0.16 and it appears fixed in 2.0.17.rc-1

Here is an example of a task that had the issue and is now working properly:

2024-11-17 Clean windows @chores @home t:2025-01-25 due:2025-01-31 rec:45d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: In Progress
Development

No branches or pull requests

3 participants