ikhal: New alarm fields should default to empty

[wavexx]

In ikhal, when adding a new alarm in a new event ("n" -> alarms -> add), the fields are pre-populated with zeros, which means I always need to erase it to enter a single digit number, since switching to the field and just typing "1" will turn it into 10 instead. This always felt incredibly weird for me. I guess that the problem is that the field lacks "pre-selection" normally enabled on default fields that would cause the exiting text to be replaced by default.

I believe a better option would be to leave all fields blank, and eventually fill them with zeros (if needed?) when the form is saved.

Overall even the date/time fields behave similarly weird. I'd almost expect overwrite behavior on those. Maybe we can switch to the thin cursor to make it more obvious?

[redotics]

Hi @wavexx

Thanks for capturing that. I am an user as well and just wanted to reinforce the importance of alarms for a valid calendar app.

The zeros are indeed a bit annoying and what I would suggest for developers is:

• consider default value as zero if the values are empty
• display empty fields then
• use configuration in order to set the alarm to a default delta-time + when (before/after, use before by default)

(then in another issue: connect the alarm to notify-send or similar)

that sounds amazing already. sorry only tested on 0.11.2