-
-
Notifications
You must be signed in to change notification settings - Fork 1
Closed
Description
Problem
After the decomposition in PR #2236 (#1579), CodebaseAnalytics.vue still has 1,073 lines of <style scoped> CSS. Many selectors now reference elements that were extracted into child components:
Dead selectors (template moved to child components)
.summary-cardseverity variants (.critical,.high,.medium,.low,.info,.type-*) — moved toProblemsReportSection.vueandCodebaseChartsSection.vue.code-intelligence-section,.code-intel-tabs— moved toCodebaseSecurityPanel.vue.card-header-content,.refresh-indicator— moved toCodebaseOverviewPanel.vue.enhanced-analytics-grid,.metrics-grid,.metric-item— moved to dashboard panels.communication-metrics,.performance-details,.quality-details— moved to dashboard panels.btn-debug,.btn-cancel— moved toAnalyticsHeaderControls.vue.source-selector-row,.source-select,.btn-manage-sources— moved toSourceManager.vue
Impact
Since Vue's <style scoped> only applies to elements in the component's own template, these selectors have zero effect. They add ~500+ lines of dead weight.
Suggested approach
- Audit each CSS selector against the current template (lines 1-339)
- Remove selectors whose target elements are no longer in the parent template
- Move any needed styles to the respective child components if not already present there
- Verify no visual regressions after cleanup
Discovered During
Impact
Low — dead code, no functional effect, but adds maintenance burden and makes the file harder to navigate.
Reactions are currently unavailable