• 
      

    Allow confetti to bounce off the vertical sides of a platform.

    Review Request #15093 — Created June 4, 2026 and updated

    Information

    Review Board
    release-8.x

    Reviewers

    The confetti view was built with the assumption that confetti would fall
    onto a completely flat surface, and chose not to handle pieces bouncing
    off the sides of a platform. An upcoming change will add confetti to
    our trophy banners, with the confetti falling onto the review request
    box. The box has a gap between the review request actions and the
    reviews/diff tabs. In order for the confetti to nicely fall on a surface
    like that, we need to update it to handle collisions with vertical
    platform sides.

    This change makes the confetti bounce off of the platforms sides when
    colliding with them laterally. The confetti will eventually land on the
    next platform that it falls to.

    Used in an upcoming change, checked that bouncing off of vertical
    walls looks nice.

    Summary ID
    Allow confetti to bounce off the vertical sides of a platform.
    The confetti view was built with the assumption that confetti would fall onto a completely flat surface, and chose not to handle pieces bouncing off the sides of a platform. An upcoming change will add confetti to our trophy banners, with the confetti falling onto the review request box. The box has a gap between the review request actions and the reviews/diff tabs. In order for the confetti to nicely fall on a surface like that, we need to update it to handle collisions with vertical platform sides. This change makes the confetti bounce off of the platforms sides when colliding with them laterally. The confetti will eventually land on the next platform that it falls to.
    50d632c917ef7f1a277177ab161a640c53c500ff
    Description From Last Updated

    The comparisons should use === for strict comparison.

    chipx86 chipx86
    Checks run (2 succeeded)
    flake8 passed.
    JSHint passed.
    chipx86
    1. 
        
    2. Show all issues

      The comparisons should use === for strict comparison.

    3.