Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | Merge branch 'toggle-whitespace-button-for-compare-page' into 'master' | Jacob Schatz | 2016-06-15 | 1 | -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" Bennett | 2016-06-10 | 1 | -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 Hu | 2016-06-12 | 2 | -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 Maan | 2016-05-20 | 1 | -4/+2 | |
|\ | | | | | | | | | # Conflicts: # app/services/system_note_service.rb | |||||
| * | Create DiffFilter and change SystemNoteService#change_title to use ↵ | Adam Butler | 2016-05-18 | 1 | -4/+2 | |
| | | | | | | | | Gitlab::Diff::InlineDiff | |||||
* | | Syntax-highlight diffs in push emails17464-backport-email-syntax-highlighting | Sean McGivern | 2016-05-17 | 1 | -1/+1 | |
|/ | | | | | Based on: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/151 | |||||
* | Use `@diff_notes_disabled` instead of `@comments_allowed` | Douwe Maan | 2016-05-13 | 3 | -5/+5 | |
| | ||||||
* | Extract LegacyDiffNote out of Note | Douwe Maan | 2016-05-13 | 2 | -10/+9 | |
| | ||||||
* | Support supressing text file diffs on the default branch with .gitattributes | Matt Oakes | 2016-04-29 | 1 | -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 mobile | Alfredo Sumaran | 2016-04-26 | 1 | -2/+2 | |
| | ||||||
* | Update syntax | Alfredo Sumaran | 2016-04-26 | 1 | -2/+2 | |
| | ||||||
* | Hide whitespace toggle button for new merge requests | Alfredo Sumaran | 2016-04-26 | 1 | -4/+6 | |
| | ||||||
* | Add Hide/show whitespace changes button on diff and commit view | Alfredo Sumaran | 2016-04-26 | 1 | -0/+4 | |
| | ||||||
* | Syntax & style updates14176-diff-paste-data | Annabel Dunstone | 2016-04-14 | 1 | -8/+8 | |
| | ||||||
* | Add line type conditional to diff line helper | Annabel Dunstone | 2016-04-14 | 1 | -1/+1 | |
| | ||||||
* | Copying and pasting doesn't grab line numbers or +/- | Annabel Dunstone | 2016-04-14 | 1 | -3/+3 | |
| | ||||||
* | Diff viewer links to correct part of the file | Phil Hughes | 2016-04-14 | 1 | -4/+4 | |
| | | | | | | Added highlight colours to diff rows Closes #13852 | |||||
* | Diff touch ups | Annabel Dunstone | 2016-04-07 | 1 | -1/+1 | |
| | ||||||
* | Diff design updates | Annabel Dunstone | 2016-04-07 | 1 | -5/+5 | |
| | ||||||
* | Update diff colors and icons | Annabel Dunstone | 2016-04-07 | 1 | -1/+1 | |
| | ||||||
* | Fix creation of merge requests for orphaned branches | Stan Hu | 2016-04-04 | 1 | -2/+5 | |
| | | | | Closes #14875 | |||||
* | Merge branch 'change_css_class_has_tooltip_to_has-tooltip' into 'master' | Jacob Schatz | 2016-03-21 | 1 | -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-tooltip | Arinde Eniola | 2016-03-20 | 1 | -1/+1 | |
| | | ||||||
* | | Merge branch 'backport-ee-premailer' into 'master' | Rémy Coutable | 2016-03-21 | 2 | -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-premailer | Douwe Maan | 2016-03-19 | 2 | -19/+28 | |
| | | ||||||
* | | Fix bug where wrong commit ID was being used in a merge request diff to show ↵ | Stan Hu | 2016-03-19 | 3 | -4/+5 | |
|/ | | | | | | | | old image Closes #3769 Closes #14327 | |||||
* | Show a notice for diffs that are too largehide-large-diffs | Yorick Peterse | 2016-03-18 | 1 | -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::DiffCollections | Jacob Vosmaer | 2016-03-03 | 3 | -11/+4 | |
| | ||||||
* | Changed too many changes buttons to whitetoo-many-changes-btns | Phil Hughes | 2016-02-29 | 1 | -6/+5 | |
| | | | | Closes #13751 | |||||
* | Emoji reponses on diffs aren't award emoji | Zeger-Jan van de Weg | 2016-02-19 | 1 | -2/+2 | |
| | ||||||
* | Fixes requested by Rémyimage-diff | Jacob Vosmaer | 2016-02-17 | 1 | -1/+1 | |
| | ||||||
* | Use /raw/ requests for image diffs | Jacob Vosmaer | 2016-02-16 | 1 | -9/+9 | |
| | ||||||
* | Eager-load image blob data in diffsfix/13458-eager-load-blob-data | Rémy Coutable | 2016-02-16 | 1 | -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 renamed | Douwe Maan | 2016-01-29 | 1 | -9/+13 | |
| | ||||||
* | Merge branch 'fix-mr-diff-edit-button' into 'master' | Robert Speicher | 2016-01-22 | 1 | -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-button | Douwe Maan | 2016-01-21 | 1 | -21/+19 | |
| | | ||||||
* | | Merge branch 'fix-diff-comments' into 'master' | Robert Speicher | 2016-01-22 | 1 | -1/+2 | |
|\ \ | | | | | | | | | | | | | More highlighted diff fixes See merge request !2545 | |||||
| * | | Restore diff comments | Douwe Maan | 2016-01-22 | 1 | -1/+2 | |
| |/ | ||||||
* | | Use consistent markup and styling for highlighting across blobs, diffs, ↵ | Douwe Maan | 2016-01-21 | 4 | -16/+16 | |
|/ | | | | blame and snippets | |||||
* | Properly highlight lines around '\ No newline at end of file' | Douwe Maan | 2016-01-20 | 2 | -0/+9 | |
| | ||||||
* | Move parallel diff logic to separate class | Douwe Maan | 2016-01-20 | 2 | -4/+4 | |
| | ||||||
* | Merge branch 'master' into issue_3945 | Rubén Dávila | 2016-01-14 | 1 | -1/+1 | |
|\ | ||||||
| * | Update files/diffs to match new layout | Dmitriy Zaporozhets | 2016-01-14 | 1 | -1/+1 | |
| | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | |||||
* | | More refactoring from last code review. #3945 | Rubén Dávila | 2016-01-14 | 1 | -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 Maan | 2016-01-14 | 2 | -3/+3 | |
| | | ||||||
* | | Refactor parallel_diff generation a bit. | Douwe Maan | 2016-01-14 | 1 | -25/+18 | |
| | | ||||||
* | | Little refactor for usage of html_safe. #3945 | Rubén Dávila | 2016-01-13 | 1 | -2/+2 | |
| | | ||||||
* | | Use html_safe instead of raw. #3945 | Rubén Dávila | 2016-01-13 | 1 | -2/+2 | |
| | | ||||||
* | | Make diff_line_content helper return a safe String. #3945 | Rubén Dávila | 2016-01-12 | 1 | -1/+1 | |
| | | ||||||
* | | Fix broken specs. #3945 | Rubén Dávila | 2016-01-08 | 1 | -2/+2 | |
| | |