summaryrefslogtreecommitdiffstats
path: root/apps/files/js
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #13212 from nextcloud/bugfix/noid/ie11-files-loadingRoeland Jago Douma2018-12-211-2/+5
|\ | | | | Files list not rendering if user has favorites navigation unfolded
| * Replace ChildNode.before with custom before helperJulius Härtl2018-12-211-2/+5
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Only check whatsnew once per hourRoeland Jago Douma2018-12-201-1/+4
|/ | | | | | | | Store the last check in the session storage. (Which gets cleared on logout). And only check once an hour. Saves a request to the server on most requests when browsing. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Do not show general warning on free space errorRoeland Jago Douma2018-12-051-1/+1
| | | | | | | | Fixes #12588 Probably needs more fixing for the other cases. But this is the quick fix I could come up with for now. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Merge pull request #10825 from greenido/fixing-issue-9931Morris Jobke2018-11-221-1/+50
|\ | | | | Copy a file to the same directory
| * JSUnit tests are not compatible with let/const yetMorris Jobke2018-11-191-10/+10
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
| * Translate name for "(copy)"Morris Jobke2018-11-191-11/+12
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
| * Per the last comment on having only (copy) for the first oneIdo Green2018-11-191-5/+19
| | | | | | | | Signed-off-by: Ido Green <greenido@gmail.com>
| * Use the new naming per the comments on #9931Ido Green2018-11-191-5/+5
| | | | | | | | | | | | More details here: https://github.com/nextcloud/server/issues/9931#issuecomment-402318120 Signed-off-by: Ido Green <greenido@gmail.com>
| * fixing issue 9931 - copy a file to the same directoryIdo Green2018-11-191-1/+35
| | | | | | | | | | Signed-off-by: Ido Green <greenido@gmail.com> Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Do not switch to root folder if filelist is already shownJulius Härtl2018-11-222-3/+13
| | | | | | | | | | | | | | Navigating to the root folder is already handled by OCA.Files.Navigation.setActiveItem in case the view doesn't change. Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Fix rendering of the sidebar in Files appDaniel Calviño Sánchez2018-11-222-11/+25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a view is rendered it should not be concerned with where it is going to be placed in the document; in general this should be a responsibility of the object using the view. Moreover, when the details view is rendered it should simply prepare a skeleton that includes the root elements provided by the plugins; those elements will be updated by the plugins as needed when a file or a tab is selected. Finally, the details view should not be explicitly rendered. The rendering removes the previous elements, but that is needed only when the details view is in a dirty state, that is, when new plugins were added since the last time that it was rendered. However, that dirty state is internally handled, and the view is automatically rendered again if needed when a file info is set. Due to all that the details view is no longer explicitly rendered when updating it with a different file. Also, as each file list has its own details view, and each details view has its own element, but there can be only one details view/sidebar element in the document, when the file list updates the details view it also replaces the current one in the document with its own details view if needed (that is, if it is not the current one already). Besides that, when the element of a details view is replaced with the element of a different details view the old one should be detached from the document, but never removed. Otherwise the event handlers would not work when that element is attached again later (when changing to a different section in the Files app and then going back to the previous one). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Fix whitespace and file action cleanupJulius Härtl2018-11-201-11/+10
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Allow to hide download option for folders shared by linkJulius Härtl2018-11-201-9/+11
|/ | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Make number of file list entries depending on the width for grid viewJulius Härtl2018-11-181-1/+8
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Show thumbnail on renameMorris Jobke2018-11-151-3/+3
| | | | | | | | * does not hide thumbnail when rename is shown * fixes layout for grid and list view * fixes #11901 Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Fix min trhumbnail sizeJohn Molakvoæ (skjnldsv)2018-11-061-2/+2
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Add grid toggle for every files viewJohn Molakvoæ (skjnldsv)2018-11-061-2/+3
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Merge pull request #12128 from tomasz-grobelny/fix_file_moveMorris Jobke2018-11-061-4/+43
|\ | | | | Fix file move operation for large number of files
| * Merge remote-tracking branch 'upstream/master' into fix_file_moveTomasz Grobelny2018-11-042-0/+18
| |\ | | | | | | | | | Signed-off-by: Tomasz Grobelny <tomasz@grobelny.net>
| * | Fix move related testsTomasz Grobelny2018-11-031-6/+10
| | | | | | | | | | | | Signed-off-by: Tomasz Grobelny <tomasz@grobelny.net>
| * | Fix file move operation for large number of filesTomasz Grobelny2018-10-301-5/+40
| | | | | | | | | | | | Signed-off-by: Tomasz Grobelny <tomasz@grobelny.net>
* | | Ask before cancelling an ongoing uploadJohn Molakvoæ (skjnldsv)2018-11-051-5/+24
| | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* | | Set the filemodel before rending the detailsviewRoeland Jago Douma2018-11-021-1/+1
| |/ |/| | | | | | | | | | | | | | | | | | | fixes #10934 Else it triggers the rendering two times. Resulting is weird state in for example the comments. Because the comments for OLD_FILEID are retrieved but then the model is changed to NEW_FILEID. But the old comments still get in and get parsed. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Merge pull request #11647 from ↵Julius Härtl2018-10-312-0/+18
|\ \ | |/ |/| | | | | nextcloud/make-possible-for-tab-views-to-add-css-classes-to-their-container Make possible for tab views to add CSS classes to their container
| * Make possible for tab views to add CSS classes to their containerDaniel Calviño Sánchez2018-10-042-0/+18
| | | | | | | | | | | | | | | | | | In general the style of the tabs container should not change depending on which tab is the currently selected one. However, this could be needed in some very specific cases, so now the tab views can specify the extra CSS classes to be used in their container when they are selected. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Fix jsdocs build on server repoJulius Härtl2018-10-243-6/+6
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Merge pull request #11573 from nextcloud/gridview-tableMorris Jobke2018-10-241-0/+27
|\ \ | | | | | | Files grid view
| * | Restore table headJohn Molakvoæ (skjnldsv)2018-10-231-0/+1
| | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
| * | Merge branch 'master' of https://github.com/nextcloud/server into gridview-tableJohn Molakvoæ (skjnldsv)2018-10-2310-14/+55
| |\ \ | | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
| * | | Public fixes, request & default to gridJohn Molakvoæ (skjnldsv)2018-10-231-3/+6
| | | | | | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
| * | | Tooltip on hover onlyJohn Molakvoæ (skjnldsv)2018-10-231-1/+1
| | | | | | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
| * | | Fix table hidden on empty folderJohn Molakvoæ (skjnldsv)2018-10-231-1/+2
| | | | | | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
| * | | TooltipJohn Molakvoæ (skjnldsv)2018-10-181-0/+1
| | | | | | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
| * | | Remember toggle and events handlerJohn Molakvoæ (skjnldsv)2018-10-181-4/+15
| | | | | | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
| * | | Show view toggle in controls barJan-Christoph Borchardt2018-10-181-1/+1
| | | | | | | | | | | | | | | | Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
| * | | Grid initJohn Molakvoæ (skjnldsv)2018-10-181-0/+10
| | | | | | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* | | | Merge pull request #11928 from stweil/fixRoeland Jago Douma2018-10-231-1/+1
|\ \ \ \ | |_|/ / |/| | | Fix use of undeclared variable
| * | | Fix use of undeclared variableStefan Weil2018-10-221-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes a warning from LGTM: Variable 'data' is used before its declaration. Signed-off-by: Stefan Weil <sw@weilnetz.de>
* | | | Merge pull request #11967 from ↵Morris Jobke2018-10-234-6/+14
|\ \ \ \ | |/ / / |/| | | | | | | | | | | nextcloud/fix-opening-a-section-again-in-the-files-app Fix opening a section again in the Files app
| * | | Remove event handler no longer neededDaniel Calviño Sánchez2018-10-211-6/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The custom handler for "URL changed" events were added to reload the file list whenever the sections for favorites and shares were opened; this was used to fix the problem of not reloading the file lists when opening them for a second time. However, besides that the handlers were not really necessary, and as the root of the bug was fixed in the previous commit those handlers are now removed. The file list for tags uses the handler for a different purpose, though, so that one was kept. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | | Fix opening a section again in the Files appDaniel Calviño Sánchez2018-10-213-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a section is open in the Files app a "show" event is triggered. File list objects handle that event by reloading themselves, but only if the file list was shown at least once. However, the file list objects of plugins are created when the "show" event is triggered for the first time for their section; as the file list objects register their handler for the "show" event when they are created they never handle the first triggered "show" event, as the handler is set while that event is being already handled. Therefore, from the point of view of the handler, the second time that a "show" event was triggered it was seen as if the file list was shown for the first time, and thus it was not reloaded. Now the "shown" property is explicitly set for those file lists that are created while handling a "show" event, which causes them to be reloaded as expected when opening their section again. Note that it is not possible to just reload the file list whenever it is shown; the file list is reloaded also when the directory changes, and this can happen when the web page is initially loaded and the URL is parsed. In that case, if file lists were reloaded when shown for the first time then it could be reloaded twice, one with the default parameters due to the "show" event and another one with the proper parameters once the URL was parsed, and the files that appeard in the list would depend on which response from the server was received the last. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | Add missing variable declarationsStefan Weil2018-10-192-2/+2
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | This fixes errors from LGTM like the following one: Variable i is used like a local variable, but is missing a declaration. Signed-off-by: Stefan Weil <sw@weilnetz.de>
* | | Merge pull request #11905 from nextcloud/design/noid/tab-iconsRoeland Jago Douma2018-10-194-3/+22
|\ \ \ | | | | | | | | Add icon to file sidebar tabs
| * | | Add icon to file sidebar tabsJulius Härtl2018-10-184-3/+22
| |/ / | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | | Merge pull request #11823 from ↵Morris Jobke2018-10-191-0/+14
|\ \ \ | | | | | | | | | | | | | | | | lex111/fix-illegible-text-color-in-size-and-modified-columns Fix illegible text color in columns Size and Modified on dark theme
| * | | Fix illegible text color in columns Size and Modified on dark theme: rework ↵Alexey Pyltsyn2018-10-181-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | after review Signed-off-by: Alexey Pyltsyn <lex61rus@gmail.com>
| * | | Fix illegible text color in columns Size and Modified on dark themeAlexey Pyltsyn2018-10-161-0/+14
| | | | | | | | | | | | | | | | Signed-off-by: Alexey Pyltsyn <lex61rus@gmail.com>
* | | | Add missing semicolonsStefan Weil2018-10-192-3/+3
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | This fixes some recommendations from LGTM: Avoid automated semicolon insertion (90% of all statements in the enclosing function have an explicit semicolon). Signed-off-by: Stefan Weil <sw@weilnetz.de>
* | | Move fileactionsmenu to compiled handlebarsRoeland Jago Douma2018-10-153-22/+55
|/ / | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
generated by cgit v1.2.3 (git 2.39.1) at 2025-01-15 16:41:30 +0000