ReadTimer + Profiles auto-exec - #14309
Conversation
| min = curr_min, | ||
| ok_text = _("Set alarm"), | ||
| title_text = _("New alarm"), | ||
| ok_text = _("Start timer"), |
There was a problem hiding this comment.
Is this change congruent with that other one?
There was a problem hiding this comment.
It's a recent change after heavily testing the new features.
What do we want and what does this SpinWidget do? We want to start the timer, not just set the time/interval.
There was a problem hiding this comment.
I'd say an alarm is not a timer (even if it acts as an x-minute timer under the hood) and the checkbox is just a side feature.
The interval is a timer though and that's probably a better word to use there.
There was a problem hiding this comment.
But also "Stop timer" for both.
Do you recommend to keep "Set alarm" and "Set interval" for spinwidget OK buttons?
There was a problem hiding this comment.
I remember the @offset-torque 's post about the timer and its wordings that they were not perfect.
There was a problem hiding this comment.
Do you recommend to keep "Set alarm" and "Set interval" for spinwidget OK buttons?
"Set alarm" or "Set time" for the alarm, "Start timer" for the interval one.
Edit: but I mean "timer" everywhere, no interval.
There was a problem hiding this comment.
There is still confusing wording in that plugin.
As of 2025.08 version, situation is this:
- "Set time" button creates a new "alarm" (but it shows a "timer" in the status bar)
- "Set interval" button sets a "timer"
Why not properly name the buttons for things which they will create.
In this case they would be:
- Set an alarm
- Set a timer
Also wording in the "New Alarm" dialog should match the thing we are creating. So instead of "Show timer in status bar" it should be "Show alarm in the status bar" to be consistent and not confusing.
Visualization of the proposed changes:
Should I move this to an issue or good here?
There was a problem hiding this comment.
It's okay here, for the next round of changes.
Now (in the nightly build) the main menu items are: Set alarm, Set interval.
I noticed to change the latter to Set timer.
In the footer, we actually show the countdown timer in both cases.
There was a problem hiding this comment.
In the footer, we actually show the countdown timer in both cases.
I just checked and status bar shows either the "alarm countdown" or the "timer countdown".
Think as a new user:
I want to set an alarm. I clicked the "Set alarm" button. I didn't click the "Set timer" button. Why am I seeing "Show timer in status bar"?!? I should see a countdown to my alarm that I am currently setting.
Which is what that checkboxes do actually. Just named incorrectly now.
"Set timer" dialog's checkboxes are (nearly) correct because they show the countdown to the "timer" which is set in that dialog.
Clearer and more accurate wording for them is:
- "Show countdown to alarm in status bar"
- "Show countdown to timer in status bar"
But it is more crowded.
3 participants
Usual restriction: profile auto-exec requires reader or file browser without any upper windows/dialogs opened.
This change is