summaryrefslogtreecommitdiff
path: root/app/helpers/diff_helper.rb
Commit message (Collapse)AuthorAgeFilesLines
* Use Gitlab::Git::DiffCollectionsJacob Vosmaer2016-03-031-27/+7
|
* Fixes syntax errorAlfredo Sumaran2016-02-191-1/+1
|
* Make diff view name getter less error proneAlfredo Sumaran2016-02-191-1/+1
|
* Sort line notes used in parallel diff by created_atRémy Coutable2016-02-171-1/+1
| | | | Fixes #13464.
* Fewer html_safesrename-inline-diffDouwe Maan2016-01-301-2/+2
|
* Mark inline difference between old and new paths when a file is renamedDouwe Maan2016-01-291-1/+10
|
* Save and use actual diff base commit for MR diff highlightingDouwe Maan2016-01-201-2/+1
|
* Restore helperDouwe Maan2016-01-201-0/+4
|
* Move parallel diff logic to separate classDouwe Maan2016-01-201-104/+0
|
* More refactoring from last code review. #3945Rubén Dávila2016-01-141-2/+2
| | | | | | * Use commit objects instead of IDs when generating diffs * Use proper references when generating MR's source and target * Update broken specs
* Refactor parallel_diff generation a bit.Douwe Maan2016-01-141-23/+63
|
* Little refactor for usage of html_safe. #3945Rubén Dávila2016-01-131-4/+5
|
* Fix broken spec. #3945Rubén Dávila2016-01-121-1/+2
|
* Make diff_line_content helper return a safe String. #3945Rubén Dávila2016-01-121-2/+2
|
* Change strategy to highlight diffs. #3945Rubén Dávila2016-01-071-2/+2
| | | | | Now we apply syntax highlighting to the whole old and new files. This basically help us to highlight adequately multiline content.
* Fix broken spec and small refactor. #3945Rubén Dávila2015-12-301-2/+2
|
* New implementation for highlighting diff files. #3945Rubén Dávila2015-12-301-3/+3
| | | | | | * It is more performant given now we process all the diff file instead of processing line by line. * Multiline comments are highlighted correctly.
* Truncate submodule commit SHAs to 7 charactersDouwe Maan2015-12-021-2/+2
|
* Merge branch 'compare-first-commit' into 'master' Robert Speicher2015-11-171-1/+1
|\ | | | | | | | | | | | | Correctly set comparison first commit when range includes a merge commit Fixes #3207. See merge request !1814
| * Correctly set comparison first commit when range includes a merge commitcompare-first-commitDouwe Maan2015-11-171-1/+1
| |
* | Merge branch 'remove-code-duplication'Dmitriy Zaporozhets2015-11-171-16/+16
|\ \ | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | Remove duplication in diff_helper.rbDmitriy Zaporozhets2015-11-171-16/+16
| |/ | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | Don't use params[:view] directly.param-viewDouwe Maan2015-11-171-2/+6
|/
* Small UI improvements to merge request pagesmall-ui-improvementsDmitriy Zaporozhets2015-11-061-2/+2
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Correctly find last known blob for file deleted in MR.merge-request-deleted-fileDouwe Maan2015-10-201-1/+2
|
* Fix 500 error on compare pagefix-500-compareDmitriy Zaporozhets2015-10-011-1/+1
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Fix bug when removed file was not appearing in merge request diffDmitriy Zaporozhets2015-09-301-0/+19
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Fix testsDmitriy Zaporozhets2015-09-031-2/+2
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Disabled expansion of top/bottom blobs for new file diffsAlex Connor2015-05-211-0/+4
|
* Improve handling of large diffsAlex Lossent2015-05-051-5/+14
| | | | | | | | | | Diffs with a large number of changed lines time out (504 HTTP error) or generate a HTML page that's so heavy web browsers struggle with it. https://github.com/gitlabhq/gitlabhq/pull/5014 introduced limits on commit line count so that only a safe portion is rendered. This was later undone by code refactoring in be5b6db8, e0eb4803 and c741fcab. This patch re-introduces a safe limit on number of lines.
* Fix "Revspec not found" errors when viewing diffs in a forked project with ↵Stan Hu2015-04-271-2/+2
| | | | | | submodules Closes #1413
* Fix merge request comments on files with multiple commitsDominik Sander2015-04-151-1/+1
| | | | | | | | | | | | | | | | Having a merge request with a comments on a line which is then changed in a later commit prevented new comments from properly showing up in the merge request show page. * `Note#set_diff` do not use stored the diff when creating a new note in merge requests (we can not be sure the diff did not changed since the last comment on that line) * Do not rely just on `outdated?` of the first note when displaying comments of a MR in the discussion tab, but partition the active/outdated notes and display them all * In the inline changes tab just select the active notes, so an outdated note does not prevent an active one from being rendered * Just show active comments in the side-by-side changes tab
* Fix broken side-by-side diff view on merge request pageStan Hu2015-03-251-0/+4
| | | | Closes #1294
* Refactor buttonsDmitriy Zaporozhets2015-03-141-2/+2
|
* show diff in submodulesHeadless2015-01-191-0/+15
|
* Set of UI changes mostly for issue and merge requestDmitriy Zaporozhets2014-12-251-0/+18
| | | | | | | | | | * return edit/close buttons to old position (right of title) * make 'Issue #1' header smaller * move mr commits to separate tab * change inline/side diff switcher to buttons from tabs * make issue sidebar start with dicsussion block Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Move organizing of comments to helper.Marin Jankovski2014-09-121-0/+14
|
* Remove unecesarry array operations.Marin Jankovski2014-09-121-9/+9
|
* Add comments on the side-by-side diff.Marin Jankovski2014-09-121-3/+9
|
* Fix typo.Marin Jankovski2014-09-091-1/+1
|
* Add line code to parallel diff for linking.Marin Jankovski2014-09-091-4/+4
|
* Use the new path to the partial, move the diff related methods to the new ↵Marin Jankovski2014-09-091-0/+11
| | | | helper.
* optimize show_diff_size_warning?Dmitriy Zaporozhets2014-09-081-5/+9
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Refactoring inside refactoring. We need to go deeperDmitriy Zaporozhets2014-09-081-4/+65
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Use new diff parsing logicDmitriy Zaporozhets2014-09-081-3/+5
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Refactor diff suppress logic and diff viewsDmitriy Zaporozhets2014-07-151-0/+22
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>