summaryrefslogtreecommitdiff
path: root/app/views/projects/diffs
Commit message (Collapse)AuthorAgeFilesLines
...
* Merge branch 'toggle-whitespace-button-for-compare-page' into 'master' Jacob Schatz2016-06-151-0/+2
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Toggle whitespace button for compare page ## What does this MR do? Adds whitespace button to compare branches page. ## Are there points in the code the reviewer needs to double check? Ruby code. ## Why was this MR needed? Compare branches UX ## What are the relevant issue numbers? Closes #17881. ## Screenshots (if relevant) ![Screen_Shot_2016-06-09_at_18.39.48](/uploads/8995a7bc8bea7fa3ff19446ebe49fb88/Screen_Shot_2016-06-09_at_18.39.48.png) ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added - [ ] [Documentation created/updated](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/doc/development/doc_styleguide.md) - [ ] API support added - [ ] Tests - [ ] Added for this feature/bug - [ ] All builds are passing - [x] Conform by the [style guides](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CONTRIBUTING.md#style-guides) - [x] Branch has no merge conflicts with `master` (if you do - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) See merge request !4569
| * added whitespace toggle to diffs page and set it to return the project ↵Luke "Jared" Bennett2016-06-101-0/+2
| | | | | | | | | | | | | | | | compare path with the selected whitespace params Updated CHANGELOG Moved CHANGELOG entry
* | Fix Error 500 when viewing a blob with binary characters after the 1024-byte ↵Stan Hu2016-06-122-0/+3
|/ | | | | | | | | | | | | | | | mark Here was the problem: 1. When determining whether a given blob is viewable text, gitlab_git reads the first 1024 bytes and checks with Linguist whether it is a text or binary file. 2. If the blob is text, GitLab will attempt to display it. 3. However, if the text has binary characters after the first 1024 bytes, then GitLab will attempt to load the entire contents, but the encoding will be ASCII-8BIT since there are binary characters. 4. The Error 500 results when GitLab attempts to display a mix UTF-8 and ASCII-8BIT. To fix this, we load as much data as we are willing to display so that the detection will work properly. Requires an update to gitlab_git: gitlab-org/gitlab_git!86 Closes #13826
* Merge branch 'adambutler/gitlab-ce-feature/support-diff-of-issue-title-rename'Douwe Maan2016-05-201-4/+2
|\ | | | | | | | | # Conflicts: # app/services/system_note_service.rb
| * Create DiffFilter and change SystemNoteService#change_title to use ↵Adam Butler2016-05-181-4/+2
| | | | | | | | Gitlab::Diff::InlineDiff
* | Syntax-highlight diffs in push emails17464-backport-email-syntax-highlightingSean McGivern2016-05-171-1/+1
|/ | | | | Based on: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/151
* Use `@diff_notes_disabled` instead of `@comments_allowed`Douwe Maan2016-05-133-5/+5
|
* Extract LegacyDiffNote out of NoteDouwe Maan2016-05-132-10/+9
|
* Support supressing text file diffs on the default branch with .gitattributesMatt Oakes2016-04-291-13/+13
| | | | | | | | This is a combination of 3 commits. - Update the bare repositories info/attributes if the default branch is updated - Check the diff attributes of a file before showing a diff - Update CHANGELOG
* Hide button on mobileAlfredo Sumaran2016-04-261-2/+2
|
* Update syntaxAlfredo Sumaran2016-04-261-2/+2
|
* Hide whitespace toggle button for new merge requestsAlfredo Sumaran2016-04-261-4/+6
|
* Add Hide/show whitespace changes button on diff and commit viewAlfredo Sumaran2016-04-261-0/+4
|
* Syntax & style updates14176-diff-paste-dataAnnabel Dunstone2016-04-141-8/+8
|
* Add line type conditional to diff line helperAnnabel Dunstone2016-04-141-1/+1
|
* Copying and pasting doesn't grab line numbers or +/-Annabel Dunstone2016-04-141-3/+3
|
* Diff viewer links to correct part of the filePhil Hughes2016-04-141-4/+4
| | | | | | Added highlight colours to diff rows Closes #13852
* Diff touch upsAnnabel Dunstone2016-04-071-1/+1
|
* Diff design updatesAnnabel Dunstone2016-04-071-5/+5
|
* Update diff colors and iconsAnnabel Dunstone2016-04-071-1/+1
|
* Fix creation of merge requests for orphaned branchesStan Hu2016-04-041-2/+5
| | | | Closes #14875
* Merge branch 'change_css_class_has_tooltip_to_has-tooltip' into 'master' Jacob Schatz2016-03-211-1/+1
|\ | | | | | | | | | | | | change the css class has_tooltip to has-tooltip universally closes #14432 See merge request !3321
| * change the css class has_tooltip to has-tooltip universallychange_css_class_has_tooltip_to_has-tooltipArinde Eniola2016-03-201-1/+1
| |
* | Merge branch 'backport-ee-premailer' into 'master' Rémy Coutable2016-03-212-19/+28
|\ \ | |/ |/| | | | | | | | | Backport premailer and diff changes from EE Backport part of https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/151 See merge request !3318
| * Move diff line to separate partialbackport-ee-premailerDouwe Maan2016-03-192-19/+28
| |
* | Fix bug where wrong commit ID was being used in a merge request diff to show ↵Stan Hu2016-03-193-4/+5
|/ | | | | | | | old image Closes #3769 Closes #14327
* Show a notice for diffs that are too largehide-large-diffsYorick Peterse2016-03-181-9/+13
| | | | | | | This builds on the changes introduced in https://gitlab.com/gitlab-org/gitlab_git/merge_requests/72 and results in merge requests with large diffs (e.g. due to them containing minified CSS) loading much faster.
* Use Gitlab::Git::DiffCollectionsJacob Vosmaer2016-03-033-11/+4
|
* Changed too many changes buttons to whitetoo-many-changes-btnsPhil Hughes2016-02-291-6/+5
| | | | Closes #13751
* Emoji reponses on diffs aren't award emojiZeger-Jan van de Weg2016-02-191-2/+2
|
* Fixes requested by Rémyimage-diffJacob Vosmaer2016-02-171-1/+1
|
* Use /raw/ requests for image diffsJacob Vosmaer2016-02-161-9/+9
|
* Eager-load image blob data in diffsfix/13458-eager-load-blob-dataRémy Coutable2016-02-161-0/+2
| | | | | | | Since gitlab_git 8.0, blob data are lazy-loaded so we have to call blob.load_all_data!(repo) to eager-load them. Fixes #13458.
* Mark inline difference between old and new paths when a file is renamedDouwe Maan2016-01-291-9/+13
|
* Merge branch 'fix-mr-diff-edit-button' into 'master' Robert Speicher2016-01-221-21/+19
|\ | | | | | | | | | | | | Fix MR diff 'Edit' button It now works and has the right size. See merge request !2542
| * Fix MR diff 'Edit' buttonfix-mr-diff-edit-buttonDouwe Maan2016-01-211-21/+19
| |
* | Merge branch 'fix-diff-comments' into 'master' Robert Speicher2016-01-221-1/+2
|\ \ | | | | | | | | | | | | More highlighted diff fixes See merge request !2545
| * | Restore diff commentsDouwe Maan2016-01-221-1/+2
| |/
* | Use consistent markup and styling for highlighting across blobs, diffs, ↵Douwe Maan2016-01-214-16/+16
|/ | | | blame and snippets
* Properly highlight lines around '\ No newline at end of file'Douwe Maan2016-01-202-0/+9
|
* Move parallel diff logic to separate classDouwe Maan2016-01-202-4/+4
|
* Merge branch 'master' into issue_3945Rubén Dávila2016-01-141-1/+1
|\
| * Update files/diffs to match new layoutDmitriy Zaporozhets2016-01-141-1/+1
| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | More refactoring from last code review. #3945Rubén Dávila2016-01-141-1/+1
| | | | | | | | | | | | * Use commit objects instead of IDs when generating diffs * Use proper references when generating MR's source and target * Update broken specs
* | Add inline diff markers in highlighted diffs.Douwe Maan2016-01-142-3/+3
| |
* | Refactor parallel_diff generation a bit.Douwe Maan2016-01-141-25/+18
| |
* | Little refactor for usage of html_safe. #3945Rubén Dávila2016-01-131-2/+2
| |
* | Use html_safe instead of raw. #3945Rubén Dávila2016-01-131-2/+2
| |
* | Make diff_line_content helper return a safe String. #3945Rubén Dávila2016-01-121-1/+1
| |
* | Fix broken specs. #3945Rubén Dávila2016-01-081-2/+2
| |