refactor: replace deprecated matchMedia listeners with addEventListener

[ThetaLogN]

Resolves #3515

The useIsMobile hook currently uses addListener and removeListener, which are deprecated in modern browsers. This updates the hook to use the standard addEventListener('change', ...) API, improving code quality.

https://sonarcloud.io/project/issues?open=AZksw2CU_PPcrA-zHUTg&id=OWASP_Nest

Checklist

• Required: I followed the contributing workflow
• Required: I verified that my code works as intended and resolves the issue as described
• Required: I ran make check-test locally: all warnings addressed, tests passed
• I used AI for code, documentation, tests, or communication related to this PR

[coderabbitai]

Summary by CodeRabbit

• Refactor
  • Simplified mobile detection logic for more consistent event handling and reliable cleanup.
  • Removed legacy browser-specific fallback branches to reduce conditional complexity.
  • Preserved existing behavior tied to change events while improving maintainability and reducing potential edge-case differences across environments.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

Walkthrough

Refactored the useIsMobile hook to always use MediaQueryList.addEventListener('change') and removeEventListener, removing the Safari <14 addListener/removeListener fallback while keeping the initial handleChange invocation. (48 words)

Changes

Cohort / File(s)	Summary
Hook event listener modernization frontend/src/hooks/useIsMobile.ts	Replaced deprecated MediaQueryList.addListener/removeListener branch with unconditional addEventListener('change', ...)/removeEventListener(...); removed Safari <14 compatibility branch; preserved initial handleChange call.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: replacing deprecated matchMedia listeners (addListener/removeListener) with the standard addEventListener API.
Description check	✅ Passed	The description is directly related to the changeset, explaining the motivation for replacing deprecated APIs with addEventListener and referencing the resolved issue.
Linked Issues check	✅ Passed	The PR successfully addresses issue #3515 by replacing deprecated addListener/removeListener with addEventListener('change', ...) in the useIsMobile hook.
Out of Scope Changes check	✅ Passed	The changes are focused solely on refactoring the useIsMobile hook to use modern API patterns, with no unrelated modifications detected.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.48%. Comparing base (dfc96ef) to head (106ac1a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3531   +/-   ##
=======================================
  Coverage   85.48%   85.48%           
=======================================
  Files         461      461           
  Lines       14240    14240           
  Branches     1896     1896           
=======================================
  Hits        12173    12173           
  Misses       1688     1688           
  Partials      379      379           

Flag	Coverage Δ
backend	84.48% <ø> (ø)
frontend	88.26% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dfc96ef...106ac1a. Read the comment docs.

🚀 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.