Skip to content

Commit

Permalink
Rename collaborator/s to person/people
Browse files Browse the repository at this point in the history
  • Loading branch information
Vincent Petry committed Sep 15, 2020
1 parent 5c53125 commit 0f23075
Show file tree
Hide file tree
Showing 10 changed files with 30 additions and 33 deletions.
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>New 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 new people'),
icon: 'group',
handler: openNewCollaboratorsPanel,
displayed: canShare
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
2 changes: 1 addition & 1 deletion 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

0 comments on commit 0f23075

Please sign in to comment.