Skip to content

Improve workflow time selector and biweekly day toggle#26

Merged
alari76 merged 1 commit intomainfrom
fix/workflow-time-selector-and-biweekly-toggle
Mar 9, 2026
Merged

Improve workflow time selector and biweekly day toggle#26
alari76 merged 1 commit intomainfrom
fix/workflow-time-selector-and-biweekly-toggle

Conversation

@alari76
Copy link
Copy Markdown
Contributor

@alari76 alari76 commented Mar 9, 2026

Summary

  • Narrowed the time input field and added color-scheme CSS so the native time picker dropdown and clock icon render correctly in both dark and light themes
  • Replaced the standalone "Bi-weekly" preset button with a Weekly / Every 2 weeks toggle that appears when a specific day is selected, so users can choose both the trigger day and frequency
  • Updated buildCron, parseCron, and describeCron to support biweekly-with-day encoding (e.g. biweekly-10 6 */14 * 1) with backward compatibility for legacy expressions

Test plan

  • All 855 existing tests pass
  • Verify time input is narrower and clock icon is visible in dark mode
  • Verify native time picker dropdown has dark theme styling
  • Select a day (e.g. Mon) and confirm Weekly/Bi-weekly toggle appears
  • Toggle to "Every 2 weeks" and confirm cron description updates (e.g. "Bi-weekly Mon at 06:00")
  • Switch days while in bi-weekly mode and confirm it stays bi-weekly
  • Select Daily/Weekdays preset and confirm toggle hides
  • Test in light mode — time picker should use light scheme

🤖 Generated with Claude Code

…kly/biweekly day toggle

- Narrow time input field (w-full → w-40) and add dark/light color-scheme
  styling so the native time picker dropdown and clock icon match the app theme
- Replace standalone Bi-weekly preset with a Weekly/Bi-weekly toggle that
  appears when a specific day is selected, allowing users to choose both
  the day and frequency together
- Support biweekly-with-day cron encoding (e.g. biweekly-1 → */14 * 1)
  while maintaining backward compatibility with legacy biweekly expressions

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@alari76 alari76 merged commit 99b5283 into main Mar 9, 2026
2 checks passed
@alari76 alari76 deleted the fix/workflow-time-selector-and-biweekly-toggle branch March 10, 2026 05:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant