Codeline Search

[chidozieononiwu]

• Add codeline search feature to APIView.
• Allows search across lazy loaded items.

[Copilot]

Copilot reviewed 5 out of 14 changed files in this pull request and generated 1 comment.

Files not reviewed (9)

• src/dotnet/APIView/APIViewWeb/Client/css/shared/theme-colors.scss: Language not supported
• src/dotnet/APIView/ClientSPA/src/app/_components/code-panel/code-panel.component.html: Language not supported
• src/dotnet/APIView/ClientSPA/src/app/_components/code-panel/code-panel.component.scss: Language not supported
• src/dotnet/APIView/ClientSPA/src/app/_components/review-page-options/review-page-options.component.html: Language not supported
• src/dotnet/APIView/ClientSPA/src/app/_components/review-page-options/review-page-options.component.scss: Language not supported
• src/dotnet/APIView/ClientSPA/src/app/_components/review-page/review-page.component.html: Language not supported
• src/dotnet/APIView/ClientSPA/src/ng-prime-overrides.scss: Language not supported
• src/dotnet/APIView/ClientSPA/src/app/_modules/shared/review-page-layout.module.ts: Evaluated as low risk
• src/dotnet/APIView/ClientSPA/src/app/_modules/shared/shared-app.module.ts: Evaluated as low risk

Comments suppressed due to low confidence (1)

src/dotnet/APIView/ClientSPA/src/app/_models/codeLineSearchInfo.ts:4

• [nitpick] The property name matchedNodeIds could be more descriptive. Consider renaming it to matchedNodeIdSet to clearly indicate it is a Set.

matchedNodeIds: Set<string>

[maririos]

Could you add a video of how it works and tests for the search section?