Skip to content
Snippets Groups Projects
Commit 6b8ab1e8 authored by P. FONTANET's avatar P. FONTANET
Browse files

fix(checkbox) la div messages devrait toujours être présente

parent 7e7f8d30
No related branches found
No related tags found
No related merge requests found
......@@ -4,6 +4,7 @@
- feat(form-password): ajout de la propriété `autocomplete="current-password"` par défaut
- feat(radio-rich): ajout du support des pictogrammes
- fix(form-checkbox): la `div` messages devrait toujours être présente
- fix(form-password): `aria-describedby` présent même en l'absence de message
- fix(form-password): `undefined` dans les `id`
- fix(form-password): quelques messages non traduits
......
......@@ -8,7 +8,7 @@
<input
role="checkbox"
[attr.aria-checked]="indeterminate ? 'mixed' : checked"
[attr.aria-describedby]="hasMessages() ? messagesGroupId : null"
[attr.aria-describedby]="messagesGroupId"
type="checkbox"
[id]="inputId"
[attr.name]="name || null"
......@@ -20,7 +20,7 @@
<ng-content *ngIf="!label" select="[label]"></ng-content>
<span *ngIf="hint" class="fr-hint-text">{{ hint }}</span>
</label>
<div *ngIf="hasMessages()" class="fr-messages-group" [id]="messagesGroupId" aria-live="polite">
<div class="fr-messages-group" [id]="messagesGroupId" aria-live="polite">
<p *ngIf="error" class="fr-message fr-message--error">{{ error }}</p>
<p *ngIf="valid" class="fr-message fr-message--valid">{{ valid }}</p>
</div>
......
......@@ -15,6 +15,9 @@ import { DefaultControlComponent } from '../../_commons';
},
],
})
/*
* - fix 1.6 : La div messages est toujours présente même dans un groupe (cf. DSFR 1.9)
*/
export class DsfrFormCheckboxComponent extends DefaultControlComponent<boolean> implements OnInit {
/**
* Message d'erreur, quand il est présent les couleurs du contrôle changent.
......@@ -67,9 +70,4 @@ export class DsfrFormCheckboxComponent extends DefaultControlComponent<boolean>
this.defaultChecked = super.value;
this.messagesGroupId = `${this.inputId}-messages`;
}
/** @internal */
hasMessages(): boolean {
return !!this.error || !!this.valid;
}
}
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment