Skip to content

fix(viewer): improve topology and sysinfo layout on narrow screens#810

Merged
thinkingfish merged 1 commit intoiopsystems:mainfrom
thinkingfish:topology-narrow-screen
Apr 19, 2026
Merged

fix(viewer): improve topology and sysinfo layout on narrow screens#810
thinkingfish merged 1 commit intoiopsystems:mainfrom
thinkingfish:topology-narrow-screen

Conversation

@thinkingfish
Copy link
Copy Markdown
Member

@thinkingfish thinkingfish commented Apr 19, 2026

Summary

  • Remove compact mode from CPU topology, always use labeled mode with responsive column wrapping
  • Group sub-groups by core type (P-core vs E-core) for balanced row packing on hybrid Intel CPUs — prevents uneven rows like 5+3 instead of 4+4
  • Fix sysinfo card overflow on narrow screens (min-width, word-break, overflow containment)
  • Remove dead compact-mode CSS (~50 lines) and legend rendering code (~100 lines)

Test plan

  • Load cachecannon.parquet in viewer, resize to narrow widths — verify topology rows are balanced
  • Check hybrid CPU topology (P+E cores) renders with even rows per core type
  • Verify sysinfo cards don't overflow on narrow screens
  • Confirm cache toggle and NUMA toggle still work correctly

🤖 Generated with Claude Code

Remove compact mode from CPU topology rendering in favor of always
using labeled mode with responsive column wrapping. Group sub-groups
by core type (P-core vs E-core) for balanced row packing on hybrid
CPUs. Fix sysinfo card overflow on narrow screens with CSS containment.
Remove dead compact-mode CSS and legend code.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@thinkingfish thinkingfish marked this pull request as ready for review April 19, 2026 04:34
@thinkingfish thinkingfish merged commit 54b5985 into iopsystems:main Apr 19, 2026
16 checks passed
@thinkingfish thinkingfish deleted the topology-narrow-screen branch April 19, 2026 04:50
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