(WIP) Diff Comment Reply Pop Up Dialog

Review Request #10315 — Created Nov. 8, 2018 and updated

Information

Review Board
release-4.0.x
e90a986...

Reviewers

Set up the view class for handling the popup dialog.
Need to remove old tooltip popup.
Handling basic popup functionality.
Handles comment selection from commentlist.
Selecting a comment will expand it to fill the dialog.
Replying in the dialog box (WIP) - Text field with action
buttons implemented, needs styling and reply posting.
Have issuebar inserted into comment when expanded.
Need to query user data from a comment to get avatars.
UI Design Aesthetics (WIP).


 
Description From Last Updated

There are a couple of places in your code with trailing whitespace

bolariinwabolariinwa

Col: 21 Expected '===' and instead saw '=='.

reviewbotreviewbot

Col: 7 'CommentTitleBarView' was used before it was defined.

reviewbotreviewbot

Col: 11 'test' is defined but never used.

reviewbotreviewbot

Col: 144 Missing semicolon.

reviewbotreviewbot

Col: 34 Missing semicolon.

reviewbotreviewbot

Col: 11 'urls' is defined but never used.

reviewbotreviewbot

Col: 7 'CommentTitleBarView' was used before it was defined.

reviewbotreviewbot

Col: 11 'timeStamp' is defined but never used.

reviewbotreviewbot

Col: 11 'test' is defined but never used.

reviewbotreviewbot

Col: 69 Missing semicolon.

reviewbotreviewbot

Since this is an ES6 file you should be able to put a trailing comma after '_onMouseLeave'

MalcolmMalcolm

There seems to be some extra white space here. In addition to that on line 57 and 58, the ending …

MalcolmMalcolm

Col: 11 'urls' is defined but never used.

reviewbotreviewbot

Col: 7 'CommentTitleBarView' was used before it was defined.

reviewbotreviewbot

Col: 11 'timeStamp' is defined but never used.

reviewbotreviewbot

Col: 26 ['viewEl'] is better written in dot notation.

reviewbotreviewbot

Col: 15 'reviewRequestEditor' is defined but never used.

reviewbotreviewbot

Col: 7 'ReplyTextFieldView' is defined but never used.

reviewbotreviewbot

Col: 11 'urls' is defined but never used.

reviewbotreviewbot

Col: 16 Duplicate key 'comment'.

reviewbotreviewbot

Col: 38 ['diffCommentsData'] is better written in dot notation.

reviewbotreviewbot

Col: 38 ['rView'] is better written in dot notation.

reviewbotreviewbot

Col: 7 'CommentTitleBarView' was used before it was defined.

reviewbotreviewbot

Col: 15 'commentIssueManager' is defined but never used.

reviewbotreviewbot

Col: 15 'interactive' is defined but never used.

reviewbotreviewbot

Col: 15 'commentID' is defined but never used.

reviewbotreviewbot
Checks run (1 failed, 1 succeeded)
flake8 passed.
JSHint failed.

JSHint

cdkushni
Review request changed
Description:
~  

Started by setting up the view class for handling the popup dialog.

~   Working on methods of handling multiple comment popup dialogs.

  ~

Set up the view class for handling the popup dialog.

  ~ Handling basic popup functionality(WIP).
  + Handles comment selection from commentlist.
  + Selecting a comment will expand it to fill the dialog.
  + Replying in the dialog box (WIP).
  + UI Design Aesthetics (WIP).

Commit:
903f486f4981cb63e74c157f0fe383ab8853e611
6b180fb93d3c6aa45803549283f51a4bef08bb3c

Checks run (1 failed, 1 succeeded)

flake8 passed.
JSHint failed.

JSHint

bolariinwa
  1. 
      
  2. Show all issues

    There are a couple of places in your code with trailing whitespace

  3. 
      
cdkushni
Review request changed
Description:
   

Set up the view class for handling the popup dialog.

~   Handling basic popup functionality(WIP).
  ~ Need to remove old tooltip popup.
  + Handling basic popup functionality.
    Handles comment selection from commentlist.
    Selecting a comment will expand it to fill the dialog.
~   Replying in the dialog box (WIP).
  ~ Replying in the dialog box (WIP) - Text field with action
  + buttons implemented, needs styling and reply posting.
  + Have issuebar inserted into comment when expanded.
  + Need to query user data from a comment to get avatars.
    UI Design Aesthetics (WIP).

Commit:
6b180fb93d3c6aa45803549283f51a4bef08bb3c
c7e59534a6144bfd7fb920aab304eb992bd0045d

Checks run (1 failed, 1 succeeded)

flake8 passed.
JSHint failed.

JSHint

cdkushni
Review request changed
Commit:
c7e59534a6144bfd7fb920aab304eb992bd0045d
18019b970cad8ca75c81812bce38529fb44cdeca

Checks run (1 failed, 1 succeeded)

flake8 passed.
JSHint failed.

JSHint

Malcolm
  1. 
      
  2. Show all issues

    Since this is an ES6 file you should be able to put a trailing comma after '_onMouseLeave'

  3. reviewboard/static/rb/js/views/replyCommentDialogView.es6.js (Diff revision 4)
     
     
     
     
     
     
     
    Show all issues

    There seems to be some extra white space here. In addition to that on line 57 and 58, the ending > for the <a> tag could be placed on the same line as the a tag itself. Additionally the closing </a> tag can be placed on a newline with the inner <input> tag being indented within the scope of the <a> section. For example look at what you did on lines 59-61 as it looks a bit nicer.

  4. 
      
cdkushni
Review request changed
Commit:
18019b970cad8ca75c81812bce38529fb44cdeca
e90a986c6822f67158610d88f991bd19ab549d6b

Checks run (1 failed, 1 succeeded)

flake8 passed.
JSHint failed.

JSHint