Skip to content

Diff comment button debounce

Luke "Jared" Bennett requested to merge debounce-comment-button-render into master

What does this MR do?

Debounce the show, delay the hide and return early as possible for performance

Debounces the rendering of the comment button when mouseover a diff line.

Delays the hiding of the comment button when mouseleave a diff line.

Returns as early as possible during the rendering/showing/hiding of the comment button to reduce wasted computation.

Are there points in the code the reviewer needs to double check?

Why was this MR needed?

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Related to https://gitlab.com/gitlab-org/gitlab-ce/issues/30404

Merge request reports