docs(storybook): update data table #841
Conversation
|
Storybook for this build: https://ds.equisoft.io/pr-841/ |
| @@ -0,0 +1,54 @@ | |||
| import { Table } from '@equisoft/design-elements-react'; | |||
There was a problem hiding this comment.
Il y a le story "Optimization" qui contient du text, il faudrait le mettre dans MDX sinon on ne le voit plus.
| <p style={{ marginBottom: '2rem' }}> | ||
| Additionally, if your columns use some dependencies, you can pass them via useRef instead, so you | ||
| don't have to recreate the columns object everytime the dependencies changes. | ||
| </p> | ||
| <Button type="button" buttonType='secondary' onClick={() => setAllowEditing(!allowEditing)}> |
There was a problem hiding this comment.
Le visuel CSS du onClick du bouton est un peu câché par le tableau. Peut-être le décaler un peu vers le haut rendrais ça plus clean ?
| * | ||
| * Additionally, if your columns use some dependencies, you can pass them via useRef instead, so you don't have to | ||
| * recreate the columns object everytime the dependencies changes. | ||
| */ | ||
| export const Optimization: Story = () => { |
There was a problem hiding this comment.
Pourquoi on appel ça Optimization alors qu'il serait plus logique selon moi d'appeler ça Editable, Editing, etc. 🤔
Qu'est-ce que vous en pensez ?
There was a problem hiding this comment.
L'objectif semblait de mettre en évidence l'optimisation expliquée dans le texte. Faudrait peut-être revoir le nommage et les stories elles-mêmes dans un 2e temps.
pylafleur
temporarily deployed
to
github-pages
GitHub Actions
Inactive
pylafleur
temporarily deployed
to
github-pages
GitHub Actions
Inactive
|
Webapp for this build: https://ds.equisoft.io/pr-841/webapp/ |
No description provided.