Skip to content

chore: remove tua-body-scroll-lock local patch#772

Merged
alexiglesias93 merged 1 commit intomasterfrom
remove-tua-body-scroll-lock-override
Dec 16, 2025
Merged

chore: remove tua-body-scroll-lock local patch#772
alexiglesias93 merged 1 commit intomasterfrom
remove-tua-body-scroll-lock-override

Conversation

@alexiglesias93
Copy link
Member

No description provided.

@vercel
Copy link

vercel bot commented Dec 16, 2025

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

Project Deployment Review Updated (UTC)
attributes Building Building Preview Dec 16, 2025 9:21am

Copy link
Contributor

Copilot AI left a comment

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 removes a local patch for the tua-body-scroll-lock package and upgrades it from version 1.6.1 to 1.6.2. The patch previously modified the order of style assignments in the library's setOverflowForPc function, moving the overflow and boxSizing style assignments to occur after the paddingRight calculation. The upgrade to version 1.6.2 suggests this fix has been incorporated upstream.

Key Changes:

  • Removed the local patch file for tua-body-scroll-lock@1.6.1
  • Updated dependency version from ^1.6.1 to ^1.6.2 in the scrolldisable package
  • Cleaned up patch configuration from pnpm workspace and lock files

Reviewed changes

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

Show a summary per file
File Description
pnpm-workspace.yaml Removed patchedDependencies configuration section
pnpm-lock.yaml Removed patch references and updated dependency version from 1.6.1 to 1.6.2
patches/tua-body-scroll-lock@1.6.1.patch Deleted the entire patch file that modified style assignment ordering
packages/scrolldisable/package.json Updated tua-body-scroll-lock dependency from ^1.6.1 to ^1.6.2
.changeset/real-doors-lie.md Added changeset documenting the removal of the local patch

The changes are clean, consistent, and properly documented. All references to the patched version have been removed, and the dependency has been correctly updated across all relevant files. No issues were identified during the review.

Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

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

@alexiglesias93 alexiglesias93 merged commit 9338171 into master Dec 16, 2025
9 checks passed
@alexiglesias93 alexiglesias93 deleted the remove-tua-body-scroll-lock-override branch December 16, 2025 09:24
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