Skip to content

Add stable CSS class hooks to Participation Summary table rows#2609

Closed
Copilot wants to merge 1 commit intocopilot/add-css-class-to-participation-summaryfrom
copilot/sub-pr-2608
Closed

Add stable CSS class hooks to Participation Summary table rows#2609
Copilot wants to merge 1 commit intocopilot/add-css-class-to-participation-summaryfrom
copilot/sub-pr-2608

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 1, 2026

TallyResultsSummary lacked stable CSS selectors on its table rows, making it impossible for operators to reliably target individual participation metrics with custom CSS overrides.

Changes

  • TallyResultsSummary.tsx

    • Added participation-summary-table class to the <Table> element.
    • Each <TableRow> now carries two classes: the shared participation-summary-row base class plus a row-specific semantic class (e.g. eligible-voters, total-valid-votes, blank-votes, weight).
    • Extracted repeated summaryRowSx and the summaryRowClassName helper to reduce duplication.
    // example rendered output
    <tr class="participation-summary-row total-valid-votes"></tr>
  • TallyResultsSummary.test.js — new Jest suite using renderToStaticMarkup that asserts every expected row class is present in the rendered markup.

  • Docsdevelopers_admin-portal.md updated to document the new CSS hooks; release-next.md entry added.

Parent issue: https://github.com/sequentech/meta/issues/2608

Copilot AI changed the title [WIP] [WIP] Address feedback on stable CSS hooks for Participation Summary rows Add stable CSS class hooks to Participation Summary table rows May 1, 2026
Copilot finished work on behalf of edulix May 1, 2026 17:01
Copilot AI requested a review from edulix May 1, 2026 17:01
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 2026


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


Thank you for your contribution! Before we can merge this PR, we need you to sign our Contributor License Agreement (CLA).

To sign the CLA, please comment on this PR with:
I have read the CLA Document and I hereby sign the CLA

This is a one-time requirement. Once you have signed, all future contributions will be covered.

You can read the full CLA document here: https://github.com/sequentech/step/blob/main/.github/cla/CLA.md


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

Copilot stopped work on behalf of edulix due to an error May 1, 2026 18:53
@edulix edulix closed this May 3, 2026
@edulix edulix deleted the copilot/sub-pr-2608 branch May 3, 2026 10:52
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