aglinxinyuan opened a new pull request, #6303: URL: https://github.com/apache/texera/pull/6303
### What changes were proposed in this PR? Backport of apache/texera#6292 to `release/v1.2` (source commit `04838a1ce7`; the original PR is still open on `main`). Sidebar active-route highlighting (#3490): the **About** item is a plain `nz-menu-item` with a `[routerLink]` but **no `nzMatchRouter`**, so — unlike Your Work / Admin — it never enters ng-zorro's `.ant-menu-item-selected` state and isn't highlighted when `/about` is active. This adds `nzMatchRouter="true"` to it. Not a clean cherry-pick — adapted to `release/v1.2`: | main (#6292) | this backport | | --- | --- | | About item: add `nzMatchRouter="true"` | same | | Feedback item: add `nzMatchRouter="true"` | dropped — `release/v1.2` has no Feedback sidebar item | | spec: About + Feedback + reference (Your Work) cases | Feedback case dropped along with it | ### Any related issues, documentation, or discussions? Backport of #6292 (which closes #3490 on `main`). ### How was this PR tested? Unit tests (Vitest), run on `release/v1.2`: ``` corepack yarn ng test --watch=false --include="**/dashboard.component.spec.ts" ``` → **13/13 passing** (11 baseline + 2 new). Re-ran with the HTML fix reverted: the new About case fails, so the spec pins the fix. Prettier check clean on both files. No `release/*` label is added — this PR *is* the backport. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Fable 5) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
