summaryrefslogtreecommitdiff
path: root/spec/javascripts
Commit message (Collapse)AuthorAgeFilesLines
* Fix test22539-display-foldersFilipa Lacerda2016-11-191-22/+11
|
* Fix broken testFilipa Lacerda2016-11-191-0/+20
|
* Changes for stop url to pathFilipa Lacerda2016-11-191-16/+0
|
* Try to fix testsKamil Trzcinski2016-11-181-4/+5
|
* Fix broken testsFilipa Lacerda2016-11-182-11/+11
|
* Swaps url for paths in mock dataFilipa Lacerda2016-11-171-11/+10
|
* Merge branch 'master' into 22539-display-foldersGrzegorz Bizon2016-11-1729-29/+29
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (154 commits) Fix typos Fix wrong changelog item Add missing item for 8.13.6 Change last_used_at to use touch Add chat_name partial Mention Git strategy none Fix code review adds fix for security issue when annonymous user does not ... Remove ToC since it's now supported in the docs portal itself Add 8.14 to versions with further additions to review apps Add Limitations sections to environments and review apps docs Add link to environments docs Fix URL to review apps docs Add a prerequisites section, add some links Link to NGINX example project for the time being Get rid most of the irrelevant sections Add note about current limitation in $CI_BUILD_REF_NAME Add an intro and an Overview section for Review Apps WIP review apps Add Review apps link to CI README ... Conflicts: spec/features/environments_spec.rb
| * explicitly disable eslint inlineexplicitly-disable-eslint-inlineLuke "Jared" Bennett2016-11-1529-29/+29
| |
* | TestsFilipa Lacerda2016-11-161-1/+1
| |
* | Adds new line to eofFilipa Lacerda2016-11-161-1/+1
| |
* | Fix eslintFilipa Lacerda2016-11-161-2/+0
| | | | | | | | Remove unused method
* | Adds missing tests for commit componentFilipa Lacerda2016-11-161-68/+107
| |
* | Adds props validationFilipa Lacerda2016-11-165-274/+68
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improves documentation Adds tests Fix prop validation for objects Finish tests for environment item Adds tests for toggle folder function Environment tests Adds tests
* | Adds testsFilipa Lacerda2016-11-161-10/+108
| |
* | Adds testsFilipa Lacerda2016-11-1611-0/+589
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds tests. Changes instance into a constructor Adds tests for environments component Adds tests assertations Adds external URL test Adds tests for Rollback component Adds tests for stop component Adds tests for actions component Fix environment item Init environment item tests
* | Adds template functionsFilipa Lacerda2016-11-161-1/+1
| | | | | | | | Adds commit component
* | Fixes the testsFilipa Lacerda2016-11-162-32/+142
| |
* | Fix whitespaceFilipa Lacerda2016-11-161-5/+1
| |
* | Adds tests for environment store.Filipa Lacerda2016-11-162-82/+50
| |
* | Removes trailing whitespaceFilipa Lacerda2016-11-161-2/+2
| |
* | Fixes filterFilipa Lacerda2016-11-162-2/+22
| |
* | Adds tests for environments storeFilipa Lacerda2016-11-162-0/+96
|/
* Merge branch 'improve-build-scroll-controls-responsive-behaviour' into 'master' Fatih Acet2016-11-083-0/+243
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improved build page scroll UX ## What does this MR do? This MR smoothes the UX of the builds page by more effectively affixing the scroll step buttons. It also ensures the scroll step buttons are always in view, even if the sidemenu is open. It also moves the autoscroll button into the same container as the scroll buttons. ## Are there points in the code the reviewer needs to double check? ## Why was this MR needed? The build scroll buttons are always in unpredictable places and are often hidden behind sidemenus. ## Screenshots (if relevant) ![2016-09-08_17.43.58](/uploads/49cb9ad5ef2764453afaa405af7111b2/2016-09-08_17.43.58.gif) ## Does this MR meet the acceptance criteria? - [ ] [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 [merge request performance guides](http://docs.gitlab.com/ce/development/merge_request_performance_guidelines.html) - [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) ## What are the relevant issue numbers? Contributes #21832 See merge request !6270
| * Added new .eslintrc for jasmine tests and corrected build_specimprove-build-scroll-controls-responsive-behaviourLuke "Jared" Bennett2016-11-042-32/+44
| |
| * Add javascript unit tests for BuildJared Deckard2016-11-042-0/+231
| | | | | | | | | | | | | | | | Move comments to the correct location Remove array extension usage Move build options to fixture to match view
* | Merge branch 'upgrade-timeago' into 'master' Fatih Acet2016-11-081-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace jQuery.timeago with timeago.js ## What does this MR do? Replaces jQuery.timeago with [timeago.js](https://github.com/hustcc/timeago.js) ## Are there points in the code the reviewer needs to double check? * Check to make sure its working everywhere :smile: * Check to make sure the timeago wording matches what we have now (I think I've got this down but an extra pair of :eyes: would help too) ## Why was this MR needed? * The jQuery.timeago version we have is outdated * timeago.js is smaller (7.19 KB => 4.52 KB) * timeago.js has no jQuery dependency * removes all inline javascript :crossed_swords: for timeago ## Screenshots (if relevant) None ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added - Tests - [x] All builds are passing - [x] Conform by the [merge request performance guides](http://docs.gitlab.com/ce/development/merge_request_performance_guidelines.html) - [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) ## What are the relevant issue numbers? Closes #21793 See merge request !6274
| * | Replace jQuery.timeago with timeago.jsClement Ho2016-11-071-1/+1
| | |
* | | Merge branch 'move-boards-interceptor' into 'master' Fatih Acet2016-11-083-108/+116
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move Ajax interceptor into describe block ## What does this MR do? Move registering the Vue resource interceptor for issue board mock data into the corresponding `describe` blocks. ## Why was this MR needed? Currently, the interceptor is registered for every test (which makes them at best fail for Ajax calls using Vue resource). See merge request !7304
| * | Move Ajax interceptor into describe block (!7304)winniehell2016-11-053-108/+116
| |/
* | Remove logging to console in diff_comments_store_specwinniehell2016-11-051-1/+0
|/
* Move jquery.timeago to vendor directoryAlfredo Sumaran2016-11-031-1/+1
|
* Change show-gl-field-errors to gl-show-field-errorsBryce Johnson2016-11-022-2/+2
|
* Upgrade gl_field_errors to support more use cases.Bryce Johnson2016-11-012-2/+2
|
* Remove leftover references to coffeescript from comments and docs.decaffeinate-the-docsBryce Johnson2016-10-311-1/+1
|
* Do not allow text input in dropdown while loadingIdo Leibovich2016-10-281-14/+64
| | | | While loading, the entire dropdown is greyed out. However, The focus is on the text input, so user can input text. Move the focus event to after loading finished to fix this.
* Merge branch 'issue-board-welcome-cookie-monster' into 'master' Robert Speicher2016-10-281-1/+4
|\ | | | | | | | | | | | | | | | | | | | | Delete issue board welcome cookie ## What does this MR do? This was seen in the idea to production demo, because the project had already existed on the exact same URL the cookie already existed which meant the blank state wasn't shown. This MR deletes the cookie on project create. See merge request !6899
| * Fixed boards store specissue-board-welcome-cookie-monsterPhil Hughes2016-10-281-1/+4
| |
* | Replace static issue fixtures by script (!6059)winniehell2016-10-284-40/+63
| |
* | Clean up issue_spec.jswinniehell2016-10-281-59/+102
|/
* Replaced jquery.cookie with js.cookieLuke Bennett2016-10-271-1/+1
|
* Enable linting for ES6 fileswinniehell2016-10-272-0/+2
|
* Merge branch 'add-todo-toggle-event' into 'master' remove-tooltip-text-truncation-from-pipeline-graph-build-node-tooltipsFatih Acet2016-10-277-1/+198
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add todo toggle event ## What does this MR do? Adds a custom jQuery event `todo:toggle` to detect when a new todo is added so that the respective todo counters (header navigation and sidebar) can update as needed ## Are there points in the code the reviewer needs to double check? * I wasn't sure whether each `spec` should be modularized based on the html templates or whether they should be separated based on function. There are some crossovers between the `dashboard_spec` and the `header_spec`. * The naming conventions for `sidebar` and `right_sidebar` were a little confusing since they were named the opposite in their `specs`. I made a few assumptions and named a few files based on what I thought they should be named. I'd be happy to change it to something else though :smile: ## Why was this MR needed? This resolves an existing issue where the todo count on the sidebar would not update (until refresh) and refactors the existing methods that are used to update the todo counters (header navigation and sidebar) ## What are the relevant issue numbers? Closes #20140 ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added - Tests - [x] Added for this feature/bug - [x] 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 !5724
| * Add todo toggle eventClement Ho2016-10-267-1/+198
| |
* | disable ESLint for all JavaScript fileswinniehell2016-10-2639-0/+39
| |
* | Merge branch 'replace-jquery-cookie-plugin' into 'master' Alfredo Sumaran2016-10-266-30/+7
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace jquery-cookie plugin with js-cookie ## What does this MR do? Removes an outdated and unmaintained dependency [jquery.cookie](https://github.com/carhartl/jquery-cookie) with its successor [js.cookie](https://github.com/js-cookie/js-cookie) which is not dependent on jQuery. Also this MR adds a global default cookie `path` setting which defaults to the GitLab root directory. Now, setting a cookie to any path other than `gon.relative_url_root` requires an explicit opt-out. This is a sensible default and should prevent accidental omission going forward. ## Are there points in the code the reviewer needs to double check? Double check that all instances of `$.cookie(x)` have been replaced by `Cookies.get(x)` and all instances of `$.cookie(x, y, z)` are replaced with `Cookies.set(x, y, z)`. All javascripts which set or retrieve cookie values should function just as before. ## Why was this MR needed? One more step toward shaking off jQuery as a dependency (#23022), also [jquery.cookie](https://github.com/carhartl/jquery-cookie) is no longer maintained and hasn't seen an update in 2+ years. ## Does this MR meet the acceptance criteria? - [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG.md) entry added - Tests - [x] All builds are passing - [x] Conform by the [merge request performance guides](http://docs.gitlab.com/ce/development/merge_request_performance_guidelines.html) - [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 it does - rebase it please) - [x] [Squashed related commits together](https://git-scm.com/book/en/Git-Tools-Rewriting-History#Squashing-Commits) ## What are the relevant issue numbers? #23022 See merge request !7085
| * | remove manual Cookie.set "path" option in favor of global settingMike Greiling2016-10-241-23/+0
| | |
| * | replace jquery.cookie vendor script with js.cookieMike Greiling2016-10-246-10/+10
| |/
* | Update gl_field_error tests for better input filtering.scope-input-errorsBryce Johnson2016-10-201-2/+2
|/
* Add tests for gl_field_errors.Bryce Johnson2016-10-152-0/+126
|
* Make UX upgrades to SignIn/Register views.Bryce Johnson2016-10-151-1/+1
| | | | | | | - Tab between register and sign in forms - Add individual input validation error messages - Validate username - Update many styles for all login-box forms