Skip to content

Conversation

@jfox-box
Copy link
Contributor

@jfox-box jfox-box commented Oct 23, 2025

This issue with scrolling in the All Filters sidepanel was fixed in a recent version of blueprint. This PR upgrades to the new version to apply the fix.

Fixed scrolling:

Screen.Recording.2025-10-23.at.3.50.55.PM.mov

@jfox-box jfox-box requested a review from a team as a code owner October 23, 2025 22:51
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 23, 2025

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • src/elements/content-sidebar/__tests__/__snapshots__/SidebarFileProperties.test.js.snap is excluded by !**/*.snap

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

Updated Box blueprint package dependency versions in package.json. @box/blueprint-web bumped to ^12.89.2 and @box/blueprint-web-assets to ^4.75.0 across both dependencies and peerDependencies sections.

Changes

Cohort / File(s) Change Summary
Dependency Version Updates
package.json
Updated @box/blueprint-web from ^12.78.0/^12.69.0 to ^12.89.2 and @box/blueprint-web-assets from ^4.69.0 to ^4.75.0 in both dependencies and peerDependencies

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

ready-to-merge

Suggested reviewers

  • tjuanitas
  • jpan-box
  • reneshen0328

Poem

🐰 A blueprint so shiny and new,
Version bumps bring fresh morning dew,
Dependencies aligned with care,
Box UI elements in the air! ✨

Pre-merge checks and finishing touches

❌ 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%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The pull request title "fix(content-picker): Upgrade blueprint to fix scrolling issue" is directly related to the main changeset. The raw_summary confirms that the changes consist of upgrading blueprint dependency versions (@box/blueprint-web to ^12.89.2 and @box/blueprint-web-assets to ^4.75.0) across both dependencies and peerDependencies in package.json. The title clearly and concisely communicates the primary change without being vague or overly broad, making it easy for reviewers to understand the purpose at a glance.
Description Check ✅ Passed The pull request description includes a clear explanation of the fix ("This issue with scrolling in the All Filters sidepanel was fixed in a recent version of blueprint. This PR upgrades to the new version to apply the fix."), along with a visual demonstration via an asset link. The description also properly includes the complete template comment block containing instructions about the ready-to-merge label, approval requirements, reviewer handling, merge queue guidance, and troubleshooting information. The description is specific rather than generic, providing context about what is being fixed and why the dependency upgrade is necessary. All required template information is present in the provided description.

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.

tjuanitas
tjuanitas previously approved these changes Oct 23, 2025
@mergify mergify bot added the queued label Oct 24, 2025
@mergify mergify bot merged commit d0ce8a6 into master Oct 24, 2025
10 checks passed
@mergify mergify bot deleted the bump-blueprint-12.89.2 branch October 24, 2025 17:18
@mergify mergify bot removed the queued label Oct 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants