Skip to content

Replace carousel item backgrounds with relevant image for portfolio item - #690

Open
conorheffron with Copilot wants to merge 12 commits into
mainfrom
copilot/replace-carousel-backgrounds
Open

Replace carousel item backgrounds with relevant image for portfolio item#690
conorheffron with Copilot wants to merge 12 commits into
mainfrom
copilot/replace-carousel-backgrounds

Conversation

Copilot AI commented May 23, 2026

Copy link
Copy Markdown
Contributor
  • Inspect current carousel implementation and existing validations
  • Run baseline frontend/backend tests/build to capture pre-change state
  • Implement minimal carousel background mapping from portfolio item data to relevant snapshots
  • Update/add focused frontend tests for the new background behavior
  • Run targeted frontend tests/build and capture a UI screenshot of the carousel
  • Run final automated validation and summarize results

Agent-Logs-Url: https://github.com/conorheffron/ironoc/sessions/f08cfaa0-f8c4-4152-9cca-5382913849bc

Co-authored-by: conorheffron <8218626+conorheffron@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the frontend portfolio carousel to display a more relevant background image per portfolio item by deriving the <img src> from either a provided image URL, a GitHub OpenGraph snapshot for GitHub repos, or a local color-based fallback.

Changes:

  • Added helper logic in ControlledCarousel to choose between direct img URLs, GitHub OpenGraph snapshot URLs, and local fallback images.
  • Updated the carousel rendering to use the new image-selection logic.
  • Extended ControlledCarousel tests to validate GitHub snapshot image selection and fallback behavior for non-GitHub links.

Reviewed changes

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

File Description
frontend/src/components/ControlledCarousel.js Adds URL validation + GitHub OpenGraph mapping and uses it for carousel item images.
frontend/src/components/tests/ControlledCarousel.test.js Adds a non-GitHub item and a test asserting GitHub snapshot vs fallback image behavior.

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

Comment thread frontend/src/components/ControlledCarousel.js Outdated
Comment thread frontend/src/components/ControlledCarousel.js
Comment thread frontend/src/components/ControlledCarousel.js
Comment thread frontend/src/components/ControlledCarousel.js
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@conorheffron
conorheffron marked this pull request as ready for review June 1, 2026 01:12
@conorheffron conorheffron changed the title [WIP] Replace carousel item backgrounds with relevant image for portfolio item Replace carousel item backgrounds with relevant image for portfolio item Jul 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

Replace carousel item backgrounds with relevant image for portfolio item

3 participants