Fixes a comment form bug in the diffs app
What does this MR do and why?
This merge request fixes a few bugs within the diffs virtual scroller:
- Fixes a bug the comment form would sometimes randomly scroll the diffs app around. This can be caused by the virtual scroller removing elements from the DOM and then re-adding them causing the
mounted
autofocus method call to happen when it shouldn't - Removes a JS error with the
ResizeObserver
- Fixes a bug where collapsed files would incorrectly get loaded when scrolling the diffs app
- Fixes a bug where the spacing between diff files would sometimes disappear when scrolling fast
Fixes a comment form bug in the diffs app
This fixes a bug in the diffs virtual scroller where the comment form would get autofocused when the virtual scroller removes and then re-adds the diff file component into the DOM.
Fixes a bug where the virtual scroller would incorrectly load a collapse file on scroll.
Fixes a bug where sometimes the diff files would have no space between them when scrolling fast.
Related Issues:
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.