-
Notifications
You must be signed in to change notification settings - Fork 42
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
FileUpload.Item: 💥 Endret API og støtte for å sette ID på knappen #2824
Conversation
Slik at man kan lenke til knappen med en <a>-tag.
🦋 Changeset detectedLatest commit: e3806c7 The changes in this PR will be included in the next version bump. This PR includes changesets to release 7 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Storybook demoEndringer til review: 129c6907bd | 60 komponenter | 209 stories |
I hvilke tilfeller vil man trenge å flytte fokus til knapp her? Tenker f.eks bare hva forskjellen på å slette filen for brukeren ved en action vs å flytte fokuset til filen og la brukeren slette den manuelt er 🤔 |
Nå har vi diskutert dette, og kommet fram til at vi ønsker å samle alle props som har med knappen å gjøre til et objekt. Noe sånt som dette: action={{
type: "delete",
onClick: () => onDelete(file),
id: "min-id",
title: "Slett filen 'filnavn'", // eventuelt
}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉 Ser bra ut. Mangler bare å oppdatere JSDOC
* @example
* Status & actions
* ```jsx
* <FileUpload.Item file={file} status="uploading" />
* <FileUpload.Item file={file} status="downloading" />
* <FileUpload.Item file={file} itemAction="retry" onRetry={...} />
* <FileUpload.Item file={file} itemAction="delete" onDelete={...} />
* ```
*/
Bra spottet! |
Ser det greit ut slik?
Eller bør det formateres slik prettier ville gjort det? |
Funker slik det er nå 👍 |
Slik at man kan lenke til knappen med en -tag.