Open martinrieder opened 3 months ago
mathisdt
commented
3 months ago :information_source: There are two different notification texts, the standard text is changed every minute but the "minimal text" is only activated once (so the re-notification doesn't get on the user's nerves, this was a problem for some people). The option "never update persistent notification" activates the minimal notification text, which also should be adapted accordingly when implementing this!
martinrieder
commented
3 months ago Thanks, I was not aware of that, but it makes sense. I think that the messages that I mentioned as additional proposal should be chosen according to that setting. The text should be kept short to avoid clipping anyways...
Additionally, I propose to change the title if even more details are required:
Worked (time) hours since (clock-in)
Then in the notification text also add the two stop times:
Flexi time (was/is) reached (in/since) (time) at (flexi-out) ... Work time (was/is) reached (in/since) (time) at (clock-out) ...
Updated every minute:
Worked (time) hours
Flexi time (was/is) reached (in/since) (time)Work time (was/is) reached (in/since) (time)Updated only at clock-in, clock-out or flexi-out:
Clocked in since (clock-in)
Flexi time (was/is) reached at (flexi-out)Work time (was/is) reached at (clock-out)I am not sure if users need to see both times. Therefore I suggest that the notification should choose the time that comes first. It would also be possible to simply provide two independent notifications (for flex and work) that could be treated separately.
PS: The text of the respective setting could also be clarified. Instead of "never update persistent notification", I suggest writing "Update on events only".
martinrieder
commented
3 months ago I wrote:
It would also be possible to simply provide two independent notifications (for flex and work) that could be treated separately.
In case that there are two or more clock-in events, i.e. when the setting "Handle methods separately" is active, the I reckon that there must be multiple notifications active.
This thought leads me to extend the notification title by the name of the active task.
The notification feature is really great, though I would like to request some improvements to make it even greater:
regular work time is over since...Should be changed to this:
Because:
Additionally, I propose to change the title if even more details are required:
Worked (time) hours since (clock-in)Then in the notification text also add the two stop times: