summaryrefslogtreecommitdiff
path: root/app/helpers/diff_helper.rb
Commit message (Collapse)AuthorAgeFilesLines
* added whitespace toggle to diffs page and set it to return the project ↵Luke "Jared" Bennett2016-06-101-0/+5
| | | | | | | | compare path with the selected whitespace params Updated CHANGELOG Moved CHANGELOG entry
* Enable Style/RedundantParentheses rubocop coprubocop/enable-redundant-parentheses-copGrzegorz Bizon2016-05-301-2/+2
| | | | See #17478
* Create DiffFilter and change SystemNoteService#change_title to use ↵Adam Butler2016-05-181-2/+2
| | | | Gitlab::Diff::InlineDiff
* Extract LegacyDiffNote out of NoteDouwe Maan2016-05-131-13/+9
|
* Allow to pass custom classes to buttonAlfredo Sumaran2016-04-261-6/+9
|
* Hide button on mobileAlfredo Sumaran2016-04-261-1/+1
|
* Syntax and code improvementsAlfredo Sumaran2016-04-261-11/+13
|
* Rename methodAlfredo Sumaran2016-04-261-4/+4
|
* Use new method hide_whitespaces?Alfredo Sumaran2016-04-261-1/+1
|
* Add Hide/show whitespace changes button on diff and commit viewAlfredo Sumaran2016-04-261-0/+22
|
* Always read diff_view setting from the cookieRobert Speicher2016-04-141-1/+7
| | | | | | | | | | | | | | Prior, when the user had their view set to "parallel" and then visited a merge request's changes tab _without_ passing the `view` parameter via query string, the view would be parallel but the `Notes` class was always instantiated with the default value from `diff_view` ("inline"), resulting in broken markup when the form to add a line note was dynamically inserted. The cookie is set whenever the view is changed, so this value should always be up-to-date. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/14557
* Syntax & style updates14176-diff-paste-dataAnnabel Dunstone2016-04-141-3/+1
|
* Add line type conditional to diff line helperAnnabel Dunstone2016-04-141-2/+4
|
* Copying and pasting doesn't grab line numbers or +/-Annabel Dunstone2016-04-141-0/+1
|
* 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>