Skip to content

Add content types filter to mobile channel editor#197

Merged
disnet merged 1 commit into
mainfrom
radial/3mlffh2y/mobile-channel-editor-needs-to-let-you-f
May 9, 2026
Merged

Add content types filter to mobile channel editor#197
disnet merged 1 commit into
mainfrom
radial/3mlffh2y/mobile-channel-editor-needs-to-let-you-f

Conversation

@disnet
Copy link
Copy Markdown
Owner

@disnet disnet commented May 9, 2026

Summary

  • Mobile channel editor (MobileFilterSheet.svelte Channel tab) now exposes the same Content Types checklist that the desktop FilteredViewModal already offers, so feed channels can be narrowed by source type from a phone.
  • Visibility and save-payload gating mirror desktop exactly: shown for feed channels except when sources are manual + include-only; cleared on save in that combo.
  • Existing channels load their typeFilter selections back into the mobile editor.

Test plan

  • cd frontend && npm run check (passing locally — 0 errors)
  • Manual viewport-narrow smoke:
    • New feed channel, Sources = All → Content Types section visible; toggle a checkbox, save, reopen → persists
    • Switch to Manual + Include only → Content Types section hides
    • Switch back to All → previously checked types still in state (only the save payload is gated, not the UI state)
    • Switch to Saved channel type → Content Types hidden
    • Switch to Smart mode → Content Types visible
    • Edit a desktop-created channel that already has typeFilter → checkboxes reflect existing selections
    • Saved feed view honors the filter (article list narrows to selected types)

🤖 Generated with Claude Code

Mirrors the desktop FilteredViewModal Content Types checklist on the
mobile channel editor (feed channels, hidden when sources are manual +
include-only).
@disnet disnet merged commit 74e497e into main May 9, 2026
3 checks passed
@disnet disnet deleted the radial/3mlffh2y/mobile-channel-editor-needs-to-let-you-f branch May 9, 2026 23:26
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