Summary:

Update ScreenshotThumbnailView to use InlineEditorView.

Review Request #12968 — Created April 21, 2023 and submitted April 25, 2023, 11:42 a.m.

Information
Owner:	david
Repository:	Review Board
Branch:	release-6.x
Bugs:
Depends On:
Reviewers
Groups:	reviewboard
People:

Description

This is another change in the process of coverting from our old
$.ui.inlineEditor to the new(er) InlineEditorView. This updates the
screenshot thumbnail caption editor, which in practice will never
actually be used anymore.

Testing Done

Ran js-tests.

Commits

Summary
	Update ScreenshotThumbnailView to use InlineEditorView.

Screenshots

Issues

Description	From	Last Updated
We do these repeated property accesses over and over. Let's pull these out into a local variable, use it, and …	chipx86	April 21, 2023, 4:29 p.m.

flake8 passed.

JSHint passed.

```
Ship It!
```

reviewboard/static/rb/js/views/screenshotThumbnailView.es6.js (Diff revision 1)

Show all issues

We do these repeated property accesses over and over. Let's pull these out into a local variable, use it, and set it. Makes it more readable and avoids unnecessary accesses.

Commits:

	Summary
-		Update ScreenshotThumbnailView to use InlineEditorView.
+		Update ScreenshotThumbnailView to use InlineEditorView.

Diff:

Revision 2 (+82 -62)

Show changes

	reviewboard/static/rb/js/views/screenshotThumbnailView.es6.js
	reviewboard/static/rb/js/views/tests/reviewRequestEditorViewTests.es6.js
	reviewboard/static/rb/js/views/tests/screenshotThumbnailViewTests.es6.js

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to release-6.x (551cf88)