Skip to content

fix: add title back to HeadMetadata#299

Merged
devksingh4 merged 1 commit intomainfrom
readd-title
Feb 15, 2026
Merged

fix: add title back to HeadMetadata#299
devksingh4 merged 1 commit intomainfrom
readd-title

Conversation

@devksingh4
Copy link
Member

@devksingh4 devksingh4 commented Feb 15, 2026

Summary by CodeRabbit

  • Bug Fixes

    • Fixed page title display to ensure "ACM @ UIUC" appears in browser tabs and window titles.
  • Tests

    • Added test assertion to verify page title renders correctly.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 15, 2026

No actionable comments were generated in the recent review. 🎉


Walkthrough

This PR adds page title support to the application. A <title> element is added to the HeadMetadata Astro component, deriving from an existing title prop that defaults to "ACM @ UIUC". A corresponding end-to-end test assertion is added to verify that the page title renders correctly with the expected value.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: add title back to HeadMetadata' clearly describes the main change: adding a title element to the HeadMetadata component, which is exactly what the changeset implements.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ 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
  • Commit unit tests in branch readd-title

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

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

@github-actions
Copy link

Deploying with Cloudflare Pages

Name Result
Last commit: 8ce0ae9
Preview URL: https://f22e645f.acmuiuc.pages.dev
Branch Preview URL: https://readd-title.acmuiuc.pages.dev

@devksingh4 devksingh4 merged commit bc32129 into main Feb 15, 2026
11 checks passed
@devksingh4 devksingh4 deleted the readd-title branch February 15, 2026 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant