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

🛹 Feature: Umsetzung eines Loading-Button mittels Expert-Slot #2476

Closed
deleonio opened this issue Feb 27, 2023 · 4 comments · Fixed by #5676
Closed

🛹 Feature: Umsetzung eines Loading-Button mittels Expert-Slot #2476

deleonio opened this issue Feb 27, 2023 · 4 comments · Fixed by #5676
Assignees
Labels
a11y-retest desybri The task is assigned to the DESYBRI design system. feature New feature or request

Comments

@deleonio
Copy link
Contributor

Note

  • Ich achte beim Erstellen des Issues darauf, dass ich keine fachlichen und unternehmensinternen Details preisgebe.

image

@deleonio deleonio added the feature New feature or request label Feb 27, 2023
@deleonio deleonio added a11y-retest desybri The task is assigned to the DESYBRI design system. labels Feb 27, 2023
@Chrisdo82
Copy link
Contributor

Chrisdo82 commented Nov 27, 2023

ich habe einen LoadingButton gebaut der nach dem Wechsel des Ladezustands, noch für ein paar Sekunden einen Haken anzeigt. Hier zu sehen: https://codesandbox.io/p/sandbox/minimal-kolibri-sample-forked-r3df27?file=%2Fsrc%2FButton.tsx%3A86%2C27

Allerdings funktioniert das Ganze nicht, wenn _iconOnly gesetzt ist, da dann der expert Slot nicht verwendet wird.

Hast du eine Idee, wie wir die gleiche Funktionalität auch für _iconOnly gewährleisten können?

@deleonio deleonio changed the title 🛹 Feature: Umgang mit Loading-Button / disabled mit betrachten 🛹 Feature: Umsetzung eines Loading-Button mittels EXpert-Slot Nov 27, 2023
@deleonio deleonio changed the title 🛹 Feature: Umsetzung eines Loading-Button mittels EXpert-Slot 🛹 Feature: Umsetzung eines Loading-Button mittels Expert-Slot Nov 27, 2023
@deleonio
Copy link
Contributor Author

Eine Lösungsidee wäre, eine Wrapper-Component umzusetzen und _hideLabel nicht an die KoliBri-Komponente weiterzureichen.

Einfach über den Expert-Slot beide Modi umsetzen:

  1. Mit Label so wie es jetzt implementiert ist
  2. Nur Icon mit Tooltip über Expert-Slot hereinreichen

@deleonio
Copy link
Contributor Author

Duplikat: #4747

@Chrisdo82
Copy link
Contributor

@deleonio
Der Lösungsansatz funktioniert, jedoch komme ich nicht drumherum eine _customClass mit _variant="custom" zu setzen, damit das CSS auch für IconOnly-Buttons funktioniert.

Ich hatte erst die Hoffnung, dass ich mit _customClass="hide-label" weiterkomme, jedoch gibt es dann weiter unten ein display:none welches verhindert, dass der Expert Slot gerendert wird.
Ich wollte dann eine neue icon-only Klasse einführen, aber die scheint es schon im CSS von eurem BaseTheme zu geben. Ist das ein Überbleibsel welches mal entfernt werden sollte?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y-retest desybri The task is assigned to the DESYBRI design system. feature New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants