Skip to content

Improve timeslot forms with compact table layout#1797

Draft
Simrayz wants to merge 9 commits intomainfrom
feat/1526-improve-timeslot-forms
Draft

Improve timeslot forms with compact table layout#1797
Simrayz wants to merge 9 commits intomainfrom
feat/1526-improve-timeslot-forms

Conversation

@Simrayz
Copy link
Contributor

@Simrayz Simrayz commented Feb 19, 2026

Scope and purpose

Fixes #1794.

Replaces the bulky stacked fieldset layout for time recurrences with a compact table-based layout. Also reworks the timeslot list to use accordion cards with lazy-loaded forms, adds flatpickr time pickers, and improves form UX with inline success/error banners.

This pull request

  • adds flatpickr as a dependency for time inputs on the timeslot page

Contributor Checklist

  • Added a changelog fragment for towncrier
  • Added/amended tests for new/changed code
  • Added/changed documentation, including updates to the user manual if feature flow or UI is considerably changed
  • Linted/formatted the code with ruff and djLint, easiest by using pre-commit
  • The first line of the commit message continues the sentence "If applied, this commit will ...", starts with a capital letter, does not end with punctuation and is 50 characters or less long. See our how-to
  • If applicable: Created new issues if this PR does not fix the issue completely/there is further work to be done
  • If this results in changes in the UI: Added screenshots of the before and after

@sonarqubecloud
Copy link

@Simrayz Simrayz requested a review from a team February 19, 2026 15:27
@Simrayz Simrayz self-assigned this Feb 19, 2026
@Simrayz Simrayz added frontend Affects frontend timeslot design HTMx Views, urls, templates... labels Feb 19, 2026
@github-actions
Copy link

Test results

    6 files    894 suites   1m 48s ⏱️
  701 tests   700 ✅ 1 💤 0 ❌
4 206 runs  4 200 ✅ 6 💤 0 ❌

Results for commit ecef61d.

@codecov
Copy link

codecov bot commented Feb 19, 2026

Codecov Report

❌ Patch coverage is 79.48718% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.73%. Comparing base (d284743) to head (ecef61d).

Files with missing lines Patch % Lines
src/argus/htmx/timeslot/views.py 68.00% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1797      +/-   ##
==========================================
+ Coverage   82.55%   82.73%   +0.17%     
==========================================
  Files         137      137              
  Lines        6552     6573      +21     
==========================================
+ Hits         5409     5438      +29     
+ Misses       1143     1135       -8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

design frontend Affects frontend HTMx Views, urls, templates... timeslot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Modernise timeslots forms

1 participant

Comments