summaryrefslogtreecommitdiff
path: root/app/assets/javascripts/files_comment_button.js
Commit message (Collapse)AuthorAgeFilesLines
* Fixes commit comments in side-by-side diff viewcommit-side-by-side-commentPhil Hughes2017-09-271-3/+3
| | | | | | | | | | This was caused by the `notes` global class not existing when the `file_comment_button` code is run. The notes class was used to check if the diff is currently in parallel view or not. To get around this I've added a check into the `file_comment_button` JS to check if the view is currently parallel or not. Closes #38117
* Render add-diff-note button with server.Bryce Johnson2017-06-291-135/+58
| | | | | | | | | | This commit moves the rendering of the button back to the server, and shows/hides it using opacity rather than display. It also removes the transform applied to the button on hover (scale). Previously, both of these factors automatically triggered a reflow, which creates a performance bottleneck on pages with larger DOM size. MR: !12103
* remove bind polyfill from files_comment_button.jsMike Greiling2017-05-081-3/+2
|
* Don't use original_discussion_idDouwe Maan2017-04-051-1/+1
|
* Fix merge conflict issueDouwe Maan2017-04-051-1/+1
|
* Add option to start a new discussion on an MRDouwe Maan2017-04-051-8/+18
|
* remove IIFEs in preparation for ES module refactorMike Greiling2017-03-131-111/+109
|
* Added discussion comments avatars to diffPhil Hughes2017-03-081-0/+3
|
* Grow the button on hoverPhil Hughes2017-02-271-14/+12
|
* Improved the diff comment button UXPhil Hughes2017-02-271-30/+25
| | | | | | It now shows the line will be commenting on my highlight the line number cells with a lighter color. The button has also been made smaller, it was previously way over the top & took over a lot more space than it should of done Closes #27543
* replace implicit this == window with explicit bindingMike Greiling2017-02-171-1/+1
|
* resolve all x-spacing and no-spaced-x eslint violationsMike Greiling2017-01-181-2/+2
|
* resolve all padded-blocks eslint violationsMike Greiling2017-01-181-3/+1
|
* resolve all instances of no-undef eslint rule violationsclean-no-undefMike Greiling2016-12-141-1/+3
|
* explicitly disable eslint inlineexplicitly-disable-eslint-inlineLuke "Jared" Bennett2016-11-151-1/+1
|
* disable ESLint for all JavaScript fileswinniehell2016-10-261-0/+1
|
* Added unfold test to parallel and added 'diff discussion' contextLuke Bennett2016-08-231-1/+1
|
* Added addtional 'renderable' validator to check 'data-note-type' attr existsLuke Bennett2016-08-231-5/+10
|
* Don't show comment button in gutter of diffs on MR discussion tabno-comment-button-on-discussion-diffsDouwe Maan2016-07-251-1/+1
|
* ES6ify all the things!Fatih Acet2016-07-241-0/+141