Fix documentation links.

Links to documentation on Read the Docs for subjects, metrics, or sources with hyphens in their name wouldn't scroll to the right location.

Fixes #10843.

components/frontend/src/utils.jsx

@@ -421,7 +421,7 @@ export function dropdownOptions(options) {
 
 export function referenceDocumentationURL(name) {
     // Return a URL to the documentation for the metric/subject/source name
-    const slug = `${name?.toLowerCase().replaceAll(" ", "-").replaceAll(/[()/]/g, "")}`
+    const slug = `${name?.toLowerCase().replaceAll(" ", "-").replaceAll("_", "-").replaceAll(/[()/]/g, "")}`
     return `${DOCUMENTATION_URL}/reference.html#${slug}`
 }
 

components/frontend/src/utils.test.jsx

@@ -1,6 +1,7 @@
 import { EDIT_ENTITY_PERMISSION, EDIT_REPORT_PERMISSION } from "./context/Permissions"
 import { defaultDesiredResponseTimes } from "./defaults"
 import {
+    DOCUMENTATION_URL,
     addCounts,
     capitalize,
     getMetricResponseDeadline,
@@ -19,6 +20,7 @@ import {
     niceNumber,
     nrMetricsInReport,
     nrMetricsInReports,
+    referenceDocumentationURL,
     scaledNumber,
     sortWithLocaleCompare,
     sum,
@@ -514,3 +516,17 @@ it("returns whether a metric's measurement is outdated", () => {
     // A metric with an outdated measurement is outdated:
     expect(isMeasurementOutdated({ latest_measurement: { outdated: true } })).toBe(true)
 })
+
+it("returns the reference documentation URL", () => {
+    const url = `${DOCUMENTATION_URL}/reference.html`
+    // Simple name
+    expect(referenceDocumentationURL("name")).toBe(`${url}#name`)
+    // Name with spaces
+    expect(referenceDocumentationURL("name with space")).toBe(`${url}#name-with-space`)
+    // Name with underscores
+    expect(referenceDocumentationURL("name_with_underscore")).toBe(`${url}#name-with-underscore`)
+    // Name with brackets
+    expect(referenceDocumentationURL("(bracketed)")).toBe(`${url}#bracketed`)
+    // Name with forward slash
+    expect(referenceDocumentationURL("forward/slash")).toBe(`${url}#forwardslash`)
+})

docs/src/changelog.md

@@ -12,6 +12,12 @@ If your currently installed *Quality-time* version is not the latest version, pl
 
 <!-- The line "## <square-bracket>Unreleased</square-bracket>" is replaced by the release/release.py script with the new release version and release date. -->
 
+### [Unreleased]
+
+### Fixed
+
+- Links to documentation on Read the Docs for subjects, metrics, or sources with hyphens in their name wouldn't scroll to the right location. Fixes [#10843](https://github.com/ICTU/quality-time/issues/10843).
+
 ## v5.25.0 - 2025-02-14
 
 ### Added