Skip to content

chore: split CSS into partials and fix refresh during selection#24

Merged
eclectic-coding merged 2 commits into
mainfrom
chore/split-css-partials
May 19, 2026
Merged

chore: split CSS into partials and fix refresh during selection#24
eclectic-coding merged 2 commits into
mainfrom
chore/split-css-partials

Conversation

@eclectic-coding
Copy link
Copy Markdown
Owner

Summary

  • Splits the 618-line application.css into 10 focused partial files (_01_base.css through _10_responsive.css) for easier maintenance
  • Updates inline_styles helper to glob _*.css files in sort order and concatenate them — runtime output is identical
  • application.css kept as an empty Sprockets manifest for compatibility
  • Fixes auto-refresh wiping checkbox selections: refresh now skips its tick whenever any checkbox inside the turbo-frame is checked, resuming normally once selections are cleared or submitted

Test plan

  • Dashboard, jobs list, and processes still auto-refresh when no checkboxes are checked
  • Selecting checkboxes on the jobs or failed jobs list pauses auto-refresh — selections persist beyond the 5/10 second interval
  • Submitting a bulk action (Discard Selected / Retry Selected) clears selections and refresh resumes
  • All styles render correctly — no visual regressions
  • All 129 specs pass

🤖 Generated with Claude Code

eclectic-coding and others added 2 commits May 19, 2026 14:03
Break the 618-line stylesheet into 10 focused partials (_01_base.css
through _10_responsive.css). inline_styles helper now globs _*.css
files in sort order and concatenates them — runtime output is identical.
application.css kept as an empty Sprockets manifest for compatibility.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Skip the reload tick if any checkbox inside the turbo-frame is checked,
preventing selections from being wiped mid-bulk-action. Refresh resumes
normally on the next tick once all boxes are cleared or the form submits.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@eclectic-coding eclectic-coding merged commit 27bea48 into main May 19, 2026
5 checks passed
@codecov
Copy link
Copy Markdown

codecov Bot commented May 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (dfb22c7) to head (725afd9).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #24   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           15        15           
  Lines          237       238    +1     
=========================================
+ Hits           237       238    +1     

☔ 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.

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