Skip to content

fix(frontend): align slider track width with tick labels in config sections - #369

Merged
javi11 merged 3 commits into
mainfrom
fix/slider-width-mismatch
Feb 27, 2026
Merged

fix(frontend): align slider track width with tick labels in config sections#369
javi11 merged 3 commits into
mainfrom
fix/slider-width-mismatch

Conversation

@javi11

@javi11 javi11 commented Feb 27, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add w-full to the Segment Verification slider in WorkersConfigSection
  • Add w-full to the Sampling Percentage and Acceptable Missing Threshold sliders in HealthConfigSection

Root cause: DaisyUI's .range class caps the track width at 20rem by default. The tick label row uses flex justify-between at full container width, causing the thumb to visually misalign with the labels. Adding w-full overrides this cap so track and labels span the same width.

This mirrors the fix already applied in StreamingConfigSection.

Test plan

  • Open Workers config → Segment Verification slider: thumb aligns with Fast/Balanced/Deep labels at 1%, 50%, 100%
  • Open Health config → Sampling Percentage slider: thumb aligns with percentage labels at all values
  • Open Health config → Acceptable Missing Threshold slider: thumb aligns with labels at all values
  • Compare against StreamingConfigSection sliders (already fixed) to confirm consistent behaviour

🤖 Generated with Claude Code

javi11 and others added 3 commits February 27, 2026 17:06
…ctions

Add `w-full` to range inputs in WorkersConfigSection and HealthConfigSection
so the slider track spans the full container width, matching the flex label
row beneath it. Without this, DaisyUI caps the track at 20rem while labels
span 100%, causing the thumb to visually misalign with tick marks.

Mirrors the fix already applied in StreamingConfigSection.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…uiltin, range over int)

- Replace `interface{}` with `any` in response_test.go and webdav/file_system.go
- Use builtin `min()` instead of manual if/clamp in iso/fs.go
- Use `for i := range uint32(scanLimit)` idiom in iso/fs.go
- Add .vscode/launch.json and .vscode/settings.json for local dev debugging

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@javi11
javi11 merged commit b705e47 into main Feb 27, 2026
1 check passed
@javi11
javi11 deleted the fix/slider-width-mismatch branch February 27, 2026 16:10
yoshitaka420 pushed a commit to yoshitaka420/altmount that referenced this pull request Jun 1, 2026
…ctions (kipsilabs#369)

* fix(frontend): align slider track width with tick labels in config sections

Add `w-full` to range inputs in WorkersConfigSection and HealthConfigSection
so the slider track spans the full container width, matching the flex label
row beneath it. Without this, DaisyUI caps the track at 20rem while labels
span 100%, causing the thumb to visually misalign with tick marks.

Mirrors the fix already applied in StreamingConfigSection.
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