summaryrefslogtreecommitdiff
path: root/app
Commit message (Collapse)AuthorAgeFilesLines
* Fix for Emojinotes_fix_stablenotes_fixValery Sizov2015-11-204-4/+6
|
* Add clipboard button to merge request cross-project referenceRobert Speicher2015-11-191-2/+4
|
* Merge branch 'rs-safari-clipboard-fallback' into 'master' Robert Speicher2015-11-191-5/+16
|\ | | | | | | | | | | | | | | | | Add a fallback for Safari copy-to-clipboard Also, hide the tooltip in a less stupid way. Closes #3547 See merge request !1844
| * Add a fallback for Safari copy-to-clipboardrs-safari-clipboard-fallbackRobert Speicher2015-11-191-5/+16
| | | | | | | | | | | | Also, hide the tooltip in a less stupid way. Closes #3547
* | Merge branch 'ci-yaml-validation' into 'master' Kamil Trzciński2015-11-193-18/+6
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commits without .gitlab-ci.yml are marked as skipped - Commits without .gitlab-ci.yml are marked as skipped - Save detailed error when YAML syntax This also fixes: #3521 #3546 /cc @jacobvosmaer See merge request !1827
| * | Don't rescue Exception, but StandardErrorci-yaml-validationKamil Trzcinski2015-11-192-8/+5
| | |
| * | Since GitLab CI is enabled by default, remove enabling it by pushing ↵Kamil Trzcinski2015-11-191-12/+0
| | | | | | | | | | | | .gitlab-ci.yml
| * | Fix testsKamil Trzcinski2015-11-191-1/+1
| | |
| * | Commits without .gitlab-ci.yml are marked as skippedKamil Trzcinski2015-11-191-1/+4
| | | | | | | | | | | | - Save detailed error when YAML syntax
* | | Merge branch 'emoji_votes' into 'master' Dmitriy Zaporozhets2015-11-1918-143/+276
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Award Emoji This it first iteration of award emoji feature. We have plan to extend emoji picker by the next release. For now, you can add award by clicking to the emoji picker or posting a regular comment with emoji like ":+1:" and any other. You can post not only emoji that listed in the emoji picker. See merge request !1825
| * | | adressing commentsValery Sizov2015-11-195-20/+21
| | | |
| * | | added spinach testsValery Sizov2015-11-194-21/+28
| | | |
| * | | Few minor improvements to emoji awards UIDmitriy Zaporozhets2015-11-191-3/+2
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Improve UI for emoji awardsDmitriy Zaporozhets2015-11-193-21/+36
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | satisfy rubocopValery Sizov2015-11-191-2/+2
| | | |
| * | | addressing commentsValery Sizov2015-11-193-7/+11
| | | |
| * | | improve styleValery Sizov2015-11-192-2/+5
| | | |
| * | | better regexpValery Sizov2015-11-191-1/+1
| | | |
| * | | Emoji: fix image of emoji when it is submitted via commentValery Sizov2015-11-195-9/+13
| | | |
| * | | fixValery Sizov2015-11-191-10/+9
| | | |
| * | | Emoji: refactoringValery Sizov2015-11-192-1/+5
| | | |
| * | | award for merge requests[ci skip]Valery Sizov2015-11-193-10/+6
| | | |
| * | | add stats on hoverValery Sizov2015-11-193-3/+41
| | | |
| * | | replace emoji references from class name to data [ci skip]Valery Sizov2015-11-193-22/+26
| | | |
| * | | css improvementsValery Sizov2015-11-194-9/+19
| | | |
| * | | award emojiValery Sizov2015-11-1916-140/+189
| | | |
* | | | Merge branch 'missing_dot_confirm_modal' into 'master' Dmitriy Zaporozhets2015-11-191-1/+1
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | | | | | | | | | Add missing dot in confirm modal See merge request !1645
| * | | Add missing dot in confirm modalAchilleas Pipinellis2015-10-191-1/+1
| | | |
* | | | Merge branch 'finding-issues-by-labels-performance' into 'master' Dmitriy Zaporozhets2015-11-193-8/+18
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve performance of finding issues with/without labels The changes in this MR ultimately lead to finding issues with(out) labels being about 2x faster due to: 1. Newly added indexes on `issues.state` and `projects.visibility_level` 2. Adjusting the query so that finding issues for multiple projects is more efficient See merge request !1787
| * | | | Use a JOIN in IssuableFinder#by_projectYorick Peterse2015-11-193-3/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When using IssuableFinder/IssuesFinder to find issues for multiple projects it's more efficient to use a JOIN + a "WHERE project_id IN" condition opposed to running a sub-query. This change means that when finding issues without labels we're now using the following SQL: SELECT issues.* FROM issues JOIN projects ON projects.id = issues.project_id LEFT JOIN label_links ON label_links.target_type = 'Issue' AND label_links.target_id = issues.id WHERE ( projects.id IN (...) OR projects.visibility_level IN (20, 10) ) AND issues.state IN ('opened','reopened') AND label_links.id IS NULL ORDER BY issues.id DESC; instead of: SELECT issues.* FROM issues LEFT JOIN label_links ON label_links.target_type = 'Issue' AND label_links.target_id = issues.id WHERE issues.project_id IN ( SELECT id FROM projects WHERE id IN (...) OR visibility_level IN (20,10) ) AND issues.state IN ('opened','reopened') AND label_links.id IS NULL ORDER BY issues.id DESC; The big benefit here is that in the last case PostgreSQL can't properly use all available indexes. In particular it ends up performing a sequence scan on the "label_links" table (processing around 290 000 rows). The new query is roughly 2x as fast as the old query.
| * | | | Memoize IssuableFinder#projectsYorick Peterse2015-11-191-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since this method's returned data doesn't change between calls on the same IssuableFinder instance we can just memoize this similar to the "project" method.
| * | | | Removed trailing whitespace from IssuableFinderYorick Peterse2015-11-191-2/+2
| | |_|/ | |/| |
* | | | Merge branch 'dbalexandre/gitlab-ce-fix-personal-snippet-access-workflow' ↵Robert Speicher2015-11-192-23/+68
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | into 'master' Improve personal snippet access workflow. Replaces !1709 Fixes #3258 See merge request !1817
| * | | | Rename `not_auth_*` ability methods to `anonymous_*`dbalexandre/gitlab-ce-fix-personal-snippet-access-workflowRobert Speicher2015-11-171-10/+9
| | | | |
| * | | | Minor refactoringDouwe Maan2015-11-171-8/+14
| | | | |
| * | | | Merge branch 'master' into ↵Douwe Maan2015-11-17165-780/+1327
| |\ \ \ \ | | | | | | | | | | | | | | | | | | dbalexandre/gitlab-ce-fix-personal-snippet-access-workflow
| * \ \ \ \ Merge branch 'master' into fix-personal-snippet-access-workflowDouglas Barbosa Alexandre2015-11-0993-309/+769
| |\ \ \ \ \
| * \ \ \ \ \ Merge branch 'master' into fix-personal-snippet-access-workflowDouglas Barbosa Alexandre2015-11-0322-61/+161
| |\ \ \ \ \ \
| * \ \ \ \ \ \ Merge branch 'master' into fix-personal-snippet-access-workflowDouglas Barbosa Alexandre2015-11-0211-37/+48
| |\ \ \ \ \ \ \
| * | | | | | | | Fix ability name for public or internal personal snippetsDouglas Barbosa Alexandre2015-11-021-1/+1
| | | | | | | | |
| * | | | | | | | Use `read` rather than `show` like the ability nameDouglas Barbosa Alexandre2015-11-021-2/+2
| | | | | | | | |
| * | | | | | | | Improve personal snippet access workflow. Fixes #3258Douglas Barbosa Alexandre2015-10-292-17/+57
| | | | | | | | |
* | | | | | | | | Merge branch 'dirceu/gitlab-ce-new-merge-request-from-file-edit' into 'master' Robert Speicher2015-11-1916-114/+169
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add option to create merge request when editing/creating a file Replaces !1611 Fixes #3059 See merge request !1820
| * | | | | | | | | DRY up codedirceu/gitlab-ce-new-merge-request-from-file-editDouwe Maan2015-11-181-30/+24
| | | | | | | | | |
| * | | | | | | | | Merge branch 'master' into dirceu/gitlab-ce-new-merge-request-from-file-editDouwe Maan2015-11-1834-356/+388
| |\ \ \ \ \ \ \ \ \
| * | | | | | | | | | Add "Start a new merge request" option to every commit formDouwe Maan2015-11-1720-157/+168
| | | | | | | | | | |
| * | | | | | | | | | Merge branch 'master' into dirceu/gitlab-ce-new-merge-request-from-file-editDouwe Maan2015-11-17392-2653/+4401
| |\ \ \ \ \ \ \ \ \ \ | | | |_|_|_|_|/ / / / | | |/| | | | | | | |
| * | | | | | | | | | Add option to create merge request when editing/creating a fileDirceu Pereira Tiegs2015-10-198-30/+80
| | | | | | | | | | |
* | | | | | | | | | | Fix 'Attach a file' link in new tag formnew-tag-textareaDouwe Maan2015-11-193-16/+9
| | | | | | | | | | |
* | | | | | | | | | | Merge branch 'builds_page_scroll_buttons' into 'master' Dmitriy Zaporozhets2015-11-191-2/+2
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix bottom position of scroll buttons in build log page Look at the up/down buttons as well as the **Enable Autoscroll**. The **Enable Autoscroll** could also be alligned with the right grey box. ## Before ![autoscroll_before](/uploads/f410dc353ca66c338d8a29c7c294758a/autoscroll_before.png) ## After ![autoscroll_after](/uploads/015fdf53618a0d6f468d666de1bbab6a/autoscroll_after.png) \cc @dzaporozhets @creamzy See merge request !1792