-
Notifications
You must be signed in to change notification settings - Fork 3
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
feat(UploadPicker): Use @nextcloud/files
filename validation by default
#1310
Conversation
Bundle ReportChanges will increase total bundle size by 10.37kB ⬆️
|
0e79542
to
884754e
Compare
…ault This allows to make use of Nextcloud 30 server capabilities. Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
884754e
to
40213fd
Compare
<NcTextField v-if="canRename" | ||
ref="textfield" | ||
class="invalid-filename-dialog__input" | ||
:error="!isValidName" | ||
:helper-text="validationError" | ||
:label="t('New filename')" | ||
:value.sync="newName" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would also like default html validation.
From an accessibility perspective, I think it's best
minlenght ? maxlength ?
Ther's also no form with submit, I think this is important for accessibility ?
} finally { | ||
const textfield = (this.$refs.textfield as ComponentPublicInstance)?.$el.querySelector('input') | ||
if (textfield) { | ||
textfield.setCustomValidity(this.validationError) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@skjnldsv it is set here as HTML validation
This allows to make use of Nextcloud 30 server capabilities ("Windows support").
We keep the legacy options to be non breaking, but we can cleanup logic once we do a major release.