Skip to content

48209 - Participants tab of a test no longer shows the number of participants - #12031

Closed
lorenzgith wants to merge 1 commit into
ILIAS-eLearning:release_10from
lorenzgith:fix/48209-participant-count
Closed

48209 - Participants tab of a test no longer shows the number of participants#12031
lorenzgith wants to merge 1 commit into
ILIAS-eLearning:release_10from
lorenzgith:fix/48209-participant-count

Conversation

@lorenzgith

Copy link
Copy Markdown
Contributor

https://mantis.ilias.de/view.php?id=48209

In ILIAS 9 the participants table was based on ilTable2GUI, which displays the number of records by default, so the number of participants was visible on the Participants tab. After the migration to the new UI data table the count is no longer surfaced anywhere. For an exam that is a number one wants to see at a glance.

This appends it to the table title, Participants (Total: 12).

Two things this deliberately gets right:

  • No additional database roundtrip. The renderer asks the data retrieval for the total row count anyway via getTotalRowCount(). Both that call and the title now go through a small memoization keyed by the filter data, and the title passes the same filter the renderer does (getFilter() ?? []), so the participants are counted once per request instead of twice. As a side effect the displayed number follows an active filter, which matches what the table shows.
  • The number is labelled with the existing language variable total instead of standing bare in parentheses, so it is clear what it denotes, visually and, since it is part of the table title, for screen readers.

@thojou

thojou commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Hey @lorenzgith,

thank you for the PR.

At the moment, I am not sure whether we can address the issue in the way you proposed.

There is a similar PR (#11633) that aims to display the total number of points in the table header, analogous to how your changes would display the total number of participants.

While this additional information is useful from a visual perspective, it introduces accessibility concerns, especially for screen reader users. Adding a total directly to a column header can make the relationship between the header, the total, and the individual cell values difficult to understand when navigating the table with assistive technologies.

As noted by the Accessibility Group in the related issue, complex tables are already challenging to use with screen readers. We should therefore first clarify which accessibility goal we want to achieve and whether there is a way to provide this information with sufficient context for all users.

For this reason, I do not think we should add the total directly to the table header for now.

Kind regards,
@thojou

@kergomard

Copy link
Copy Markdown
Contributor

Thanks also from my part and thanks to @thojou for the longer explanation. I will close this PR and we should continue the discussion in the PR by @thojou .

@kergomard kergomard closed this Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants