fix(desktop): remove "View PR on GitHub" tooltip from ChangesHeader

[chasemcdo]

Summary

• Remove the "View PR on GitHub" tooltip from the PR status link in the ChangesHeader component
• The tooltip was overlapping with nearby buttons, making them difficult to click
• The PR icon and number are already intuitive, so the tooltip didn't add significant value

Test plan

• Open a workspace with an associated PR
• Verify the PR number/icon link still opens GitHub when clicked
• Verify no tooltip appears on hover
• Verify nearby buttons (stash, view mode, refresh) are easier to click

Resolves #922

Summary by CodeRabbit

• Style
  • Removed tooltip from PR status link in the Changes view.

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

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR removes a Tooltip wrapper around the PR status link in the ChangesHeader component. The link now returns a plain anchor element with the same href, target, rel, and class names, while loading and null-PR states remain unchanged.

Changes

Cohort / File(s)	Summary
PR Status Link Unwrapping apps/desktop/src/renderer/screens/main/components/WorkspaceView/Sidebar/ChangesView/components/ChangesHeader/ChangesHeader.tsx	Removed Tooltip wrapper around PR status link; link now renders as plain anchor element with preserved attributes (href, target, rel, class names). Loading and null states unchanged.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• feat(desktop): polish UI for changes sidebar and workspace management #612: Modifies the PR status link indicator in the Changes view header—relevant as this PR removes a UI wrapper around the same component.
• feat(desktop): improve side-by-side diff view UI and fix content reset #836: Alters the ChangesHeader component with header control additions—related as this PR modifies the same component file.
• feat(desktop): add expanded changes sidebar view #858: Changes tooltip-related UI in the ChangesHeader component—relevant as this PR simplifies tooltip usage in the same file.

Poem

🐰 A tooltip takes its final bow,
The PR link stands bare now,
No wrapper, no fluff in sight,
Just clean, honest markup—right!
Simplicity wins the day, meow~

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: removing a tooltip from the PR status link in ChangesHeader.
Description check	✅ Passed	The description provides a clear summary, test plan, and issue reference, though it deviates from the template structure with a non-standard format.

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

✨ Finishing touches

• 📝 Generate docstrings

---

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

[Kitenite]

Works for me. Thanks @chasemcdo !