Disallow clicking on the "No issues" text in the issue summary table.
Review Request #12060 — Created Feb. 15, 2022 and submitted
The issue summary table had a mouse cursor and click event enabled for
every row, and this included the row with the "There are no issues" text
we show when the active filter turns up 0 results. Clicking this would
take the user to a 404 page with "undefined" at the end of the URL.This change makes the click handler more specific, only listening to
events on rows with an issue ID set.The mouse cursor for tabular fields is now the default for text, rather
than the "it's clickable" pointer. This is a more correct default. The
issue summary table specializes that, setting the cursor only for rows
with an issue ID.
Unit tests pass.
Tested the mouse cursor on issue rows and non-issue rows. Verified that
the cursor only indicated a row was clickable for issue rows.Tested clicking issue rows and non-issue rows. Verified that the click
was only handled for issue rows.
Summary | ID |
---|---|
803e82944bb5d4198551a2e0bdfa4a60c27cc988 |