feat(slash)!: update loader component style

[MartinWeb]

Fix #877

Default :

FullScreen :

Content :

[MartinWeb]

attendre le retour de @adrien-dos pour le nom du variant content

[JLou]

j'aurais bien remplacé default par "inline" ou un truc du style. Pour que ce soit + parlant

[MartinWeb]

done

[JLou]

Ca fait un sacré breaking quand meme. On proposerait pas les 2 composants en parallele ? Et c'est quoi la bonne pratique pour arreter d'utiliser le gros carré qui tourne ?

[MartinWeb]

Ca fait un sacré breaking quand meme. On proposerait pas les 2 composants en parallele ? Et c'est quoi la bonne pratique pour arreter d'utiliser le gros carré qui tourne ?

Ca fait un gros breaking effectivement. Par contre, pour conserver les 2 en parallèle, comment on ferait en terme de nommage ? on mettrait l'ancien en LoaderLegacy et en deprecated ?
Je comprends pas bien pour la bonne pratique pour arrêter d'utiliser l'ancien, tu veux dire comment on migre sur le nouveau ?

[JLou]

Je comprends pas bien pour la bonne pratique pour arrêter d'utiliser l'ancien, tu veux dire comment on migre sur le nouveau ?

Je vais donner un exemple, ca sera surement + parlant.
Sur les applis de souscriptions souvent on masque la page entiere par ce carré qui tourne.
La, le nouveau loader c'est juste un cercle qui tourne, non ?

Est ce qu'on doit faire un composant qui masque la page, avec le loader rond, ou est ce que on doit repenser nos apps pour mettre le loader a l'endroit ou on a cliqué par exemple ?

EDIT: Je viens de voir le mode fullscreen, qui répond partiellement a ma question.

[MartinWeb]

Je comprends pas bien pour la bonne pratique pour arrêter d'utiliser l'ancien, tu veux dire comment on migre sur le nouveau ?

Je vais donner un exemple, ca sera surement + parlant. Sur les applis de souscriptions souvent on masque la page entiere par ce carré qui tourne. La, le nouveau loader c'est juste un cercle qui tourne, non ?

Est ce qu'on doit faire un composant qui masque la page, avec le loader rond, ou est ce que on doit repenser nos apps pour mettre le loader a l'endroit ou on a cliqué par exemple ?

EDIT: Je viens de voir le mode fullscreen, qui répond partiellement a ma question.

Oui tout à fait, avant le loader n'était presque pas partie intégrante de l'application, c'était un wrapper avec le contenu en children.
Là sur cette version ce n'est plus le cas, il fait partie du contenu de l'application, ce qui nous permet de le faire à l'endroit où l'on veut pour afficher un message de chargement.
Le variant default permet de l'afficher comme du texte plus ou moins.
Le variant content permet de l'afficher sur tout le contenu disponible (height 100% et centrage) pour un bloc de contenu dans le cas où on aurait un chargement pour une partie spécifique de la page.
Le variant fullScreen permet lui d'afficher un message de chargement en plein écran (en dehors du loader, headerTitle et footer).

[JLou]

j'aurais bien remplacé default par "inline" ou un truc du style. Pour que ce soit + parlant

[JLou]

Pour moi il va manquer des stories de mise en contexte pour les autres variants.

J'aimerais voir une story avec une full page genre form, et devant le fullscreen, et une story avec une page complete, mais le loader qui est en mode content, et qui ne prend que l'espace du parent (si j'ai bien compris le mode content)

[MartinWeb]

bah y'a pas de mode form réellement, le fullScreen c'est du fullScreen et on ne masque plus de contenu comme l'ancien Loader pouvait le faire.
Pour le content je peux voir pour faire quelque chose

[JLou]

j'ai ptetre mal exprimé ce que je voulais. En gros j'aimerais que la story soit une page de formulaire fictive, avec le loader en mode fullscreen devant, pour voir le rendu.
Ca permettra a nos utilisateurs du loader actuel de voir comment remplacer leur loader en mode wrapper actuel egalement.

Un truc du genre :

<form>

<Loader mode="fullscreen" />

<TextInput label="name" name="name"/>
<DateInput label="date of birth" name="dob" />

{* etc... *}
</form>

[MartinWeb]

comme je te disais par message ce genre d'implémentation n'existera plus. Si on a un formulaire, avec un loader avant on aurait quelque chose comme cela plutôt :

{isLoading ? 
  <Loader mode="fullscreen" /> 
: 
<form>
  <TextInput label="name" name="name"/>
  <DateInput label="date of birth" name="dob" />
</form>

[sonarqubecloud]

Quality Gate failed

Failed conditions
36.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

[pplancq]

il me semble que

Suggested change

	"./logo-axa.svg": "./files/assets/logo-axa.svg",
	"./dist/common/assets/logo-axa.svg": "./files/assets/logo-axa.svg",
	"./spinner.svg": "./files/assets/spinner.svg",
	"./*.svg": "./files/assets/*.svg",

devrait marché