feat(gui): comprehensive UX/UI improvements and code cleanup#30
Merged
Conversation
## Critical fixes - Fix e2e smoke tests to use current routes - Consolidate duplicate Catalog/Registry pages into Catalog with merged features - Define missing .spinner CSS class and .data-table styles - Add 5-minute context timeout to RunSkell to prevent GUI hangs - Fix Catalog lifecycle filter to show all 5 stages ## UX improvements - Make breadcrumbs clickable in ProjectPageHeader and SkillDetail - Add cursor-pointer to all clickable non-button elements (modals, backdrop, file tree) - Make project names/paths clickable in Repositories page - Standardize heading hierarchy (text-2xl font-bold) across all pages - Standardize card backgrounds using .card CSS class - Fix broken navigation links (/skills, /registry, Shared Library) ## Performance - Add React.memo to SkillCard, 4 Badge components, CollapsibleSection - Batch repository health fetches in groups of 2 to prevent process storms - Add client-side pagination (50/page) to InstalledSkills and Activity ## Code cleanup - Delete 6 orphaned page files (Registry, AuditLog, Doctor, Sync, Cache, Dashboard) - Delete 6 stale test files for deleted pages - Replace catch (e: any) with catch (e: unknown) in Settings - Clean up eslint-disable comment in SkillDetail ## CI/CD - Update CI and Release workflows to use Wails v3 CLI (was v2)
Exclude auto-generated Android/iOS/Docker/Linux build scaffolding and macOS binary assets from version control.
Include Sidebar nav updates, store changes, Wails bindings, and supporting components (AddSkillButton, CurrentProjectSummary, RecentProjects, navigation helpers).
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Comprehensive GUI UX/UI improvements across the entire Skell desktop app.
🔴 Critical fixes
.spinnerCSS class and.data-tablestylesRunSkellto prevent GUI hangs🖱️ UX improvements
cursor-pointerto all clickable non-button elements (modals, backdrops, file tree entries)text-2xl font-bold) across all pages.cardCSS class/skills→ dead redirect, Shared Library, ContributeInfo)⚡ Performance
React.memoto SkillCard, 4 Badge components, CollapsibleSection🧹 Code cleanup
catch (e: any)withcatch (e: unknown)in Settings.gitignoreto cover Wails v3 generated build artifacts🔧 CI/CD
Test results