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

feat: playground #6

Merged
merged 7 commits into from
Jul 11, 2024
Merged

feat: playground #6

merged 7 commits into from
Jul 11, 2024

Conversation

revolunet
Copy link
Collaborator

@revolunet revolunet commented Jun 28, 2024

  • Ajout d'une copie du playground rjsf
  • J'ai mis DSFR par défaut et gardé le thème bootstrap 4 pour avoir un référentiel
  • Ajustements sur quelques widgets
  • Ajout de Radio/Textarea
  • En dev ça à l'air de bien marcher
  • J'ai ajouté un job pour envoyer sur gh-pages mais je ne suis pas 100% sûr !

@revolunet revolunet marked this pull request as draft June 28, 2024 16:15
@EmileRolley
Copy link
Collaborator

Trop bien ! Je vois que c'est en draft, tu as encore des choses à ajouter ?

@revolunet
Copy link
Collaborator Author

je vais re-nettoyer un peu;

dans le hintText que tu avais implémenté, il me semble que cette feature (icon clickable avec aide) n'existe pas dans le DSFR ? il y a seulement description qui est toujours visible.

Sauf erreur il y a donc deux notions a gérer : la ui:description et le ui:help

@EmileRolley
Copy link
Collaborator

EmileRolley commented Jul 1, 2024

je vais re-nettoyer un peu;

Ok, je ferai une review complète à ce moment là alors (sauf si tu as besoin d'en avoir une maintenant ?)

dans le hintText que tu avais implémenté, il me semble que cette feature (icon clickable avec aide) n'existe pas dans le DSFR ? il y a seulement description qui est toujours visible.

Sauf erreur il y a donc deux notions a gérer : la ui:description et le ui:help

Oui le hintText des <Input> de react-dsfr sont tout le temps visible, j'ai donc implémenté une la possibilité de les cacher. En effet, dans le cas de catala-dsfr, ça altérait grandement la lisibilité du formulaire. A voir donc si on a envie de garder ce composant par défaut, ou simplement utiliser les composant Input du DSFR comme prévu.

@EmileRolley
Copy link
Collaborator

Sauf erreur il y a donc deux notions a gérer : la ui:description et le ui:help

Après avoir lu la doc de ces propriétés, il semblerait en effet, que ce serait plus pertinent d'utiliser ui:description pour afficher les indications sur le champs plutôt que ui:help que l'on utilise actuellement sur catala-dsfr. Reste à savoir si on ne devrait pas avoir une propriété ui:collasableDescription pour le composant custom actuel et utiliser le contenu de ui:description pour le paramètre hintText de l'input de react-dsfr ?

@revolunet
Copy link
Collaborator Author

Vu que le ui:help existe dans rjsf, autant le garder avec ton mécanisme de toggle ?
et le ui-description lui permet d'afficher la description statique ?

@EmileRolley
Copy link
Collaborator

Vu que le ui:help existe dans rjsf, autant le garder avec ton mécanisme de toggle ? et le ui-description lui permet d'afficher la description statique ?

Les deux sont supportés nativement pas rjsf. ui:help est utilisez pour afficher un message d'information pendant que le champ est entrain d'être modifié. Alors que ui:description a pour but de décrire ce qui est attendu pour le champ.
Cependant, comme il ne me semble pas que les composant de react-dsfr ne permettent pas de faire cette nuance, il serait en effet logique de garder les ui:description pour les hintText des composants et ui:help pour le fonctionnement actuel 👍

@revolunet
Copy link
Collaborator Author

J'ai fait quelques ajustements :)

@revolunet revolunet marked this pull request as ready for review July 7, 2024 13:28
@EmileRolley
Copy link
Collaborator

J'ai fait quelques ajustements :)

J'essaie de tester ça demain matin !

Copy link
Collaborator

@EmileRolley EmileRolley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ca à l'air de bien fonctionner sur la plus part des exemples. Cependant, les nested objects/array avec le composant Tabs de react-dsfr ne sont pas interactifs. Mais je pense que l'on peut quand même merge la PR pour tester le GH Pages et commencer à itérer sur le widgets ensuite 👌

@EmileRolley EmileRolley merged commit f436d55 into main Jul 11, 2024
1 check passed
@revolunet
Copy link
Collaborator Author

trop cool. oui il y a un souci avec les tabs

@revolunet revolunet deleted the playground branch July 11, 2024 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants