Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fixes commit comments in side-by-side diff viewcommit-side-by-side-comment | Phil Hughes | 2017-09-27 | 1 | -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 Johnson | 2017-06-29 | 1 | -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.js | Mike Greiling | 2017-05-08 | 1 | -3/+2 |
| | |||||
* | Don't use original_discussion_id | Douwe Maan | 2017-04-05 | 1 | -1/+1 |
| | |||||
* | Fix merge conflict issue | Douwe Maan | 2017-04-05 | 1 | -1/+1 |
| | |||||
* | Add option to start a new discussion on an MR | Douwe Maan | 2017-04-05 | 1 | -8/+18 |
| | |||||
* | remove IIFEs in preparation for ES module refactor | Mike Greiling | 2017-03-13 | 1 | -111/+109 |
| | |||||
* | Added discussion comments avatars to diff | Phil Hughes | 2017-03-08 | 1 | -0/+3 |
| | |||||
* | Grow the button on hover | Phil Hughes | 2017-02-27 | 1 | -14/+12 |
| | |||||
* | Improved the diff comment button UX | Phil Hughes | 2017-02-27 | 1 | -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 binding | Mike Greiling | 2017-02-17 | 1 | -1/+1 |
| | |||||
* | resolve all x-spacing and no-spaced-x eslint violations | Mike Greiling | 2017-01-18 | 1 | -2/+2 |
| | |||||
* | resolve all padded-blocks eslint violations | Mike Greiling | 2017-01-18 | 1 | -3/+1 |
| | |||||
* | resolve all instances of no-undef eslint rule violationsclean-no-undef | Mike Greiling | 2016-12-14 | 1 | -1/+3 |
| | |||||
* | explicitly disable eslint inlineexplicitly-disable-eslint-inline | Luke "Jared" Bennett | 2016-11-15 | 1 | -1/+1 |
| | |||||
* | disable ESLint for all JavaScript files | winniehell | 2016-10-26 | 1 | -0/+1 |
| | |||||
* | Added unfold test to parallel and added 'diff discussion' context | Luke Bennett | 2016-08-23 | 1 | -1/+1 |
| | |||||
* | Added addtional 'renderable' validator to check 'data-note-type' attr exists | Luke Bennett | 2016-08-23 | 1 | -5/+10 |
| | |||||
* | Don't show comment button in gutter of diffs on MR discussion tabno-comment-button-on-discussion-diffs | Douwe Maan | 2016-07-25 | 1 | -1/+1 |
| | |||||
* | ES6ify all the things! | Fatih Acet | 2016-07-24 | 1 | -0/+141 |