File Uploader #80
File Uploader #80
Conversation
Release 1.0.0 beta.18
jfnadeau
force-pushed
the
feature/fileupload
branch
from
0e75f38 to
3140a3a
Compare
jfnadeau
force-pushed
the
feature/fileupload
branch
from
744256d to
3140a3a
Compare
jfnadeau
force-pushed
the
feature/fileupload
branch
from
b201d05 to
f937985
Compare
| @@ -45,6 +46,10 @@ export class MDialog extends ModulVue implements PortalMixinImpl { | |||
| @Prop({ default: true }) | |||
| public paddingFooter: boolean; | |||
|
|
|||
| public closeDialog(): void { | |||
| this.as<PortalMixin>().tryClose(); | |||
There was a problem hiding this comment.
La mécanique de close des modals a évolué depuis... Est-ce qu'on a validé que ça fonctionnait bien dans le cas où le téléverseur pop à partir d'une fenêtre modale? Et surtout que la fermeture s'effectue correctement lorsqu'on clique sur le backdrop par exemple.
There was a problem hiding this comment.
J'ai testé le téléverser dans une fenêtre modale et ça fonctionne. Pour le clic sur l'arrière-plan, j'ai mis l'option à false, je vais valider si c'est un comportement qu'on souhaite.
| "add": "Add", | ||
| "cancel": "Cancel", | ||
| "rejectFileMsg": "The following file can not be added.", | ||
| "rejectFilesMsg": "The following files can not be added.", |
There was a problem hiding this comment.
Est-ce que le service $i18n avait des limitation avec la notation au pluriel finalement?
There was a problem hiding this comment.
Je vais modifier m-i18n dans une autre pull-request :-)
| } | ||
| }) | ||
| export class MFileUpload extends ModulVue { | ||
| @Prop() public extensions?: string[]; |
There was a problem hiding this comment.
Mettre l'annotation sur une ligne différente
| }) | ||
| export class MFileUpload extends ModulVue { | ||
| @Prop() public extensions?: string[]; | ||
| @Prop() public maxSizeKb?: number; |
There was a problem hiding this comment.
Si on ne met pas une valeur par défaut, est-ce qu'une prop n'est pas nécessairement optionnelle? Ou bien si dans le code on veut nécessaire traiter le cas qu'elle puisse être d'un type T | undefined? Typiquement, je crois qu'on utilisais plus l'annotation myProp: string | undefined plutôt que le "?".
There was a problem hiding this comment.
Je peux changer pour string | undefined mais le ? c'est exactement la même chose. Je m'en sers dans le code, je ne fais pas les validations si c'est undefined.
Regarde il y a une nouvelle option "strict" dans typescript 2.7:
Avec cette switch là, toute prop optionnelle devrait être ? (ou type | undefined)
src/components/progress/progress.ts
Outdated
| public state: MProgressState; | ||
|
|
||
| private mode: string; | ||
| private styleTag; |
src/components/progress/progress.ts
Outdated
| return this.circle === false; | ||
| } | ||
|
|
||
| private get isIndeterminate() { |
src/components/progress/progress.ts
Outdated
| }; | ||
| } | ||
|
|
||
| private get svgViewbox() { |
src/components/progress/progress.ts
Outdated
| return `0 0 ${this.diameter} ${this.diameter}`; | ||
| } | ||
|
|
||
| private get svgStyles() { |
There was a problem hiding this comment.
type de retour - voir pour tous les getters
| return this.getStore(storeName).files; | ||
| } | ||
|
|
||
| public setValidationOptions( |
There was a problem hiding this comment.
Mettre la valeur de retour :void aux méthodes qui ne retourne rien (je sais c'est téteux, mais c'est comme ça!)
There was a problem hiding this comment.
Faudrait checker si y'a moyen de configurer tslint pour ça car facile à oublier
| import ProgressPlugin, { MProgressState } from '../progress/progress'; | ||
| import WithRender from './file-upload.html?style=./file-upload.scss'; | ||
|
|
||
| const COMPLETED_FILES_VISUAL_HINT_DELAY = 1000; |
No description provided.