summaryrefslogtreecommitdiff
path: root/app
Commit message (Collapse)AuthorAgeFilesLines
* Add visibility icon4424-improve-ui-admin-areaAlfredo Sumaran2016-07-141-0/+3
|
* Remove previously introduced CSS that will not be used anymoreAlfredo Sumaran2016-07-141-23/+0
|
* Make admin/groups view consistent with dashboard/groupsAlfredo Sumaran2016-07-141-17/+20
|
* Fix New Group button spacing on mobileAlfredo Sumaran2016-07-141-1/+1
|
* Make CSS consistent againAlfredo Sumaran2016-07-143-11/+5
|
* Rename .group-controls to .controlsAlfredo Sumaran2016-07-142-3/+3
|
* Remove unnecesary CSS classAlfredo Sumaran2016-07-142-2/+2
|
* Add margin between buttons if both retry and cancel are presentAnnabel Dunstone2016-07-142-6/+13
|
* Add margin between labels; remove underline hover style on status buttonAnnabel Dunstone2016-07-141-0/+12
|
* Merge branch 'fix/persistent-import-data' into 'master' Fatih Acet2016-07-144-5/+16
|\ | | | | | | | | | | | | | | | | | | | | Fix import_data being saved as a result of an invalid import_url Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/19479 Seem that in all cases the `import_url` was an empty string. The `import_url` is now validated correctly and no longer creates the `import_data`even if it fails validation. <= Should cover cases 1, 3, and 4 of https://gitlab.com/gitlab-org/gitlab-ce/issues/19479#note_13005276 Also, it now rescues from all exceptions when importing from a URL, so we make sure the `import_status` is failed after that. Covers case 2 of https://gitlab.com/gitlab-org/gitlab-ce/issues/19479#note_13005276 See merge request !5206
| * udpated JS based on feedbackfix/persistent-import-dataJames Lopez2016-07-141-5/+2
| |
| * some JS magic to fix empty URL bugJames Lopez2016-07-132-2/+10
| |
| * updated create_or_update_import_data to use guard clauseJames Lopez2016-07-131-1/+3
| |
| * Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ce into ↵James Lopez2016-07-1363-268/+484
| |\ | | | | | | | | | fix/persistent-import-data
| * | fix more specsJames Lopez2016-07-131-1/+5
| | |
| * | fix specsJames Lopez2016-07-131-1/+1
| | |
| * | fixes a few issues to do with import_url not being saved correctly for ↵James Lopez2016-07-122-3/+3
| | | | | | | | | | | | imports. This should prevent the import_data to be created when it should not and output an error properly validating before creating it.
* | | Merge branch 'project-files-header-cursor' into 'master' Fatih Acet2016-07-141-2/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use default cursor for table header of project files ## What does this MR do? Use default cursor for table header of project files. ## Are there points in the code the reviewer needs to double check? Nope. ## Why was this MR needed? The table header had a pointer cursor even though it was not clickable. ## What are the relevant issue numbers? fixes #19612 See merge request !5165
| * | | Use default cursor for table header of project files (!5165)winniehell2016-07-141-2/+1
| | | |
* | | | Merge branch 'lbennett/gitlab-ce-18120-focus-filter-shortcut' into 'master' Fatih Acet2016-07-144-12/+20
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added shortcut to focus filters ## What does this MR do? Adds shortcut `f` to focus filter inputs. ## Are there points in the code the reviewer needs to double check? Is this a valid implementation? Please check [this comment below](https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/4570#note_12359978)! ## Why was this MR needed? Keyboard ninja UX :crossed_swords: ## What are the relevant issue numbers? Closes #18120. ## Screenshots (if relevant) ![Screen_Shot_2016-06-09_at_18.34.56](/uploads/d4cfb6ec340c118f5227be3f16ffbed1/Screen_Shot_2016-06-09_at_18.34.56.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 !4763
| * | | Review changeslbennett/gitlab-ce-18120-focus-filter-shortcutLuke "Jared" Bennett2016-07-112-8/+9
| | | |
| * | | Added shortcut to focus all search tags apart from the top nav searchLuke "Jared" Bennett2016-07-114-9/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updated CHANGELOG Added docs for filter shortcut Moved CHANGELOG entry Added phils fixes Fixed search form submitting with empty input Added review fixes Readability and selector perf fix
* | | | Merge branch 'top-links-label' into 'master' Fatih Acet2016-07-142-10/+10
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Aria labels on top header links ## What does this MR do? Added `aria-label` to the links in top header bar for accessibility. ## What are the relevant issue numbers? Closes #12797 See merge request !5205
| * | | | Aria labels on top header linkstop-links-labelPhil Hughes2016-07-122-10/+10
| | | | | | | | | | | | | | | | | | | | Closes #12797
* | | | | Merge branch 'issuable-author-tooltip' into 'master' Fatih Acet2016-07-142-3/+4
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added author username tooltip in issuable header ## What does this MR do? Adds the authors username as a tooltip to the header of issuables ## What are the relevant issue numbers? Closes #13771 ## Screenshots (if relevant) ![Screen_Shot_2016-07-12_at_16.00.34](/uploads/9485150f1c9d4006384f18d08a07430f/Screen_Shot_2016-07-12_at_16.00.34.png) See merge request !5209
| * | | | | Fixed Rubocopissuable-author-tooltipPhil Hughes2016-07-131-2/+1
| | | | | |
| * | | | | Added author username tooltip in issuable headerPhil Hughes2016-07-122-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | Closes #13771
* | | | | | Improves left static sidebar behaviourDmitriy Zaporozhets2016-07-143-7/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | List of changes: * removed bottom 50px space that is no longer necessary with pin button at the top * increased side padding so sidebar links are closer to counters * use fixed weight pin icon so it correctly aligned with counters Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | | | Merge branch 'fix-label-reference-filter' into 'master' Robert Speicher2016-07-131-3/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix markdown rendering for label references Fixes #14424, #19753, #19760 See merge request !5224
| * | | | | | Simplify regex for string-based multi-word label surrounded in quotesfix-label-reference-filterDouglas Barbosa Alexandre2016-07-131-1/+1
| | | | | | |
| * | | | | | Doesn't match empty label references surrounded in quotesDouglas Barbosa Alexandre2016-07-131-1/+1
| | | | | | |
| * | | | | | Fix markdown rendering for label references that contains `.`Douglas Barbosa Alexandre2016-07-131-2/+2
| | | | | | |
| * | | | | | Fix markdown rendering for label references that begin with a digitDouglas Barbosa Alexandre2016-07-131-1/+4
| | | | | | |
| * | | | | | Fix markdown rendering for consecutive label referencesDouglas Barbosa Alexandre2016-07-131-1/+1
| | |_|_|_|/ | |/| | | |
* | | | | | Revert "Merge branch '18193-developers-can-merge' into 'master' "Robert Speicher2016-07-1315-47/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 9ca633eb4c62231e4ddff5466c723cf8e2bdb25d, reversing changes made to fb229bbf7970ba908962b837b270adf56f14098f.
* | | | | | Merge branch 'dropdown-load-fix' into 'master' Douwe Maan2016-07-131-1/+1
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Dropdown loading time preformance fix ## What does this MR do? Optimizes the performance of the dropdown load time by just sending the required data to load the dropdown instead of the full object This MR aims to fix #17474 See merge request !5113
| * | | | | removes redundant filter_branches filetiagonbotelho2016-07-121-8/+0
| | | | | |
| * | | | | fixes test for simplified version of projecttiagonbotelho2016-07-121-0/+8
| | | | | |
| * | | | | adds test to check json fields on simple request and changes the url request ↵tiagonbotelho2016-07-121-1/+0
| | | | | | | | | | | | | | | | | | | | | | | | format
| * | | | | fixes missing field on basicprojectdetailstiagonbotelho2016-07-121-0/+1
| | | | | |
| * | | | | removes debugging printtiagonbotelho2016-07-121-1/+0
| | | | | |
| * | | | | gets the new object through /simple endpoint and renders on the dropdowntiagonbotelho2016-07-121-0/+1
| | | | | |
| * | | | | adds test to check json fields on simple request and changes the url request ↵tiagonbotelho2016-07-122-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | format
| * | | | | fixes missing field on basicprojectdetailstiagonbotelho2016-07-121-0/+1
| | | | | |
| * | | | | changes the usage of simpleprojectdetails to already implemented ↵tiagonbotelho2016-07-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | basicprojectsdetails and changes the url to a more reader friendly format
| * | | | | removes debugging printtiagonbotelho2016-07-121-1/+0
| | | | | |
| * | | | | gets the new object through /simple endpoint and renders on the dropdowntiagonbotelho2016-07-122-1/+2
| |/ / / /
* | | | | Implement last round of review comments from !4892.18193-developers-can-mergeTimothy Andrew2016-07-131-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. Fix typos, minor styling errors. 2. Use single quotes rather than double quotes in `user_access_spec`. 3. Test formatting.
* | | | | Move all "checks" under `GitLab::Checks`.Timothy Andrew2016-07-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | - https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/4892#note_12892160 - This is more consistent.
* | | | | Don't ask the user to "merge this request manually".Timothy Andrew2016-07-132-1/+6
| | | | | | | | | | | | | | | | | | | | 1. If they are a developer with "Developers can Merge" switched on.
View Lorry Controller Status