Update UI for the comments hint.
Review Request #13672 — Created March 27, 2024 and submitted
This change ports the comments hint model and view to TypeScript, and
updates the UI to include the new commit range information. Along with
this, the implementation has been simplified a bit to attach event
handlers individually to each item in the list, rather than having a
generic event handler and setting data on the list items.
- Ran js-tests.
- Viewed a diff while having a review that had comments on lots of
different revisions/interdiffs/commits. Saw that the comment hint
correctly showed all of the revisions that had draft comments, and
that clicking on all of them correctly loaded the associated revision.
Summary | ID |
---|---|
6ba2f4f635f172e1b3df75acd7e2eacb94407e4c |
Description | From | Last Updated |
---|---|---|
Is the babel-plugin-htm dependency update with all the package-lock.json and package.json changes supposed to be part of this change? |
maubin | |
This one can be an import type. |
chipx86 | |
Unless I missed runtime usage, this can be import type. |
chipx86 | |
We should avoid h1, as that's the main title foe the page. h2 should be the (in theory) the review … |
chipx86 | |
Let's pull out this.model. We access it a lot. |
chipx86 |
Change Summary:
Fix a typo.
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+644 -366) |
Checks run (2 succeeded)
-
-
reviewboard/static/rb/js/reviews/models/diffViewerPageModel.ts (Diff revision 2) This one can be an
import type
. -
reviewboard/static/rb/js/reviews/views/diffCommentsHintView.ts (Diff revision 2) Unless I missed runtime usage, this can be
import type
. -
reviewboard/static/rb/js/reviews/views/diffCommentsHintView.ts (Diff revision 2) We should avoid
h1
, as that's the main title foe the page.h2
should be the (in theory) the review request summary. This should probably beh3
orh4
. -
reviewboard/static/rb/js/reviews/views/diffCommentsHintView.ts (Diff revision 2) Let's pull out
this.model
. We access it a lot.
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 3 (+704 -28526) |
Checks run (2 succeeded)
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 4 (+656 -366) |