jwstegemann / fritz2

Easily build reactive web-apps in Kotlin based on flows and coroutines.
https://www.fritz2.dev
MIT License
636 stars 25 forks source link

Update tooltip.md with hint to include portalRoot #807

Closed jkapeller closed 8 months ago

jkapeller commented 9 months ago

I think a simple hint to include portalRoot in order to use components using the portalling mechanism would be helpful.

Lysander commented 9 months ago

thx for the hint!

I think we should find a proper solution for alle portalling components, not only for Tooltips. Also I rather dislike the warning state of the message and the too prominent placing at the very top above all.

I would propose to use the standard info state and place the blocks directly at the end of the "Basic Example" sections.

That said, I have encountered, that the documentation is outdated as consequence of PR #804.

This must also be fixed... would you dare to do so?

fixes #809

Lysander commented 8 months ago

Closed in favor of #817