• 
      

    Make sure region comment coordinates are rounded to integers.

    Review Request #13990 — Created June 19, 2024 and submitted

    Information

    Review Board
    release-7.x

    Reviewers

    When saving the x, y, width and height coordinates for a region comment,
    we previously would not round these and they would sometimes include
    many decimal places.
    
    This caused a bug with serializing comments that are made on the same
    region (e.g. creating a new comment on top of an existing one). We represent
    these types of comments as a single comment box that shows the number of
    comments in the corner. When serializing the comments, we use a key
    built of the x, y, width and height so that comments on the same region
    get mapped to the same box.
    
    Without rounding the decimal places for coordinates, there could be slight
    differences in the coordinate values for the same visual region.
    Comments created on top of one another would sometimes fail to be
    serialized into the same comment box due to the decimal differences.
    
    This change ensures that the coordinate values are rounded.

    Tested creating region comments on top of one another, saw that they
    were properly placed into the same box instead of two separate
    overlapping boxes.

    Summary ID
    Make sure region comment coordinates are rounded to integers.
    When saving the x, y, width and height coordinates for a region comment, we previously would not round these and they would sometimes include many decimal places. This caused a bug with serializing comments that are made on the same region (e.g. creating a new comment on top of an existing one). We represent these types of comments as a single comment box that shows the number of comments in the corner. When serializing the comments, we use a key built of the x, y, width and height so that comments on the same region get mapped to the same box. Without rounding the decimal places for coordinates, there could be slight differences in the coordinate values for the same visual region. Comments created on top of one another would sometimes fail to be serialized into the same comment box due to the decimal differences. This change ensures that the coordinate values are rounded.
    ee0ca6907eb8eca1506e06cdd72857b7560652b6
    david
    1. Ship It!
    2. 
        
    chipx86
    1. Ship It!
    2. 
        
    maubin
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-7.x (863a49e)