summaryrefslogtreecommitdiffstats
path: root/apps
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | Do not log user errors as log level 4Morris Jobke2018-01-031-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | * hides log entries for the case the parent folder does not exist Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | | Show warning if slash is entered as filenameMorris Jobke2018-01-031-0/+2
|/ / / | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | Merge pull request #7634 from Abijeet/bug-7539Roeland Jago Douma2018-01-034-1/+33
|\ \ \ | | | | | | | | Fixes the delete button not appearing on certain lower resolutions
| * | | Remove margin for selected files summaryDaniel Calviño Sánchez2018-01-031-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The selected files summary shown in the multiselect header has a margin to align it with the names of the files in the contents of the file list. However, on very narrow screens, and depending on the verbosity of the language, the summary can overlap with the actions when the selection contains files and folders. To try to mitigate this, besides showing only the icons for the actions, the summary margin is removed too in very narrow screens. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | | Increase touch area for header iconsDaniel Calviño Sánchez2018-01-031-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On narrow screens only the action icons are shown in the multiselect header of the file list. In that case the padding of those icons is increased to provide a larger touch area (the padding used is the same as in the inline actions of the file list). Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | | Fixes the delete icon not appearing on certain resolutions.Abijeet2018-01-024-1/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #7539 Also fixes overlap of text on mobile devices by resorting to just icons on lower resolutions. Signed-off-by: Abijeet <abijeetpatro@gmail.com>
* | | | Merge pull request #7599 from nextcloud/quieter-debug-logMorris Jobke2018-01-032-3/+6
|\ \ \ \ | | | | | | | | | | don't show recurring log msg when paged result was turned off
| * | | | don't show recurring msg when pages result was turned offArthur Schiwon2017-12-212-3/+6
| | |_|/ | |/| | | | | | | | | | | | | | | | | | and only as debug level otherwise. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | | | Merge pull request #7658 from ↵Morris Jobke2018-01-032-6/+7
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | nextcloud/update-external-storages-to-current-file-list-layout Update "External storages" to current file list layout
| * | | | Disable selection in "External storages" file listDaniel Calviño Sánchez2017-12-271-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the checkbox was moved to where the favourite icon was shown before the layout of the file list was modified. The checkbox is no longer a descendant of the ".filename" element, so it is no longer removed by the "External storages" file list. However, even before the checkbox was moved, explicitly removing it was not the best approach, as file list rows could still be selected using "Ctrl/Shift+click". This did not provide much value, as the selection header has no actions; it simply states the number of selected elements. The proper way to disable the selection is by setting "_allowSelection" to false in the file list instead. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | | | Fix selectors for file list elementsDaniel Calviño Sánchez2017-12-271-5/+5
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the checkbox was moved to where the favourite icon was shown before the layout of the file list was modified. The first column is no longer the file name, so neither the thumbnail nor the name link were found. Due to this the thumbnail was not set to the appropriate icon, and the dummy event handler was not removed from the name link, so clicks on the name were basically ignored. Now the selectors are based on the ".filename" CSS class instead of relying on the column position. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | Merge pull request #7663 from ↵Roeland Jago Douma2018-01-021-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | nextcloud/fix-file-list-visible-behind-the-multiselect-header Fix file list visible behind the multiselect header
| * | | | Fix file list visible behind the multiselect headerDaniel Calviño Sánchez2018-01-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some parts of the file list contents (file name and actions) had a higher z-index than the file list multiselect header. That header is fixed in place, so when the file list contents were scrolled and those parts with a higher z-index overlapped the multiselect header they were fully visible. Now the z-index for the multiselect header has a higher value (the same used in the controls bar) to ensure that the contents are shown behind the header. Fixes #7540 Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | | Merge pull request #7662 from nextcloud/ldap-shutdownRoeland Jago Douma2018-01-021-7/+10
|\ \ \ \ \ | | | | | | | | | | | | throw ServerNotAvailableException when LDAP is caught shutting down
| * | | | | throw ServerNotAvailableException when LDAP is caught shutting downArthur Schiwon2018-01-021-7/+10
| |/ / / / | | | | | | | | | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | | | | Merge pull request #7598 from ↵Morris Jobke2018-01-021-2/+5
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | nextcloud/fix-closing-menu-on-second-click-in-sharing-page Fix closing menu on second click in sharing page
| * | | | Fix closing menu on second click in sharing pageDaniel Calviño Sánchez2017-12-211-2/+5
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a "mouseup" event was triggered on any element except on the share menu or its descendants the share menu was closed. The share menu toggle is not a descendant of the share menu, so clicking on it when the share menu was shown closed it by removing its "open" CSS class. However, once that happened the click event was then handled by the share menu toggle, which toggled the "open" CSS class in the share menu and thus added it again. So, from the user point of view, nothing happened when clicking on the share menu toggle if the share menu was open. Now a "mouseup" event on the share menu toggle no longer closes the share menu, and thus toggling the "open" CSS class when handling the "click" event works as expected. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | Merge pull request #7644 from nextcloud/fix_7184Roeland Jago Douma2018-01-024-48/+1
|\ \ \ \ | | | | | | | | | | Do not run SyncJob in cron
| * | | | Do no run SyncJob in cronRoeland Jago Douma2018-01-024-48/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #7184 The SyncJob can be very resource intensive. Since it requests all users on the system to create the system addressbook. In order to do this it creates a vcard for every user and updates the addressbook. There is no need for this job since the proper signals are emitted and handled in the carddav backend to update the addressbook live. Worst comes to worst there is always the occ command to bring the address book in sync again. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | | Show max warning comment lenth againRoeland Jago Douma2018-01-022-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes #7414 Since we no longer use an input field we have to use text instead of val. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | | Merge pull request #7624 from ↵Morris Jobke2018-01-022-0/+27
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | nextcloud/fix-fileActions-currentFile-not-set-before-using-it Fix "fileActions.currentFile" not set before using it
| * | | | Fix "fileActions.currentFile" not set before using itDaniel Calviño Sánchez2017-12-262-0/+27
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When an empty area of a file row was clicked and the "Details" action was executed "fileActions.currentFile" was not guaranteed to be set to the appropriate object (it depended on the previous actions of the user), so when it was used by "getCurrentMimeType()" and other FileActions functions they may not work as expected. Now it is explicitly set to the appropriate value before its use. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | Merge pull request #7626 from nextcloud/fixed-files-multiselect-top-marginRoeland Jago Douma2017-12-281-1/+1
|\ \ \ \ | | | | | | | | | | Fixed top position of multiselected files table
| * | | | Fixed top position of multiselected files tableJohn Molakvoæ (skjnldsv)2017-12-261-1/+1
| |/ / / | | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* | | | Merge pull request #7592 from nextcloud/update-quota-on-files-uploadRoeland Jago Douma2017-12-285-3/+31
|\ \ \ \ | |/ / / |/| | | Update quota on files upload
| * | | Fixed testsJohn Molakvoæ (skjnldsv)2017-12-262-3/+0
| | | | | | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
| * | | Update quota on file upload and deletionJohn Molakvoæ (skjnldsv)2017-12-214-1/+32
| | | | | | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
| * | | Fixed pinned state of quota if no apps registered an app nav itemJohn Molakvoæ (skjnldsv)2017-12-141-1/+1
| | |/ | |/| | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* | | Merge pull request #7591 from ↵Morris Jobke2017-12-224-14/+123
|\ \ \ | | | | | | | | | | | | | | | | nextcloud/trigger-events-before-and-after-a-file-action-is-executed Trigger events before and after a file action is executed
| * | | Remove internal unused propertyDaniel Calviño Sánchez2017-12-211-9/+0
| | | | | | | | | | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | | Trigger the "Details" action when clicking on an empty file row spaceDaniel Calviño Sánchez2017-12-211-1/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clicking on an empty space in a file row causes the details view to be shown. As it is a user initiated action on the file list now it is done by triggering the Details action instead of directly calling "_updateDetailsView"; the result is the same in both cases, but using the action is more consistent (clicking on the file name triggers the default action, and clicking on the inline actions triggers those actions) and also makes possible to use the "beforeTriggerAction" and "afterTriggerAction" listeners. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | | Trigger events before and after a file action is executedDaniel Calviño Sánchez2017-12-193-4/+105
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the same way that other elements can know when a FileAction is registered or a default action is set this commit makes possible to be notified before and after a FileAction is executed. This is achieved by wrapping the registered action handler in a custom function that notifies the listeners before and after executing the handler itself. Due to this approach only FileActions registered through "registerAction" trigger the events, although that is not a problem as this is how the actions should be added to the FileActions anyway. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | Merge pull request #7597 from ↵Roeland Jago Douma2017-12-221-3/+3
|\ \ \ | | | | | | | | | | | | | | | | nextcloud/fix-anchor-change-when-clicking-on-sharing-page-menu Fix anchor change when clicking on sharing page menu
| * | | Fix anchor change when clicking on sharing page menuDaniel Calviño Sánchez2017-12-211-3/+3
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The share menu toggle and some share menu items included an 'href="#"' attribute, so they were handled as internal links by the browser, which changed the current anchor when they were clicked. However, there was no real need to change the anchor in those cases, and it could interfere with other apps (for example, the PDF viewer sets the current anchor to "#pdfviewer" when it is shown and it hides itself when that anchor is modified). According to the HTML 5 spec the "href" attribute is not mandatory for "a" elements, so they were removed. Other options would have been to change the elements from "a" to "div" or something like that, but that would have required changes to the CSS rules too, or to prevent the default event handling for those elements through JavaScript, which would have been a workaround instead of the proper solution. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | Fixed breadcrumbs testsJohn Molakvoæ (skjnldsv)2017-12-211-12/+31
| | | | | | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* | | Properly show action iconsMorris Jobke2017-12-201-1/+0
| | | | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | Make contorls stick to the top when scrollingMarin Treselj2017-12-201-4/+0
| | | | | | | | | | | | Signed-off-by: Marin Treselj <marin@pixelipo.com>
* | | Fixed breadcrumbs calculation and actions flowJohn Molakvoæ (skjnldsv)2017-12-202-1/+6
|/ / | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* | Merge pull request #7556 from nextcloud/smb-stat-exceptionMorris Jobke2017-12-181-5/+7
|\ \ | | | | | | handle exceptions in SMB::stat
| * | handle exceptions in SMB::statRobin Appelman2017-12-181-5/+7
| | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Fix testsRoeland Jago Douma2017-12-181-1/+1
| | | | | | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | Also replace all other occurencesMorris Jobke2017-12-183-5/+5
| | | | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | don't use deprecated method for requesting memcacheArthur Schiwon2017-12-181-1/+1
| | | | | | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | | Merge pull request #7555 from nextcloud/smb-copy-rename-logRobin Appelman2017-12-181-0/+2
|\ \ \ | |/ / |/| | additional debug logging on smb copy/rename failures
| * | additional debug logging on smb copy/rename failuresRobin Appelman2017-12-181-0/+2
| |/ | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Merge pull request #7527 from ↵Morris Jobke2017-12-153-1/+6
|\ \ | |/ |/| | | | | nextcloud/fix-drag-shadow-not-visible-when-dragging-a-file-on-a-narrow-screen Fix drag shadow not visible when dragging a file on a narrow screen
| * Fix drag shadow not visible when dragging a file on a narrow screenDaniel Calviño Sánchez2017-12-143-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When a file from the file list is dragged a drag shadow (a copy of the file row that follows the cursor position) is created. The drag shadow element is created as a direct child of the body element, so it needs a higher "z-index" than the one used for the file list to be visible. In narrow screens the "#app-content" uses a "z-index" of 1000 in order to be visible over the "#navigation-bar" when they overlap, so the "z-index" of the drag shadow must be at least 1000 to be visible over the file list. Instead of updating the hardcoded "z-index" it was removed and replaced by CSS rules for ".dragshadow" elements to ease theming. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Use proper authorative-autoloader for app autoloadersRoeland Jago Douma2017-12-1438-209/+38
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Merge pull request #7485 from nextcloud/fix_7001Roeland Jago Douma2017-12-141-1/+1
|\ \ | | | | | | Also cache invalid DiscoveryService response
| * | Also cache invalid DiscoveryService responseRoeland Jago Douma2017-12-131-1/+1
| |/ | | | | | | | | | | | | | | * Cache it for a day so we will retry eventually * Cache the status.php response as well so we will try it once a day as well Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>