Skip to content

review: 3189#3205

Closed
kkafar wants to merge 4 commits intomainfrom
@kkafar/review-3189-2
Closed

review: 3189#3205
kkafar wants to merge 4 commits intomainfrom
@kkafar/review-3189-2

Conversation

@kkafar
Copy link
Member

@kkafar kkafar commented Sep 8, 2025

Description

Suggested review changes for #3189.

See discussion under #3189 for reasoning.

Changes

  1. Prevented callback scheduling when there is no need for one,
  2. Added (IMO) necessary comments.

Test code and steps to reproduce

Checklist

@kkafar kkafar changed the base branch from main to @kmichalikk/animation-duration-transparent-modal-3074 September 8, 2025 14:01
@kmichalikk kmichalikk force-pushed the @kmichalikk/animation-duration-transparent-modal-3074 branch from 434c86e to 2cf2d6a Compare September 9, 2025 07:47
Base automatically changed from @kmichalikk/animation-duration-transparent-modal-3074 to main September 9, 2025 08:16
@kmichalikk
Copy link
Contributor

Added to #3189

@kmichalikk kmichalikk closed this Sep 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants