Skip to content

Commit

Permalink
Merge pull request #3622 from owncloud/move-create-button
Browse files Browse the repository at this point in the history
  • Loading branch information
Lukas Hirt authored Jun 24, 2020
2 parents da7fa7c + 3ca616c commit e5c1c71
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 6 deletions.
2 changes: 1 addition & 1 deletion apps/files/src/components/FileList.vue
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
key="files-list-results-existence"
gutter="small"
flex
class="uk-padding-small"
class="uk-padding-small uk-padding-remove-top uk-padding-remove-bottom uk-margin-xsmall-bottom"
>
<div>
<oc-checkbox
Expand Down
2 changes: 1 addition & 1 deletion apps/files/src/components/FilesAppBar.vue
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
@progress="onFileProgress"
/>
<oc-grid flex gutter="small">
<div class="uk-flex-1">
<div>
<div class="uk-flex">
<oc-breadcrumb
v-if="showBreadcrumb"
Expand Down
5 changes: 5 additions & 0 deletions changelog/unreleased/move-create-button
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
Change: Move create new button

We've moved the create new button in the files app bar to the left directly next to breadcrumbs.

https://github.com/owncloud/phoenix/pull/3622
10 changes: 6 additions & 4 deletions tests/acceptance/pageObjects/filesPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -255,10 +255,9 @@ module.exports = {
return (
this.waitForElementVisible('@dialog')
.waitForAnimationToFinish()
// clicking file overwrite dialog overlay to remove file upload
// popup, as we upload the file directly using `setValue`. The overwrite
// dialog is too fast and does not give time to close the dropdown beforehand
.clickElementAt(this.elements.dialog.selector, 0, 0)
// clicking new resource dropdown to hide it, as we upload the file directly using `setValue`.
// The overwrite dialog is too fast and does not give time to close the dropdown beforehand
.clickElementAt(this.elements.newResourceDropdown.selector, 0, 0)
.waitForElementNotVisible('@newFolderButton')
.click('@dialogConfirmBtn')
.waitForElementNotPresent('@dialog')
Expand Down Expand Up @@ -288,6 +287,9 @@ module.exports = {
deleteSelectedButton: {
selector: '#delete-selected-btn'
},
newResourceDropdown: {
selector: '#new-file-menu-drop'
},
newFolderButton: {
selector: '#new-folder-btn'
},
Expand Down

0 comments on commit e5c1c71

Please sign in to comment.