Summary

Convert js/ui to ES6

Review Request #8049 — Created March 10, 2016 and submitted March 11, 2016, 9:37 a.m.

Information

Owner

david

Repository

Review Board

Branch

release-2.6.x

Bugs

Depends On

Commit

31ef8ac...

Reviewers

Groups

reviewboard

People

Description

This change converts the views in js/ui/views to ES6. While I was in here, I
went through and updated our docstrings to the modern format.

Testing Done

Ran js-tests.

Issues

Description	From	Last Updated
These methods could also be replaced with the methodName(args) sugar	brennie	March 11, 2016, 9:37 a.m.
Would you mind completing this doc comment?	brennie	March 11, 2016, 9:37 a.m.

Tool: Pyflakes
Processed Files:
    reviewboard/staticbundles.py

Ignored Files:
    reviewboard/static/rb/js/ui/views/dialogView.es6.js
    reviewboard/static/rb/js/ui/views/dialogView.js
    reviewboard/static/rb/js/ui/views/splitButtonView.es6.js
    reviewboard/static/rb/js/ui/views/textEditorView.es6.js



Tool: PEP8 Style Checker
Processed Files:
    reviewboard/staticbundles.py

Ignored Files:
    reviewboard/static/rb/js/ui/views/dialogView.es6.js
    reviewboard/static/rb/js/ui/views/dialogView.js
    reviewboard/static/rb/js/ui/views/splitButtonView.es6.js
    reviewboard/static/rb/js/ui/views/textEditorView.es6.js

Ship it!

Minor nit: "docstring" -> "doc comment' in your description.

reviewboard/static/rb/js/ui/views/splitButtonView.es6.js (Diff revision 1)
The issue has been resolved. Show all issues
```
These methods could also be replaced with the methodName(args) sugar
```
reviewboard/static/rb/js/ui/views/splitButtonView.es6.js (Diff revision 1)
The issue has been resolved. Show all issues
```
Would you mind completing this doc comment?
```

Status: Closed (submitted)

Change Summary:

Pushed to release-2.6.x (ab0cdce)