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

Bugs #13218: correctly handle the disabled attribute #2004

Merged
merged 1 commit into from
Aug 30, 2024
Merged

Conversation

ebernard
Copy link
Contributor

Utilisé comme ceci :

                  <input
                    type="checkbox"
                    id="TIMESTAMP"
                    formControlName="declaredTimestamp"
                    [attr.disabled]="signedDocumentPolicyIsDisabled()"
                  />

Si on renvoie la valeur false, l'attribut reste disabled, il faut donc renvoyer null au lieu de false.

@ebernard ebernard added bug Something isn't working small pr embarquant peu de changements et à review rapide, ne nécessitant qu'un reviewer labels Aug 30, 2024
@ebernard ebernard added this to the IT 141 milestone Aug 30, 2024
@ebernard ebernard merged commit e844f53 into develop Aug 30, 2024
7 of 8 checks passed
@ebernard ebernard deleted the bugs_13218 branch August 30, 2024 14:09
@marob
Copy link
Contributor

marob commented Sep 3, 2024

@ebernard Pourquoi ne pas plutôt changer le template en [disabled]="signedDocumentPolicyIsDisabled()" et garder le code TS tel quel ? C'est un fonctionnement plus logique.

@ebernard
Copy link
Contributor Author

ebernard commented Sep 3, 2024

@ebernard Pourquoi ne pas plutôt changer le template en [disabled]="signedDocumentPolicyIsDisabled()" et garder le code TS tel quel ? C'est un fonctionnement plus logique.

Oui, j'y ai pensé, mais en faisant cela, ça revient à faire un revert sur un changement qui a été fait pour la migration vers Angular 14.

@ebernard
Copy link
Contributor Author

ebernard commented Sep 3, 2024

@ebernard Pourquoi ne pas plutôt changer le template en [disabled]="signedDocumentPolicyIsDisabled()" et garder le code TS tel quel ? C'est un fonctionnement plus logique.

Oui, j'y ai pensé, mais en faisant cela, ça revient à faire un revert sur un changement qui a été fait pour la migration vers Angular 14.

Traité sur #2007

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working small pr embarquant peu de changements et à review rapide, ne nécessitant qu'un reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants