Skip to content

Disable notebook tile image compression#4392

Merged
hlbmtc merged 1 commit intomainfrom
fix/disable-notebook-image-optimizations
Feb 19, 2026
Merged

Disable notebook tile image compression#4392
hlbmtc merged 1 commit intomainfrom
fix/disable-notebook-image-optimizations

Conversation

@hlbmtc
Copy link
Contributor

@hlbmtc hlbmtc commented Feb 19, 2026

Summary by CodeRabbit

  • Chores
    • Updated image rendering behavior in notebook tiles.

@hlbmtc hlbmtc deployed to testing_env February 19, 2026 00:12 — with GitHub Actions Active
@hlbmtc hlbmtc merged commit 91c1497 into main Feb 19, 2026
6 of 7 checks passed
@hlbmtc hlbmtc deleted the fix/disable-notebook-image-optimizations branch February 19, 2026 00:12
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 19, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

A single Image component prop was modified in the notebook tile component, replacing quality={100} with unoptimized to disable Next.js image optimization for that image element.

Changes

Cohort / File(s) Summary
Image Optimization Configuration
front_end/src/components/post_card/notebook_tile.tsx
Modified Image component to disable Next.js optimization by replacing quality={100} prop with unoptimized prop.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A single prop, so swift and clean,
Unoptimized—the image scene,
No quality bounds to constrain,
Just natural pixels, free and plain!
Hop along, this change is keen!

✨ 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 fix/disable-notebook-image-optimizations

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

Comments