summaryrefslogtreecommitdiff
path: root/app/assets/javascripts/ide
Commit message (Collapse)AuthorAgeFilesLines
* Disable vue-i18n for non-autofix filesEzekiel Kigbo2019-08-083-0/+4
| | | | | | | Disables the vue-i18n eslint rules for files that are not able to be autofixed, this will stop these offences causing pipeline failures but they will need manual fixing.
* Autofix i18n lints for vue filesEzekiel Kigbo2019-07-251-1/+3
| | | | | Autofixes additional linting errors found in vue files.
* Encapsulate file view modes with constantsChris Toynbee2019-07-233-9/+20
|
* Fixed regression for re-naming files in WebIDE64784-re-name-regressionDenys Mishunov2019-07-191-4/+8
|
* Added Usage Data for some Web IDE actionsFrancisco Javier López2019-07-191-1/+1
| | | | | | | The actions tracked in the web IDE are: - creation of commits - creation of merge requests - projects loaded
* FE remove create branch call in IDE commitwebide-commit-use-correct-parentPaul Slaughter2019-07-182-8/+2
| | | | | | | Previously `start_sha` was intercepted on the frontend to create the correct branch in a separate API call. Now that the commits API supports the `start_sha` parameter directly this workaround is not needed anymore.
* Initiate editor for any file without content60856-deleting-binary-fileDenys Mishunov2019-07-052-14/+16
| | | | | | Important to get file data for any file that has no content set yet: we need this information in order to build a previe for changed/staged files if they were not yet opened in the editor
* Mark images as binary on initDenys Mishunov2019-07-051-2/+3
|
* Merge branch 'vue-i18n-js-ide-directory' into 'master'Tim Zallmann2019-07-035-14/+22
|\ | | | | | | | | Vue-i18n: autofix for app/assets/javascripts/ide directory See merge request gitlab-org/gitlab-ce!29967
| * Vue-i18n: app/assets/javascripts/ide directoryvue-i18n-js-ide-directoryEzekiel Kigbo2019-07-015-14/+22
| | | | | | | | | | i18n linting for .vue files under the app/assets/javascripts/ide directory
* | Allow adding new entries to existing pathDenys Mishunov2019-07-027-4/+29
| | | | | | | | | | | | | | If an entry has been removed from the tree and later, during the same session, a user is trying to add a new item with the same name/path we should allow for that as long as the original entry is marked with `deleted`.
* | Merge branch '45120-fix-ide-editor-to-update-size-on-show-change' into 'master'Phil Hughes2019-07-011-7/+27
|\ \ | |/ |/| | | | | | | | | Fix IDE editor to update dimensions on show change Closes #45120 See merge request gitlab-org/gitlab-ce!30135
| * Fix IDE editor to update dimensions on show change45120-fix-ide-editor-to-update-size-on-show-changePaul Slaughter2019-06-281-7/+27
| |
* | Re-name entries in one commitDenys Mishunov2019-06-266-15/+23
|/ | | | | | | | Update window's URL after entry's re-name Set proper hook for correct fileLanguage Filtered out `moved` entries from the tree
* Keep empty folders in the tree listDenys Mishunov2019-06-251-4/+0
| | | | | | | | When removing the last file from a folder in WebIDE, do not remove the folder from the view. We keep those to give users possibility to re-upload files to the same folders. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/60860
* Upgrade prettier to latest (v1.18.2)Mike Greiling2019-06-211-6/+2
| | | | | | | | | | | | | Upgrades prettier to the latest version, and auto-corrects files with yarn prettier-all-save The diffs all seem to stem from formatting changes in brackets within template literals. Nothing major. Relevant changelogs here: https://prettier.io/blog/2019/04/12/1.17.0.html https://prettier.io/blog/2019/06/06/1.18.0.html
* Fix IDE commit to use start_ref59023-fix-web-ide-creating-branches-off-new-commitsPaul Slaughter2019-06-173-3/+17
| | | | | | | | | **Why?** The branch HEAD could be changed since the IDE was opened. This leads to user's unintentionally creating commits that overwrite other changes. https://gitlab.com/gitlab-org/gitlab-ce/issues/59023
* Extract ide_status_list from ide_status_barPaul Slaughter2019-06-123-14/+27
| | | | | **Why?** The ide_status_list will be used and extended in EE.
* IDE trigger files change eventce-5276-2-update-stores-for-terminal-file-syncPaul Slaughter2019-06-074-0/+20
| | | | | | | | - On CMD+S when editor is focused - On editor blur - When file is created in tree - When file is removed in tree - When file is renamed in tree
* Default MR checkbox to true in most cases60034-default-web-ide-s-merge-request-checkbox-to-trueSam Bigelow2019-06-068-37/+95
| | | | | | | | | | | - Pull the new MR option out into it's own component - Default MR checkbox to true when creating a new MR and committing to a branch that does not have an MR - Still change the MR checkbox to false when a user is on a branch that already has an MR - Hide MR option when on a branch that already has an MR and committing to current branch - Don't default to true when committing directly to master
* Fix Web IDE add template testMark Lapierre2019-06-052-1/+2
| | | | | | | | | Wait for the new file modal to stop animating before clicking it We now need to click the commit button 3 times: 1. To enter commit mode 2. To being staging changes 3. To submit the commit
* Comply with `no-implicit-coercion` rule (CE)Nathan Friend2019-06-037-9/+10
| | | | | | | | This commit is the result of running `yarn eslint --fix` after enabling the `no-implicit-coercion` ESLint rule. This rule has been added to our ESLint config here: https://gitlab.com/gitlab-org/gitlab-eslint-config/merge_requests/14
* Updated heading for default state in Web IDEDenys Mishunov2019-05-2910-93/+180
| | | | Resolves https://gitlab.com/gitlab-org/gitlab-ce/issues/60107
* Merge branch 'fix-ide-relative-url-bug' into 'master'Phil Hughes2019-05-021-3/+4
|\ | | | | | | | | Fix IDE get file data with '/' as relative root See merge request gitlab-org/gitlab-ce!27911
| * Fix IDE get file data with '/' as relative rootfix-ide-relative-url-bugPaul Slaughter2019-04-301-3/+4
| | | | | | | | https://gitlab.com/gitlab-org/gitlab-ce/issues/60932
* | Merge branch 'js-i18n-ide' into 'master'Kushal Pandya2019-04-302-3/+5
|\ \ | |/ |/| | | | | Internationalisation of ide directory See merge request gitlab-org/gitlab-ce!27680
| * Internationalisation of ide directoryBrandon Labuschagne2019-04-252-3/+5
| | | | | | | | | | | | | | | | This is one of many MRs opened in order to improve the overall internationalisation of the GitLab codebase. i18n documentation https://docs.gitlab.com/ee/development/i18n/externalization.html
* | Add auto direction to support rtl languagesAhmad Haghighi2019-04-251-0/+1
|/
* Fix API and IDE path with `/` relative_url_rootPaul Slaughter2019-04-241-5/+5
| | | | | | | | **Why?** Previously we simply concatenated our paths, which led to requesting `//api/v4/...` and obviously failed. The BE supports a relative_url_root of `/`. It's a bug that the FE does not.
* Create constants for IDE commit action valuesPaul Slaughter2019-04-242-4/+13
| | | | | | | | | | **Why?** These values will be used to help build the mirroring diff. It is helpful keeping it controlled in a constant. **Links:** - https://gitlab.com/gitlab-org/gitlab-ee/issues/10232 - https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/11478
* Revert "Increased page size to 100"Natalia Tepluhina2019-04-232-11/+17
| | | | This reverts commit 2869a61f3b7aeb8b6d79c74332c93e32cec20caa.
* Merge branch '59514-uploading-images-base64' into 'master'Phil Hughes2019-04-235-2/+20
|\ | | | | | | | | | | | | Resolve "Web IDE uploading images shows base64 string" Closes #59514 See merge request gitlab-org/gitlab-ce!27471
| * Stores 'binary' and 'rawPath' attr after uploadDenys Mishunov2019-04-184-2/+14
| | | | | | | | | | | | | | | | | | | | - When uploading a file, we need to flag binary files correspondingly so that it gets propagated all the way to RepoEditor as it decides on whether to show editor for this or that file or not (shouldHideEditor()) - rawPath needs to be propagate all the way to ContentViewer as well so that uploaded images could be rendered as a base64 image right away instead of raw base64 string
| * Always hides editor in Web IDE for binary filesDenys Mishunov2019-04-171-1/+1
| |
| * Passes file type down to ContentViewerDenys Mishunov2019-04-171-0/+6
| | | | | | | | | | | | | | Instead of ContentViewer attempts to compute file type and render appropriate viewer type, compute it on the parent RepoEditor having all file-related information at its disposal and pass it down to ContentViewer as a prop
* | Revert "Created a hack to prevent setting content on image"Natalia Tepluhina2019-04-222-3/+3
|/ | | | This reverts commit e8c1c930c3bc7c159463ed3bed8765db01b40328.
* Fixes Web IDE not loading merge request filesPhil Hughes2019-04-111-0/+3
| | | | Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/60243
* Merge branch '57482-shortcut-to-create-merge-request-from-web-ide' into 'master'Phil Hughes2019-04-059-37/+75
|\ | | | | | | | | | | | | Resolve "Shortcut to create merge request from Web IDE" Closes #57482 See merge request gitlab-org/gitlab-ce!26792
| * Shortcut to create MR in web IDE57482-shortcut-to-create-merge-request-from-web-ideSam Bigelow2019-04-049-37/+75
| | | | | | | | | | | | | | | | | | Before the user had to choose between committing to a new branch, committing to a new branch AND creating an MR, or committing to the current branch regardless of whether or not it already has an MR. This commit separates the creation of an MR from whether or not they commit to an existing or new branch
* | Merge branch '57668-create-file-from-url' into 'master'Phil Hughes2019-04-053-4/+43
|\ \ | | | | | | | | | | | | | | | | | | Resolve "Support creating new file from URL in the Web IDE" Closes #57668 See merge request gitlab-org/gitlab-ce!26622
| * | Set tree list accounting for already-opened files57668-create-file-from-urlDenys Mishunov2019-04-042-4/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since we can create files from URL now, this means that these files will not exist in the tree returned from API: they exist on the client only before the first commit. In order to still show the newly-created files in the tree, we should not override the tree, but merge the tree existing on the client and the one coming from API. Changelog entry Moved trees merging into mutation
| * | Create a new file if URL references non-existent oneDenys Mishunov2019-04-041-0/+5
| | |
* | | Merge branch '60021-fix-ide-file-dropdown' into 'master'Phil Hughes2019-04-052-18/+10
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Fix IDE file dropdown Closes #60021 See merge request gitlab-org/gitlab-ce!27015
| * | | Refactor new_dropdown/index to receive isOpen prop60021-fix-ide-file-dropdownPaul Slaughter2019-04-042-18/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | **Why?** Previously it managed this as an internal data property, but there was extra complexity involved because the component doesn't have all the information it needs (i.e. the `mouseOver` prop). This is a good opportunity to colocate the data prop with the component that has the information to change it (i.e. `file_row`) and let `new_dropdown/index` receive this value as a prop.
* | | | Fix IDE detecting MR from fork branchPaul Slaughter2019-04-051-1/+2
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | **Why?** Currently the IDE loads a merge request based on only the `source_branch` name. This means it loads MR's from forks that have the same branch name (not good). - This required updating the BE API to accept `source_project_id`
* | | Fix vue render error for IDE status barPaul Slaughter2019-04-043-5/+13
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | **What?** A Vue warning that `ide_status_bar` sent a `Boolean` to a `String` property (`img-src). **What was the fix?** Previously, `latestPipeline` could be one of the following values: | | | |----------|--------| | `null` | The pipeline hasn't loaded yet | | `false` | The pipeline has loaded, but nothing was returned. | | `Object` | The piepline has loaded. | Giving a semantic meaning to different falsey values hurts maintainability. This commit fixes the above problem by removing the `false` value and introducing a `hasLoadedPipeline` state property.
* | Merge branch 'feature/webide_escaping' of gitlab.com:hiddentiger/gitlab-ce ↵Kieran Andrews2019-04-031-2/+4
|/ | | | into feature/webide_escaping
* Fixed commit logic to pick a branchNatalia Tepluhina2019-03-252-5/+6
| | | | - fixed getter spec with correct getter name
* Implemented trimming for item's name in Web IDEDenys Mishunov2019-03-181-1/+1
| | | | Resolves https://gitlab.com/gitlab-org/gitlab-ce/issues/57540
* Improve files_decorator performancePaul Slaughter2019-03-075-152/+160
| | | | | | | | | | | | | | **How?** Previously the files_decorator inserted parent folders inefficiently. It started at the first part and ensured each was inserted. Since the file entries are given to use in alphabetical order, we can start at the last part of the blob's parents. If this exists, we can short circuit and be done inserting parents. **What else?** - Improve performance of decorateData. The object spread operator is not needed because the object is brand new.