summaryrefslogtreecommitdiff
path: root/app/helpers
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | Merge pull request #8607 from skburgart/fix-typosDmitriy Zaporozhets2015-01-211-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Fix various typos
| * | | | | Fix various typosSteven Burgart2015-01-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | signe-in -> signed-in go_to_gihub_for_permissions -> go_to_github_for_permissions descendand -> descendant behavour -> behaviour recepient_email -> recipient_email generate_fingerpint -> generate_fingerprint dependes -> depends Cant't -> Can't wisit -> visit notifcation -> notification sufficent_scope -> sufficient_scope? levet -> level
* | | | | | Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ceDmitriy Zaporozhets2015-01-211-0/+6
|\ \ \ \ \ \
| * \ \ \ \ \ Merge branch 'show_tags_in_commit_view' into 'master'Dmitriy Zaporozhets2015-01-211-0/+6
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Show tags in commit view ### What does this MR do? With this MR you can see the assigned tags for a commit in the commit view. ### Are there points in the code the reviewer needs to double check? I don't think so. But it would be awesome if the reviewer can tell me how to add a tag to the sample_commit in spec/support/repo_helpers.rb. Then I can add a test for the new tag_names_contains method as soon as possible :) ### Why was this MR needed? At the moment if one click on a commit will see details such as: parent commit, amount of additions, deletions, but will not see if this commit has Tag linked to it. Showing branch, tag details will give better overview about the commit ### What are the relevant issue numbers / Feature requests? Feature request: http://feedback.gitlab.com/forums/176466-general/suggestions/3962044-show-tags-in-commit-view ### Screenshots ![gitlab_feature_show_tags](https://gitlab.com/uploads/haynes/gitlab-ce/771c1a36bb/gitlab_feature_show_tags.png) @dblessing Can you take a look at this MR as well please? See merge request !297
| | * | | | | | Show tags in commit viewHannes Rosenögger2015-01-171-0/+6
| | | | | | | |
* | | | | | | | Merge pull request #7389 from skv-headless/show-submodules-in-diffDmitriy Zaporozhets2015-01-212-2/+17
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | show diff in submodules
| * | | | | | | show diff in submodulesHeadless2015-01-192-2/+17
| | |_|_|/ / / | |/| | | | |
* | | | | | | developer can push to protected branchesValery Sizov2015-01-202-12/+3
|/ / / / / /
* | | | | | Improve edited ago helpersDmitriy Zaporozhets2015-01-182-4/+8
| | | | | |
* | | | | | Merge branch 'syntax_highlight_diff_email' into 'master'Dmitriy Zaporozhets2015-01-181-0/+10
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add syntax colors to diffs in email-on-push notifications ## What does this MR do? This MR adds syntaxhighlighting to the email-on-push notifications. ## Are there points in the code the reviewer needs to double check? No ## Why was this MR needed? Currently the diff in email-on-push notifications is just a monochrome diff that's hard to read. This MR adds colored syntaxhighlighting to the mails. ## What are the relevant issue numbers / Feature requests? * It closes https://github.com/gitlabhq/gitlabhq/issues/6994 * There is a Discussion in the Feedback Forum: http://feedback.gitlab.com/forums/176466-general/suggestions/5802256-add-color-to-diffs-in-email-on-push-service-to-htm * A pull request for this already existed, but was closed. https://github.com/gitlabhq/gitlabhq/pull/7739 ## Screenshots see prior Pull Request for a screenshot See merge request !295
| * | | | | | Add Diff syntax colors for email-on-push notificationsHannes Rosenögger2015-01-171-0/+10
| |/ / / / /
* | | | | | Merge pull request #8176 from yglukhov/stability-fixDmitriy Zaporozhets2015-01-181-1/+3
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Stability improvement
| * | | | | Stability improvementuran2014-12-241-1/+3
| | | | | |
* | | | | | Allow to specify home page for non logged-in usersDmitriy Zaporozhets2015-01-161-5/+0
| |/ / / / |/| | | |
* | | | | Merge pull request #7925 from cirosantilli/unused-argumentsDmitriy Zaporozhets2015-01-151-1/+1
|\ \ \ \ \ | | | | | | | | | | | | Remove or prepend underscore _ to unused method arguments
| * | | | | Remove or prepend _ to unused method argumentsCiro Santilli2014-12-271-1/+1
| | | | | |
* | | | | | Replace highlight.js with rouge-fork rugmentsStefan Tatschner2015-01-152-21/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | I decided to create a fork of rouge as rouge lacks a HTML formatter with the required options such as wrapping a line with <span> tags. Furthermore I was not really convinced about the clarity of rouge's source code. Rugments 1.0.0beta3 for now only includes some basic linting and a new HTML formatter. Everything else should behave the same.
* | | | | | Merge branch 'github_importer'Dmitriy Zaporozhets2015-01-131-0/+16
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: app/helpers/projects_helper.rb
| * | | | | | Github ImporterValery Sizov2015-01-101-0/+16
| | | | | | |
* | | | | | | First entry in wiki history leads to newest revision.yglukhov2015-01-131-0/+5
| |_|_|_|/ / |/| | | | |
* | | | | | Merge branch 'flatten-dirs' into 'master'Dmitriy Zaporozhets2015-01-111-0/+10
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Flatten the directory hierarchy while there is only one directory descendant In some languages, especially java, the directory structure reflects the package structure.This means that using the file tree browser to reach the directories that matter you have to click a lot of directories containing only another subdirectory. This merge request tries to enhance this by showing and providing the link to the first directory that doesn't contain only one directory. See also [this feature request](http://feedback.gitlab.com/forums/176466-general/suggestions/6236769-jump-forward-to-the-first-folder-that-has-more-tha) This is very similar to what [GitHub does](https://github.com/blog/1877-folder-jumping), but I don't think graying out the empty directories is really necessary. Screenshots: [Before](https://gitlab.com/uploads/marmis85/gitlab-ce/b683a0000f/before.png) - [After](https://gitlab.com/uploads/marmis85/gitlab-ce/a23725a1c1/after.png) I'm actually very new at ruby, rails, haml, and almost everything involved in Gitlab's development... From what I could understand, when we render each directory item in the list we only have the data structure representing the item but not his descendants. To iterate through the hierarchy I call Gitlab::Git:Tree.where() but I'm not sure if this is the most efficient way. I also considered making every directory a clickable link, but I tried that and in the end it wasn't very useful and reduced the click target for the most useful directory. See merge request !275
| * | | | | | Flatten the directory hierarchy while there is only one directory descendantmarmis852014-12-311-0/+10
| | |_|_|/ / | |/| | | |
* | | | | | Finally fix stuff related to dynamic configDmitriy Zaporozhets2015-01-082-3/+7
| | | | | |
* | | | | | Improve application settings and write testsDmitriy Zaporozhets2015-01-082-8/+11
| | | | | |
* | | | | | Init ApplicationSettings resource with defaults from config fileDmitriy Zaporozhets2015-01-082-0/+10
| |/ / / / |/| | | |
* | | | | Redesign signin/singup pagesDmitriy Zaporozhets2015-01-071-0/+5
| | | | |
* | | | | Restyle and refactor dashboard projects page filteringDmitriy Zaporozhets2015-01-061-0/+2
| | | | |
* | | | | Restyle and refactor milestones filterDmitriy Zaporozhets2015-01-062-12/+9
|/ / / /
* | | | Merge branch 'user_ssh_keys_on_admin' into 'master'Dmitriy Zaporozhets2014-12-301-0/+8
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | User ssh keys on admin Show user ssh keys on admin user page. See merge request !1362
| * | | | Share the key table between admin and profile resources.Marin Jankovski2014-12-301-0/+8
| | | | |
* | | | | Helper for ajax group selectboxDmitriy Zaporozhets2014-12-301-0/+9
|/ / / / | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | Merge branch 'check_browser_version' of https://github.com/vicvega/gitlabhq ↵Dmitriy Zaporozhets2014-12-281-0/+4
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | into vicvega-check_browser_version Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> Conflicts: CHANGELOG app/assets/stylesheets/generic/common.scss
| * | | check browser version, blacklisting outdated IE (version < 10)Francesco Coda Zabetta2014-12-241-0/+4
| |/ /
* | | Show assigned issues/mr be default on dashboardDmitriy Zaporozhets2014-12-261-34/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | This was default before but now it fixed with providing assignee_id parameter making url shareble and dont reset when other filters users. Also this commit removes old methods that are not used any more. Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | Fix nav_link support for several path optionsDmitriy Zaporozhets2014-12-251-19/+33
| | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | 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>
* | Merge branch 'sidebar-nav' into 'master'Dmitriy Zaporozhets2014-12-246-66/+38
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Sidebar navigation This MR changes layout of the GitLab to sidebar navigation. Layouts: - [x] Project layout - [x] Project settings layout - [x] Dashboard layout - [x] Profile layout - [x] Group layout - [x] Snippets layout - [x] Admin layout Pages: - [x] Activity pages - [x] Issue/mr pages - [x] Issues/merge requests pages Common: - [x] Fix tests See merge request !1329
| * | Refactor issues and merge requests listsDmitriy Zaporozhets2014-12-233-31/+18
| | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | One column issue/mr lists for projectDmitriy Zaporozhets2014-12-231-25/+0
| | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | Render MR diff full size of screenDmitriy Zaporozhets2014-12-221-2/+5
| | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | Merge branch 'master' into sidebar-navDmitriy Zaporozhets2014-12-212-5/+30
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> Conflicts: app/views/layouts/group.html.haml
| * | | Restyle group page and event filterDmitriy Zaporozhets2014-12-161-7/+6
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | Left-side navigation for group layoutDmitriy Zaporozhets2014-12-161-1/+9
| | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | Prevent 500 after merge MR if you check remove source branchDmitriy Zaporozhets2014-12-231-1/+8
| |/ / |/| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | Merge pull request #7886 from cirosantilli/disable-blob-edit-mrDmitriy Zaporozhets2014-12-201-5/+26
|\ \ \ | | | | | | | | Disable / hide MR edit blob button if cannot edit.
| * | | Disable / hide MR edit blob button if cannot edit.Ciro Santilli2014-10-021-5/+26
| | | |
* | | | Add theme type css classDrew Blessing2014-12-151-0/+4
| |/ / |/| |
* | | DRY and refactor atom codeDmitriy Zaporozhets2014-12-042-0/+37
| | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | Merge branch 'multi_provider' into 'master'Dmitriy Zaporozhets2014-12-042-3/+7
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Supporting for multiple omniauth provider for the same user fixes https://dev.gitlab.org/gitlab/gitlabhq/issues/1721 and https://dev.gitlab.org/gitlab/gitlabhq/issues/1760 See merge request !1296
| * | | added helperValery Sizov2014-12-041-0/+4
| | | |