Skip to content

Image fix#92

Merged
SimonBurmer merged 4 commits intomainfrom
image-fix
Apr 9, 2026
Merged

Image fix#92
SimonBurmer merged 4 commits intomainfrom
image-fix

Conversation

@SimonBurmer
Copy link
Copy Markdown
Collaborator

@SimonBurmer SimonBurmer commented Apr 9, 2026

Summary by CodeRabbit

  • New Features

    • Added interactive detail panels for advisory board members with smooth animations.
    • Updated advisory board with named members and local imagery.
  • Changes

    • Advisory board is now always visible on the About Us page.
    • Optimized news card image display on the Home page.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
start-munich Ready Ready Preview, Comment Apr 9, 2026 1:02pm

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 9, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 02c41b23-e04f-4135-9389-f497f4591f0f

📥 Commits

Reviewing files that changed from the base of the PR and between 52dfdb3 and ac4e6e3.

📒 Files selected for processing (1)
  • app/home/HomeClient.tsx

📝 Walkthrough

Walkthrough

The PR updates the About Us advisory board section to use local images and named team members instead of generic entries, introduces an AdvisorDetail component with mount-based animation, and removes environment-variable control of advisory visibility. Additionally, Latest News image URLs are updated with new query parameters.

Changes

Cohort / File(s) Summary
Advisory Board Refactor
app/about-us/AboutUsContent.tsx
Replaced remote advisory data with named members and local images; added AdvisorDetail component with requestAnimationFrame-driven animation; hardcoded showAdvisoryBoard = true; modified grid selection to disable for members without bio; increased detail panel max-height to 800px; updated image sizing and object positioning.
News Image Updates
app/home/HomeClient.tsx
Updated LinkedIn thumbnail URLs for three Latest News cards, changing feedshare-shrink sizes and query parameter values.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

  • Feat/add adv board #89: Introduces identical AdvisorDetail component and AdvisoryMember type to AboutUsContent.tsx with same data and visibility changes.
  • Image fix #92: Modifies AboutUsContent.tsx with AdvisorDetail component and advisory board data replacement.
  • Fix: about us page #42: Updates About Us advisory/board content by replacing remote portraits with local image assets and adjusting detail behavior.

Poem

🐰 Local images hop into place,
Named advisors show their face,
RAF animates with grace,
No more env vars to chase,
Detail panels embrace the space! ✨

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

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.
Title check ❓ Inconclusive The title 'Image fix' is vague and does not clearly describe the main changes, which include advisory board restructuring, UI modifications, environment variable removal, and component additions beyond just image fixes. Consider a more descriptive title that captures the primary changes, such as 'Refactor advisory board with new detail component and image handling' or be more specific about which image fixes are being addressed.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

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

✨ Finishing Touches
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch image-fix

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.

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