Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename collaborator/s to person/people #4070

Merged
merged 2 commits into from
Sep 15, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions apps/files/src/components/Collaborators/Collaborator.vue
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@
</oc-button>
<oc-spinner
v-else-if="$_loadingSpinnerVisible"
:aria-label="$gettext('Removing collaborator…')"
:aria-label="$gettext('Removing person')"
size="small"
/>
<oc-icon v-else name="lock" class="uk-invisible"></oc-icon>
Expand Down Expand Up @@ -112,7 +112,7 @@
}}</span>
<translate
v-if="collaborator.collaborator.name === user.id"
translate-comment="Indicator for current user in collaborators list"
translate-comment="Indicator for current user in list of people"
class="uk-text-meta files-collaborators-collaborator-additional-info"
>
(me)
Expand Down
18 changes: 8 additions & 10 deletions apps/files/src/components/Collaborators/NewCollaborator.vue
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
<template>
<div class="files-collaborators-collaborator-add-dialog">
<label for="oc-sharing-autocomplete"><translate>New Collaborators:</translate></label>
<label for="oc-sharing-autocomplete"><translate>Add People:</translate></label>
<oc-grid gutter="small">
<oc-autocomplete
id="oc-sharing-autocomplete"
ref="ocSharingAutocomplete"
:aria-label="$gettext('Select a collaborator to add')"
:aria-label="$gettext('Select a person to add')"
:items="autocompleteResults"
:items-loading="autocompleteInProgress"
:placeholder="$_ocCollaborationStatus_autocompletePlaceholder"
Expand All @@ -24,7 +24,7 @@
<oc-grid v-if="selectedCollaborators.length > 0" gutter="small">
<div>
<div>
<translate>Selected collaborators:</translate>
<translate>Selected people:</translate>
</div>
<oc-table middle class="uk-width-expand files-collaborators-collaborator-autocomplete-item">
<oc-table-row
Expand Down Expand Up @@ -66,13 +66,11 @@
</oc-button>
<oc-button v-if="saving" key="new-collaborator-saving-button" :disabled="true">
<oc-spinner
:aria-label="$gettext('Adding Collaborators')"
:aria-label="$gettext('Adding People')"
class="uk-position-small uk-position-center-left"
size="xsmall"
/>
<span v-translate :aria-hidden="true" class="uk-margin-small-left"
>Adding Collaborators</span
>
<span v-translate :aria-hidden="true" class="uk-margin-small-left">Adding People</span>
</oc-button>
<oc-button
v-else
Expand All @@ -82,7 +80,7 @@
variation="primary"
@click="$_ocCollaborators_newCollaboratorsAdd(selectedCollaborators)"
>
<translate>Add Collaborators</translate>
<translate>Share</translate>
</oc-button>
</div>
</oc-grid>
Expand Down Expand Up @@ -126,11 +124,11 @@ export default {
...mapGetters(['user']),

$_ocCollaborationStatus_autocompletePlaceholder() {
return this.$gettext("Add new collaborator by name, email or federation ID's")
return this.$gettext("Add new person by name, email or federation ID's")
},

$_announcementWhenCollaboratorAdded() {
return this.$gettext('Collaborator was added')
return this.$gettext('Person was added')
},

$_isValid() {
Expand Down
8 changes: 4 additions & 4 deletions apps/files/src/components/Collaborators/SharedFilesList.vue
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,8 @@
v-if="!$_isSharedWithMe"
key="shared-with-header-cell"
class="uk-visible@s uk-text-nowrap uk-text-meta uk-width-medium uk-text-right"
translate-context="Collaborators table column"
v-text="$gettext('Collaborators')"
translate-context="'People' table column"
v-text="$gettext('People')"
/>
<div
v-else
Expand Down Expand Up @@ -231,9 +231,9 @@ export default {
]),

/**
* Prepare the given collaboratoes list for display.
* Prepare the given peoples list for display.
* Sorts first by share type (user, group, link, remote)
* and then by the collaborator's display name using natural
* and then by the people's display name using natural
* sort. Public links entries are deduplicated into a single
* one in order to only show "Public" once even when
* there are multiple link shares.
Expand Down
11 changes: 5 additions & 6 deletions apps/files/src/components/FileLinkSidebar.vue
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
</oc-button>
</div>
<div class="uk-text-meta">
<i><translate>Only invited collaborators can use this link.</translate></i>
<i><translate>Only invited people can use this link.</translate></i>
</div>
<hr />
</section>
Expand All @@ -44,11 +44,10 @@
<div class="uk-text-meta">
<i
><translate
>Any external collaborator with the respective link can access this resource. No
sign-in required. Assign a password to avoid unintended document
exposure.</translate
></i
>
>Any external person with the respective link can access this resource. No sign-in
required. Assign a password to avoid unintended document exposure.
</translate>
</i>
</div>
<div class="uk-margin-small-top uk-margin-small-bottom">
<oc-button
Expand Down
6 changes: 3 additions & 3 deletions apps/files/src/components/FileSharingSidebar.vue
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
:key="PANEL_SHOW"
:aria-hidden="currentPanel !== PANEL_SHOW"
>
<oc-loader v-if="sharesLoading" :aria-label="$gettext('Loading collaborator list')" />
<oc-loader v-if="sharesLoading" :aria-label="$gettext('Loading people list')" />
<template v-else>
<div v-if="$_ocCollaborators_canShare" class="uk-margin-small-top uk-margin-small-bottom">
<oc-button
Expand All @@ -14,7 +14,7 @@
class="files-collaborators-open-add-share-dialog-button"
@click="$_ocCollaborators_addShare"
>
<translate>Add Collaborators</translate>
<translate>Share</translate>
</oc-button>
</div>
<p
Expand Down Expand Up @@ -107,7 +107,7 @@ const PANEL_NEW = 'newCollaborator'

export default {
title: $gettext => {
return $gettext('Collaborators')
return $gettext('People')
},
name: 'FileSharingSidebar',
components: {
Expand Down
4 changes: 2 additions & 2 deletions apps/files/src/mixins/filesListIndicators.js
Original file line number Diff line number Diff line change
Expand Up @@ -97,8 +97,8 @@ export default {

shareUserIconLabel(item) {
return this.isDirectUserShare(item)
? this.$gettext('Directly shared with collaborators')
: this.$gettext('Shared with collaborators through one of the parent folders')
? this.$gettext('Directly shared with people')
: this.$gettext('Shared with people through one of the parent folders')
},

shareLinkIconLabel(item) {
Expand Down
2 changes: 1 addition & 1 deletion apps/files/src/quickActions.js
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ function canShare(item, store) {
export default {
collaborators: {
id: 'collaborators',
label: $gettext('Add new collaborators'),
label: $gettext('Add people'),
icon: 'group',
handler: openNewCollaboratorsPanel,
displayed: canShare
Expand Down
8 changes: 8 additions & 0 deletions changelog/unreleased/rename-collaborators-to-people
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
Change: Renamed collaborators to people

All visible occurrences of "collaborator" or "collaborators" have been replaced by "person" or "people" respectively.
Additionally, the action "Add Collaborator" was changed to "Share".

https://github.com/owncloud/phoenix/pull/4070
https://github.com/owncloud/product/issues/231

24 changes: 12 additions & 12 deletions tests/acceptance/features/webUIFiles/fileDetails.feature
Original file line number Diff line number Diff line change
Expand Up @@ -17,15 +17,15 @@ Feature: User can open the details panel for any file or folder
Then the app-sidebar should be visible
And the thumbnail should be visible in the app-sidebar
And the "versions" details panel should be visible
When the user switches to "collaborators" tab in details panel using the webUI
Then the "collaborators" details panel should be visible
When the user switches to "people" tab in details panel using the webUI
Then the "people" details panel should be visible

@files_versions-app-required
Scenario: View different areas of the app-sidebar for a folder in files page
When the user picks the row of folder "simple-folder" in the webUI
Then the app-sidebar should be visible
And the thumbnail should be visible in the app-sidebar
And the "collaborators" details panel should be visible
And the "people" details panel should be visible
And no "links" tab should be available in the details panel
And no "versions" tab should be available in the details panel

Expand All @@ -37,8 +37,8 @@ Feature: User can open the details panel for any file or folder
Then the app-sidebar should be visible
And the thumbnail should be visible in the app-sidebar
And the "versions" details panel should be visible
When the user switches to "collaborators" tab in details panel using the webUI
Then the "collaborators" details panel should be visible
When the user switches to "people" tab in details panel using the webUI
Then the "people" details panel should be visible

@files_versions-app-required @skipOnOCIS @ocis-reva-issue-39
Scenario: View different areas of the app-sidebar for a folder in favorites page
Expand All @@ -47,7 +47,7 @@ Feature: User can open the details panel for any file or folder
When the user picks the row of folder "simple-folder" in the webUI
Then the app-sidebar should be visible
And the thumbnail should be visible in the app-sidebar
And the "collaborators" details panel should be visible
And the "people" details panel should be visible
And no "links" tab should be available in the details panel
And no "versions" tab should be available in the details panel

Expand All @@ -74,8 +74,8 @@ Feature: User can open the details panel for any file or folder
When the user picks the row of folder "simple-folder" in the webUI
Then the app-sidebar should be visible
And the thumbnail should be visible in the app-sidebar
When the user switches to "collaborators" tab in details panel using the webUI
Then the "collaborators" details panel should be visible
When the user switches to "people" tab in details panel using the webUI
Then the "people" details panel should be visible
# When the user switches to "comments" tab in details panel using the webUI
# Then the "comments" details panel should be visible
When the user switches to "links" tab in details panel using the webUI
Expand All @@ -89,8 +89,8 @@ Feature: User can open the details panel for any file or folder
When the user picks the row of folder "simple-folder" in the webUI
Then the app-sidebar should be visible
And the thumbnail should be visible in the app-sidebar
When the user switches to "collaborators" tab in details panel using the webUI
Then the "collaborators" details panel should be visible
When the user switches to "people" tab in details panel using the webUI
Then the "people" details panel should be visible
# When the user switches to "comments" tab in details panel using the webUI
# Then the "comments" details panel should be visible
When the user switches to "links" tab in details panel using the webUI
Expand All @@ -105,8 +105,8 @@ Feature: User can open the details panel for any file or folder
When the user picks the row of folder "simple-folder (2)" in the webUI
Then the app-sidebar should be visible
And the thumbnail should be visible in the app-sidebar
When the user switches to "collaborators" tab in details panel using the webUI
Then the "collaborators" details panel should be visible
When the user switches to "people" tab in details panel using the webUI
Then the "people" details panel should be visible
# When the user switches to "comments" tab in details panel using the webUI
# Then the "comments" details panel should be visible
When the user switches to "links" tab in details panel using the webUI
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -698,11 +698,11 @@ Feature: Share by public link
| name |
| versions |
| links |
| collaborators |
| people |
# Then the following tabs should not be visible in the details dialog
# | name |
# | links |
# | collaborators |
# | people |

@issue-2060 @skipOnOCIS @issue-ocis-reva-243
Scenario: sharing indicator inside a shared folder
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ module.exports = {
locateStrategy: 'xpath'
},
sidebarCollaboratorsTab: {
selector: '//div[@class="sidebar-container"]//a[contains(text(),"Collaborators")]',
selector: '//div[@class="sidebar-container"]//a[contains(text(),"People")]',
locateStrategy: 'xpath'
},
sidebarVersionsTab: {
Expand Down
6 changes: 3 additions & 3 deletions tests/acceptance/pageObjects/FilesPageElement/filesList.js
Original file line number Diff line number Diff line change
Expand Up @@ -551,10 +551,10 @@ module.exports = {
}
this.api.elementIdAttribute(element.ELEMENT, 'aria-label', attr => {
switch (attr.value) {
case 'Directly shared with collaborators':
case 'Directly shared with people':
indicators.push('user-direct')
break
case 'Shared with collaborators through one of the parent folders':
case 'Shared with people through one of the parent folders':
indicators.push('user-indirect')
break
case 'Directly shared with links':
Expand Down Expand Up @@ -747,7 +747,7 @@ module.exports = {
selector: '#files-list-container .files-list-no-content-message'
},
shareButtonInFileRow: {
selector: '//button[@aria-label="Collaborators"]',
selector: '//button[@aria-label="People"]',
locateStrategy: 'xpath'
},
fileRowByName: {
Expand Down
5 changes: 4 additions & 1 deletion tests/acceptance/pageObjects/filesPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,10 @@ module.exports = {
},
isPanelVisible: function(panelName, callback) {
let selector = ''
if (panelName === 'collaborators') {
if (panelName === 'people') {
selector = this.elements.collaboratorsPanel
} else if (panelName === 'collaborators') {
// FIXME: rename once renamed in all tests
selector = this.elements.collaboratorsPanel
} else if (panelName === 'versions') {
selector = this.elements.versionsPanel
Expand Down
6 changes: 3 additions & 3 deletions tests/acceptance/stepDefinitions/sharingContext.js
Original file line number Diff line number Diff line change
Expand Up @@ -703,7 +703,7 @@ Then('it should not be possible to share file/folder {string} using the webUI',
const sidebarCollaboratorsTabState = await appSideBar.isCollaboratorsTabPresentOnCurrentSidebar()
assert.ok(
!sidebarCollaboratorsTabState,
`Error: Sidebar 'Collaborators' tab for resource ${resource} is present`
`Error: Sidebar 'People' tab for resource ${resource} is present`
)
})

Expand Down Expand Up @@ -954,7 +954,7 @@ Then(
'the share {string} shared with user {string} should have no expiration information displayed on the WebUI',
async function(item, user) {
await client.page.FilesPageElement.filesList().clickRow(item)
await client.page.filesPage().selectTabInSidePanel('collaborators')
await client.page.filesPage().selectTabInSidePanel('people')
const elementID = await client.page.FilesPageElement.SharingDialog.collaboratorsDialog().getCollaboratorExpirationInfo(
user
)
Expand All @@ -970,7 +970,7 @@ Then(
'the expiration information displayed on the WebUI of share {string} shared with user {string} should be {string} or {string}',
async function(item, user, information1, information2) {
await client.page.FilesPageElement.filesList().clickRow(item)
await client.page.filesPage().selectTabInSidePanel('collaborators')
await client.page.filesPage().selectTabInSidePanel('people')
const actualInfo = await client.page.FilesPageElement.SharingDialog.collaboratorsDialog().getCollaboratorExpirationInfo(
user
)
Expand Down