Part of Phase 3.2 (task 1): factor out keyboard shortcut setup into a separate function in the TracePage

[Parship12]

Which problem is this PR solving?

• Resolves part of [🧑‍🎓 LFX]: Upgrading Core Routing and State Management #3313
• Resolves part of [UI Refactor] Migrate TracePage to Functional Component #3381

Description of the changes

• Factor out keyboard shortcut setup from componentDidMount into _setupKeyboardShortcuts.
• This is a readability refactor, no logic was changed, just reorganized.

How was this change tested?

• Unit tests
• keyboard shortcuts working correctly

Checklist

• I have read https://github.com/jaegertracing/jaeger/blob/main/CONTRIBUTING_GUIDELINES.md
• I have signed all commits
• I have added unit tests for the new functionality
• I have run lint and test steps successfully: make lint test

AI Usage in this PR (choose one)

See AI Usage Policy.

• None: No AI tools were used in creating this PR
• Light: AI provided minor assistance (formatting, simple suggestions)
• Moderate: AI helped with code generation or debugging specific parts
• Heavy: AI generated most or all of the code changes

[Copilot]

Pull request overview

Refactors TracePageImpl keyboard shortcut initialization by extracting the setup logic out of componentDidMount() into a dedicated helper method, improving readability while keeping behavior the same.

Changes:

• Moved keyboard shortcut wiring from componentDidMount() into _setupKeyboardShortcuts().
• Updated the existing unit test to hook into _setupKeyboardShortcuts() rather than componentDidMount().

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
packages/jaeger-ui/src/components/TracePage/index.tsx	Extracts keyboard shortcut setup into _setupKeyboardShortcuts() and calls it from componentDidMount().
packages/jaeger-ui/src/components/TracePage/index.test.js	Adjusts the keyboard-shortcut setup test to patch _setupKeyboardShortcuts() for verifying adjViewRange wiring.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.15%. Comparing base (9edb560) to head (219efd9).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3609   +/-   ##
=======================================
  Coverage   89.15%   89.15%           
=======================================
  Files         304      304           
  Lines        9725     9727    +2     
  Branches     2505     2593   +88     
=======================================
+ Hits         8670     8672    +2     
- Misses       1051     1052    +1     
+ Partials        4        3    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.