[DQT] Fix random scroll jump when deleting selected fields#10361
[DQT] Fix random scroll jump when deleting selected fields#10361arnav-makkar wants to merge 1 commit intoaces:mainfrom
Conversation
|
LGTM |
|
Thank you @adamdaudrich Could you please mark this PR as approved as well so that it can be merged. Same for all these other PRs that you've already reviewed: #10300 #10304 #10305 #10307 |
|
please assign me to review them |
|
I do not have write access so cannot do that. @driusan Please confirm |
|
@arnav-makkar This PR is already assigned to him. I'm not sure what's being asked of me. |
|
Hi @driusan, I just wanted to know why @adamdaudrich wasn't able to mark this as reviewed. A few PRs (#10300 #10304 #10305 #10307 #10311) have already been manually checked, but they’re still showing as “Review Required.” I wanted to clarify if there’s something pending on my side or if it’s related to permissions/workflow. Since these are all very minor changes, there’s absolutely no rush. I only wanted to understand how the review/approval process works in such cases. Thank you! 🙂 |
Brief summary of changes
Added
e.stopPropagation()to the delete click handler.Testing
Before:
Screen.Recording.2026-02-13.at.00.56.09.mov
After:
Screen.Recording.2026-02-13.at.01.02.41.mov
Link(s) to related issue(s)