Filip/matrix-spec
1
0
Fork 0
mirror of https://github.com/matrix-org/matrix-spec synced 2026-04-30 14:14:09 +02:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Only use plural of 'results' when actually necessary

Browse source 
Signed-off-by: Johannes Marbach <n0-0ne+github@mailbox.org>
This commit is contained in:
Johannes Marbach 2026-03-20 13:44:11 +01:00
parent 9edb9b3e5b
commit 15f36d1934
1 changed files with 11 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
assets/js/offline-search.js
View file

@ -155,7 +155,7 @@ search backend.
.text(r.meta.title)) .text(r.meta.title))
.append($("<span>") .append($("<span>")
.addClass("text-body-secondary") .addClass("text-body-secondary")
.text(` – ${r.sub_results.length} result(s)`))); .text(` – ${r.sub_results.length} ${resultsString(r.sub_results.length)}`)));
// Render the first 3 subresults per page and wrap the rest // Render the first 3 subresults per page and wrap the rest
// in a collapsed container. // in a collapsed container.
@ -164,6 +164,7 @@ search backend.
r.sub_results.forEach((s, index_s) => { r.sub_results.forEach((s, index_s) => {
if (index_s === LIMIT) { if (index_s === LIMIT) {
const num_hidden_results = r.sub_results.length - index_s;
const wrapper_id = `collapssible-subresults-${index_r}`; const wrapper_id = `collapssible-subresults-${index_r}`;
const $action = $("<span>").text("▶ Show"); const $action = $("<span>").text("▶ Show");
const $expander = $("<a>") const $expander = $("<a>")
@ -174,7 +175,7 @@ search backend.
.attr("aria-expanded", "false") .attr("aria-expanded", "false")
.attr("aria-controls", wrapper_id) .attr("aria-controls", wrapper_id)
.append($action) .append($action)
.append($("<span>").text(` ${r.sub_results.length - index_s} more result(s) from ${r.meta.title}`)); .append($("<span>").text(` ${num_hidden_results} more ${resultsString(num_hidden_results)} from ${r.meta.title}`));
$searchResultBody.append($("<p>").append($expander)); $searchResultBody.append($("<p>").append($expander));
$wrapper = $("<div>") $wrapper = $("<div>")
@ -219,3 +220,11 @@ search backend.
}; };
}); });
})(jQuery); })(jQuery);
//
// Helpers
//
const resultsString = (n) => {
return n > 1 ? "results" : "result";
};