Skip to content

Commit 4b3aa01

Browse files
skjnldsvsusnux
andcommitted
fix(files_sharing): fix label usage of file request feature
Co-authored-by: Ferdinand Thiessen <opensource@fthiessen.de> Signed-off-by: John Molakvoæ <skjnldsv@users.noreply.github.com>
1 parent df42a10 commit 4b3aa01

13 files changed

+17
-15
lines changed

apps/files_sharing/src/components/NewFileRequestDialog.vue

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,8 @@
2222
<!-- Main form -->
2323
<form ref="form"
2424
class="file-request-dialog__form"
25-
aria-labelledby="file-request-dialog-description"
25+
aria-describedby="file-request-dialog-description"
26+
:aria-label="t('files_sharing', 'File request')"
2627
aria-live="polite"
2728
data-cy-file-request-dialog-form
2829
@submit.prevent.stop="">
@@ -69,7 +70,7 @@
6970
data-cy-file-request-dialog-controls="back"
7071
type="tertiary"
7172
@click="currentStep = STEP.FIRST">
72-
{{ t('files_sharing', 'Previous') }}
73+
{{ t('files_sharing', 'Previous step') }}
7374
</NcButton>
7475

7576
<!-- Next -->

apps/files_sharing/src/components/NewFileRequestDialog/NewFileRequestDialogDatePassword.vue

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,7 @@
3030
id="file-request-dialog-expirationDate"
3131
:disabled="disabled"
3232
:hide-label="true"
33+
:label="t('files_sharing', 'Expiration date')"
3334
:max="maxDate"
3435
:min="minDate"
3536
:placeholder="t('files_sharing', 'Select a date')"
@@ -59,7 +60,7 @@
5960
<NcPasswordField ref="passwordField"
6061
:check-password-strength="true"
6162
:disabled="disabled"
62-
:label-outside="true"
63+
:label="t('files_sharing', 'Password')"
6364
:placeholder="t('files_sharing', 'Enter a valid password')"
6465
:required="false"
6566
:value="password"

apps/files_sharing/src/components/NewFileRequestDialog/NewFileRequestDialogFinish.vue

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@
2020
@click="copyShareLink"
2121
@click-trailing-button="copyShareLink">
2222
<template #trailing-button-icon>
23-
<IconCheck v-if="isCopied" :size="20" @click="isCopied = false" />
24-
<IconClipboard v-else :size="20" @click="copyShareLink" />
23+
<IconCheck v-if="isCopied" :size="20" />
24+
<IconClipboard v-else :size="20" />
2525
</template>
2626
</NcInputField>
2727

apps/files_sharing/src/components/NewFileRequestDialog/NewFileRequestDialogIntro.vue

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@
1212
</legend>
1313
<NcTextField :value="label"
1414
:disabled="disabled"
15-
:label-outside="true"
15+
:label="t('files_sharing', 'Request subject')"
1616
:placeholder="t('files_sharing', 'Birthday party photos, History assignment…')"
1717
:required="false"
1818
name="label"
@@ -27,7 +27,7 @@
2727
<NcTextField :value="destination"
2828
:disabled="disabled"
2929
:helper-text="t('files_sharing', 'The uploaded files are visible only to you unless you choose to share them.')"
30-
:label-outside="true"
30+
:label="t('files_sharing', 'Upload destination')"
3131
:minlength="2/* cannot share root */"
3232
:placeholder="t('files_sharing', 'Select a destination')"
3333
:readonly="false /* cannot validate a readonly input */"
@@ -51,7 +51,7 @@
5151
</legend>
5252
<NcTextArea :value="note"
5353
:disabled="disabled"
54-
:label-outside="true"
54+
:label="t('files_sharing', 'Note for recipient')"
5555
:placeholder="t('files_sharing', 'Add a note to help people understand what you are requesting.')"
5656
:required="false"
5757
name="note"

dist/4845-4845.js

Lines changed: 2 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/4845-4845.js.map

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/4845-4845.js.map.license

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
4845-4845.js.license

dist/5693-5693.js

Lines changed: 0 additions & 2 deletions
This file was deleted.

dist/5693-5693.js.map

Lines changed: 0 additions & 1 deletion
This file was deleted.

0 commit comments

Comments
 (0)