Skip to content

chore: Revert "fix: offset calculation to handle transformed elements with translateY or translateX in scrollIntoView (#7717)"#9592

Merged
LFDanLu merged 2 commits intomainfrom
revert_4a815c3
Feb 3, 2026
Merged

chore: Revert "fix: offset calculation to handle transformed elements with translateY or translateX in scrollIntoView (#7717)"#9592
LFDanLu merged 2 commits intomainfrom
revert_4a815c3

Conversation

@LFDanLu
Copy link
Member

@LFDanLu LFDanLu commented Feb 3, 2026

This reverts commit 4a815c3.

Closes

✅ Pull Request Checklist:

  • Included link to corresponding React Spectrum GitHub Issue.
  • Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • Filled out test instructions.
  • Updated documentation (if it already exists for this component).
  • Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

Try testing v3 TableView's many rows story and arrow down through the rows. The horizontal scroll position shouldn't move at all

🧢 Your Project:

RSP

…ranslateY or translateX in scrollIntoView (#7717)"

This reverts commit 4a815c3.
@LFDanLu LFDanLu changed the title Revert "fix: offset calculation to handle transformed elements with translateY or translateX in scrollIntoView (#7717)" chore: Revert "fix: offset calculation to handle transformed elements with translateY or translateX in scrollIntoView (#7717)" Feb 3, 2026
devongovett
devongovett previously approved these changes Feb 3, 2026
@LFDanLu LFDanLu added the release label Feb 3, 2026
@LFDanLu LFDanLu enabled auto-merge February 3, 2026 23:25
@rspbot
Copy link

rspbot commented Feb 3, 2026

@LFDanLu LFDanLu added this pull request to the merge queue Feb 3, 2026
Merged via the queue into main with commit 47858e0 Feb 3, 2026
29 checks passed
@LFDanLu LFDanLu deleted the revert_4a815c3 branch February 3, 2026 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants