Skip to content

Commit 7728e10

Browse files
Remove historical-behavior references from useFollowActionBadgeTarget comments
Co-authored-by: Aimane Chnaif <aimane-chnaif@users.noreply.github.com>
1 parent dfdc14f commit 7728e10

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

src/pages/inbox/report/useFollowActionBadgeTarget.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ type UseFollowActionBadgeTargetParams = {
3030

3131
/**
3232
* When the action-badge target is resolved (e.g. the user approves/pays/submits an older report preview), it advances to the next
33-
* preview requiring action. This hook scrolls down to follow it immediately on action, rather than waiting for the resolve animation.
33+
* preview requiring action. This hook scrolls down to follow it immediately on action.
3434
*/
3535
function useFollowActionBadgeTarget({
3636
isProduction,
@@ -59,8 +59,8 @@ function useFollowActionBadgeTarget({
5959
if (Navigation.getTopmostReportId() !== reportID || !!Navigation.getReportRHPActiveRoute()) {
6060
return;
6161
}
62-
// Scroll to the next target on the next frame so the forward-scroll starts as soon as the user acts, rather than waiting for
63-
// the resolve animation. The resolved preview keeps animating in place while the list scrolls.
62+
// Scroll to the next target on the next frame so the forward-scroll starts as soon as the user acts. The resolved preview
63+
// keeps animating in place while the list scrolls.
6464
const animationFrameID = requestAnimationFrame(() => scrollToActionBadgeTargetRef.current());
6565
return () => cancelAnimationFrame(animationFrameID);
6666
// eslint-disable-next-line react-hooks/exhaustive-deps

0 commit comments

Comments
 (0)