Skip to content

Remove duplicate AIC repository table - #503

Merged
pelikhan merged 4 commits into
mainfrom
copilot/update-repositories-view
Sep 1, 2026
Merged

Remove duplicate AIC repository table#503
pelikhan merged 4 commits into
mainfrom
copilot/update-repositories-view

Conversation

Copilot AI commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

The repositories view repeats the pie chart’s repository-level AIC data in a table directly below it, before the more detailed activity table.

Changes

  • Hide the redundant chart table only on the repositories page.
  • Preserve the AIC pie chart, legend, links, and detailed repository activity table.
  • Add regression coverage for the page-specific styling.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title Remove redundant repositories AIC table Remove duplicate AIC repository table Sep 1, 2026
Copilot AI requested a review from pelikhan September 1, 2026 21:14
Comment thread dashboard/site/src/styles.js Outdated
.value-empty h3 { margin: 16px 0 5px; font-size: 1.125rem; }
.value-empty p { max-width: 620px; margin: 0; color: var(--muted); }
.repositories-page .custom-view-grid { display: block; }
.repositories-page .chart-view-pie > .table-region { display: none; }

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot update the JSON and remove table. do not modify CSS.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Addressed in abf152f. The CSS workaround is reverted, and the repositories AIC chart now uses the validated JSON option "table": false.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan
pelikhan marked this pull request as ready for review September 1, 2026 21:19
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan

pelikhan commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

@copilot resolve the merge conflicts in this pull request

…tories-view

# Conflicts:
#	dashboard/site/src/components/data-view.js

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot AI commented Sep 1, 2026

Copy link
Copy Markdown
Contributor Author

@copilot resolve the merge conflicts in this pull request

Resolved in merge commit 394fe1f. The JSON-driven table removal is preserved alongside the current pie-chart layout from main.

@pelikhan
pelikhan merged commit 8950f82 into main Sep 1, 2026
5 checks passed
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.

2 participants