UI tweaks - #3
Merged
Merged
Conversation
…ray matching hints - Reduce sidebar/analysis-panel section top margin (32px -> 20px) - Replace inline array-matching toggle-hint text with ? tooltip icons, matching the existing sidebar Comparison section pattern
…ment fix, and toolbar 'Changes only' checkbox - Reduce left sidebar width 300px -> 270px - Add -moz-appearance: none fallback so the switch thumb renders in Firefox - Apply the same pill-switch re-skin to the toolbar's 'Changes only' checkbox
Thumb used var(--card), which is identical to the track's var(--muted) background in dark theme. Switch to var(--muted-foreground) for contrast in both themes and both checked/unchecked states.
- Add collapse/expand toggle to the analysis panel resize handle, with localStorage persistence separate from the dragged width. - Move the two mobile drawer trigger buttons out of SidebarComponent/ AnalysisPanelComponent into a shared HomeComponent .mobile-panel-bar styled as a continuation of the app header. - Re-skin the Comparison section's checkboxes as pill switches (CSS-only, native <input type="checkbox"> semantics preserved), with a -moz-appearance fix for Firefox ::after support. - Fix reload flashes: analysis panel collapsed state and width both start from Angular defaults (avoiding a hydration mismatch) and are corrected via afterNextRender; a pre-hydration index.html script applies both as pure visual overrides ahead of first paint. The collapsed override lives in the global styles.css (component-scoped :root selectors are dead CSS under Angular's view encapsulation, which scopes every compound selector including :root). The width override is an injected global <style> tag, removed once Angular's own signal is corrected so it never pins a later drag-resize. A home.component.ts effect keeps the collapsed data-attribute in sync with the live signal so expanding after a collapsed reload works.
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
1 participant
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.
UI Tweaks