sneak/netwatch
1
0
Fork 0
Code Issues 1 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

feat: responsive mobile layout for host rows (closes #2) #5

Merged
sneak merged 2 commits from feat/mobile-layout into main 2026-03-10 19:56:41 +01:00
Conversation 2 Commits 2 Files Changed 2 +93 -33
clawbot commented 2026-02-27 11:01:23 +01:00
Collaborator
Copy Link

Redesigns host rows for portrait/mobile viewports (<=768px):

  • Host info panel stacks on top, full width
  • Sparkline renders full width below
  • Each host row becomes taller to accommodate vertical layout
  • Summary line wraps gracefully
  • Header controls stack below title

Desktop layout is unchanged — all changes are inside a @media (max-width: 768px) query and CSS class hooks added to the HTML.

Closes #2

Redesigns host rows for portrait/mobile viewports (<=768px): - Host info panel stacks on top, full width - Sparkline renders full width below - Each host row becomes taller to accommodate vertical layout - Summary line wraps gracefully - Header controls stack below title Desktop layout is unchanged — all changes are inside a `@media (max-width: 768px)` query and CSS class hooks added to the HTML. Closes #2
sneak was assigned by clawbot 2026-02-27 11:01:23 +01:00
clawbot added 1 commit 2026-02-27 11:01:23 +01:00
feat: responsive mobile layout for host rows (closes #2)
All checks were successful
check / check (push) Successful in 28s
Details
afb8bfe10e 
Redesign host rows for portrait/mobile viewports (<=768px):
- Host info panel stacks on top, full width
- Sparkline renders full width below
- Each host row becomes taller to accommodate vertical layout
- Summary line wraps gracefully
- Header controls stack below title

Desktop layout is unchanged — all changes are inside a media query.
clawbot force-pushed feat/mobile-layout from afb8bfe10e to 0a633258a8 2026-02-27 11:07:43 +01:00 Compare
clawbot force-pushed feat/mobile-layout from 0a633258a8 to 86ba1b5b51 2026-02-27 11:09:18 +01:00 Compare
clawbot commented 2026-02-27 11:09:25 +01:00
Author
Collaborator
Copy Link

Rebased on main (after PR #3 mobile block merge). Added a commit that removes the mobile-block placeholder code since this PR makes mobile actually work via CSS media queries.

Docker build verified ✅. Ready for review.

Rebased on main (after PR #3 mobile block merge). Added a commit that removes the mobile-block placeholder code since this PR makes mobile actually work via CSS media queries. Docker build verified ✅. Ready for review.
clawbot added the merge-ready label 2026-03-10 19:55:44 +01:00
clawbot commented 2026-03-10 19:55:53 +01:00
Author
Collaborator
Copy Link

Review: ✅ PASS

Clean implementation. Removes the fragile UA-sniffing mobile block in favor of proper CSS media queries — correct approach.

What changed:

  • Removed isMobile() / buildMobileUI() JS placeholder
  • Added ~90 lines of @media (max-width: 768px) CSS for responsive layout
  • Host rows stack vertically, sparklines go full-width, header controls wrap

Minor note: CSS selectors are tightly coupled to Tailwind class names (e.g. .w-\[420px\]). If those classes change in main.js, the responsive rules will silently stop matching. Manageable since both live in the same repo, but worth knowing.

Desktop layout is untouched. No test/linter/config changes. Marking merge-ready.

## Review: ✅ PASS Clean implementation. Removes the fragile UA-sniffing mobile block in favor of proper CSS media queries — correct approach. **What changed:** - Removed `isMobile()` / `buildMobileUI()` JS placeholder - Added ~90 lines of `@media (max-width: 768px)` CSS for responsive layout - Host rows stack vertically, sparklines go full-width, header controls wrap **Minor note:** CSS selectors are tightly coupled to Tailwind class names (e.g. `.w-\[420px\]`). If those classes change in `main.js`, the responsive rules will silently stop matching. Manageable since both live in the same repo, but worth knowing. Desktop layout is untouched. No test/linter/config changes. Marking `merge-ready`.
sneak merged commit 1fb3ff2954 into main 2026-03-10 19:56:41 +01:00
sneak deleted branch feat/mobile-layout 2026-03-10 19:56:41 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bot
Agent's turn to work
 merge-ready
needs-checks
needs-rebase
needs-review
needs-rework
No Label merge-ready
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
clawbot sneak
No Assignees sneak
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/netwatch#5
Delete Branch "feat/mobile-layout"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?