This change has been marked as completed.

Describe the completed change (optional):

Pushed to release-6.x (fb8c5f2)

Summary

Convert the ReviewReplyEditorView to use InlineEditorView.

Review Request #12964 — Created April 20, 2023 and submitted 2 years ago

Information

Owner

david*

Repository

Review Board

Branch

release-6.x

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description*

We've had a long-term goal of replacing all our uses of the old
$.inlineEditor with the new InlineEditorView. This change updates
the review reply for that.

Testing Done


Verified that creating new reply comments and editing existing reply

  comments worked correctly, building the editor with the correct

  content and saving/cancelling as expected.
Ran js-tests.

Commits

Summary	ID
Convert the ReviewReplyEditorView to use InlineEditorView. We've had a long-term goal of replacing all our uses of the old `$.inlineEditor` with the new `InlineEditorView`. This change updates the review reply for that. Testing Done: - Verified that creating new reply comments and editing existing reply comments worked correctly, building the editor with the correct content and saving/cancelling as expected. - Ran js-tests.	857443a09373a913d0d60f2d6140708a0b84a516

Summary

Convert the ReviewReplyEditorView to use InlineEditorView.

We've had a long-term goal of replacing all our uses of the old `$.inlineEditor` with the new `InlineEditorView`. This change updates the review reply for that. Testing Done: - Verified that creating new reply comments and editing existing reply comments worked correctly, building the editor with the correct content and saving/cancelling as expected. - Ran js-tests.

857443a09373a913d0d60f2d6140708a0b84a516

Issues

Description	From	Last Updated
Let's keep the elements grouped together separately from the view.	chipx86	2 years ago
We do these repeated property accesses over and over. Let's pull these out into a local variable, use it, and …	chipx86	2 years ago
There are no open issues

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

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

The issue has been resolved. Show all issues

Let's keep the elements grouped together separately from the view.

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

The issue has been resolved. 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	ID
	Convert the ReviewReplyEditorView to use InlineEditorView. We've had a long-term goal of replacing all our uses of the old `$.inlineEditor` with the new `InlineEditorView`. This change updates the review reply for that. Testing Done: - Verified that creating new reply comments and editing existing reply comments worked correctly, building the editor with the correct content and saving/cancelling as expected. - Ran js-tests.	2832f783249153d2916bccda0d22a048639c93e1
	Convert the ReviewReplyEditorView to use InlineEditorView. We've had a long-term goal of replacing all our uses of the old `$.inlineEditor` with the new `InlineEditorView`. This change updates the review reply for that. Testing Done: - Verified that creating new reply comments and editing existing reply comments worked correctly, building the editor with the correct content and saving/cancelling as expected. - Ran js-tests.	857443a09373a913d0d60f2d6140708a0b84a516

Diff:

Revision 2 (+90 -86)

Show changes

reviewboard/static/rb/js/reviewRequestPage/views/reviewReplyEditorView.es6.js

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Status:: Completed
Change Summary:: Pushed to release-6.x (fb8c5f2)