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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃摉 Help: description in ToasterService #5442

Closed
Chrisdo82 opened this issue Oct 17, 2023 · 6 comments 路 Fixed by #5477
Closed

馃摉 Help: description in ToasterService #5442

Chrisdo82 opened this issue Oct 17, 2023 · 6 comments 路 Fixed by #5477
Assignees
Labels
desybri The task is assigned to the DESYBRI design system. help wanted Extra attention is needed

Comments

@Chrisdo82
Copy link
Contributor

Note

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

Hilfeanfrage

Hi, die Dokumentation geht nur darauf ein wie man den ToasterService verwendet, wenn die description ein string ist. Das ist bei uns aber eher selten der Fall. Stattdessen m枚chte ich alles reingeben k枚nnen und hier nicht eingeschr盲nkt werden. Mit alles meine ich Markup.

@deleonio

@Chrisdo82 Chrisdo82 added the help wanted Extra attention is needed label Oct 17, 2023
@deleonio
Copy link
Contributor

@Chrisdo82, der Tooltip ist nur f眉r kurze Texte bis glaube 32 (80, Label) vorgesehen. F眉r richtige Overlays haben wir dir Popover-Komponente mit Slot bereitgestellt.

-> https://public-ui.github.io/docs/components/popover

@Chrisdo82
Copy link
Contributor Author

Chrisdo82 commented Oct 17, 2023

Warum Tooltips? Ich spreche vom Toast-Element?! Hast du evtl. aufs falsche Ticket geantwortet?

Mit dem unten stehenden Codeschnipsel, komme ich der L枚sung recht nah.
Damit das funktioniert, br盲uchte ich allerdings eine HTML Referenz aufs einzelne Toast-Element.
Ohne diese Referenz, kann ich die 1.7.x Version leider nicht nutzen, da eure Anpassung die vorherige Toast Nutzung leider nicht mehr funktionieren l盲sst (h盲tte m.E. dadurch eigentlich ins 2.x Release geh枚rt).

toaster.enqueue({
  type: notification.type,
  label: notification.headline,
  description: (() => {
    if (typeof notification.message === 'string') {
      return notification.message
    }
    ReactDOM.render(<>{notification.message}</>, ref.current)
    return ''
  })(),
})

@deleonio
Copy link
Contributor

Hi @Chrisdo82,

Danke f眉r die R眉ckmeldung.

  • Wenn Du die Toast-Komponent weiterverwendest, wie bisher - also ohne ToastService, kannst Du ja auch auf 1.7 wechseln. Oder?
  • Klar der ToastService verh盲lt sich anders, ja.
  • Okay, Du brauchst am besten Referenzen auf die Toast's.

@sdvg Hast du da eine Idee?

@deleonio deleonio added the desybri The task is assigned to the DESYBRI design system. label Oct 20, 2023
@Chrisdo82
Copy link
Contributor Author

Chrisdo82 commented Oct 20, 2023

@deleonio

Wenn Du die Toast-Komponent weiterverwendest, wie bisher - also ohne ToastService, kannst Du ja auch auf 1.7 wechseln. Oder?

Leider nicht, zum einen scheint es kein _heading mehr zu geben (ok, das kann ich ersetzen mit _label) und zum anderen scheinen auch die Styles von euch angepasst worden zu sein. Zumindest sind sie nicht mehr absolut positioniert. Ich wei脽 nicht wie sinnvoll es ist, hier auf unserer Seite einen Workaround drumzubasteln.

@deleonio
Copy link
Contributor

@Chrisdo82

Wir arbeiten an einer Render-Funktion, wie bei der Table. Hier wird die Referenz auf den Slot 眉bergeben, so dass eigener Kontext gerendert werden kann.

@Chrisdo82
Copy link
Contributor Author

Funktioniert, danke :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
desybri The task is assigned to the DESYBRI design system. help wanted Extra attention is needed
Projects
Archived in project
3 participants