Make sure region comment coordinates are rounded to integers.

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

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.

Commits

Files