entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 44 Releases Activity

a11y: add accordion pattern to form view sections (#73116) #61

Merged
fpeters merged 1 commits from wip/73116-a11y-foldable-accordion into main 2023-03-24 10:06:33 +01:00
Conversation 3 Commits 1 Files Changed 4 +27 -9
fpeters commented 2023-01-24 18:17:12 +01:00
Owner
Copy Link
No description provided.
fpeters force-pushed wip/73116-a11y-foldable-accordion from 3182057598 to 8b0e760285 2023-02-03 09:54:51 +01:00 Compare
fpeters commented 2023-02-03 10:55:26 +01:00
Author
Owner
Copy Link

De redmine : ça ennuie d'avoir un vrai <button> qu'il faut alors déstyler donc voici avec un span role=button.

De redmine aussi :

Enfin ce point ne semble pas être traité dans le patch :
The accordion header button element has aria-controls set to the ID of the element containing the accordion panel content.

Mais là il y avait déjà, aria-controls="sect-dataview", donc soit tu es passé à côté, soit je n'ai pas compris l'affaire.

Aussi dans cette nouvelle version de la branche j'applique le même traitement à la section "données de traitement".

De redmine : ça ennuie d'avoir un vrai `<button>` qu'il faut alors déstyler donc voici avec un span role=button. De redmine aussi : > Enfin ce point ne semble pas être traité dans le patch : > The accordion header button element has aria-controls set to the ID of the element containing the accordion panel content. Mais là il y avait déjà, aria-controls="sect-dataview", donc soit tu es passé à côté, soit je n'ai pas compris l'affaire. Aussi dans cette nouvelle version de la branche j'applique le même traitement à la section "données de traitement".
tjund commented 2023-03-13 17:16:32 +01:00
Owner
Copy Link

Mais là il y avait déjà, aria-controls="sect-dataview", donc soit tu es passé à côté, soit je n'ai pas compris l'affaire.

Oui, je suis passé à côté.

L'étape prochaine sera de voir s'il est possible de mutualiser le code JS foldable de combo et wcs dans gadjo.

> Mais là il y avait déjà, aria-controls="sect-dataview", donc soit tu es passé à côté, soit je n'ai pas compris l'affaire. Oui, je suis passé à côté. L'étape prochaine sera de voir s'il est possible de mutualiser le code JS foldable de combo et wcs dans gadjo.
tjund requested review from tjund 2023-03-13 17:16:45 +01:00
tjund approved these changes 2023-03-13 17:16:59 +01:00
fpeters force-pushed wip/73116-a11y-foldable-accordion from 8b0e760285 to 2d19cbf07c 2023-03-24 09:14:03 +01:00 Compare
fpeters force-pushed wip/73116-a11y-foldable-accordion from 2d19cbf07c to ea157971fc 2023-03-24 09:40:56 +01:00 Compare
fpeters commented 2023-03-24 10:06:27 +01:00
Author
Owner
Copy Link

Il a fallu ajuster pour réindenter les gabarits pour djhtml.

Il a fallu ajuster pour réindenter les gabarits pour djhtml.
fpeters merged commit b253bdc6be into main 2023-03-24 10:06:33 +01:00
fpeters deleted branch wip/73116-a11y-foldable-accordion 2023-03-24 10:06:33 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
tjund
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#61
No description provided.
Delete Branch "wip/73116-a11y-foldable-accordion"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?