Skip to content

Conversation

@hlageek
Copy link
Contributor

@hlageek hlageek commented Aug 10, 2025

Adds an ability to provide contextual help for selected tools.

  • with_help() function wraps content with an on-hover help icon
  • the argument help_item in with_help() must have a corresponding help definition in help_items() function

@hlageek hlageek requested a review from skvrnami August 10, 2025 17:11
margin-left: 5px;
cursor: pointer;
vertical-align: middle;
visibility: hidden;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nemělo by to být defaultně viditelné (aby bylo jasné, že nápověda k prvku existuje)?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Původně jsem to tak zamýšlel, ale pak jsem si zkusil představit, jak by to vypadalo, kdyby ten otazník byl u všeho. Takže jsem zvolil tuhle cestu v logice toho, aby requal na první pohled nepůsobil složitě a komplexita byla spíš treasure hunt pro pokročilejší usery. Uznávám, že u helpu tahle logika jít vstříc jednoduchému prvnímu dojmu trochu skřípe.... Ale představa, že na mě kouká 10 otazníků, k tomu nějaký ikony... Případně kompromis by byl dát visibility jako argument to wrap funkce, default jako hidden, ale třeba zrovna u csv importu by se zadalo visible... Představuju si ale, že jakmile user zjistí, že jsou někde helpery schovaný, začne se zkoušet dívat, jestli není nějaký tam, kde by ho potřeboval. Obecně bych teda komplexitu schovával, ale v tomhle konkrétním případě smysl visibility chápu.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zatím je otazník jen u tohohle, proto bych to dal visible by default. V případě, že bude otazníků přibývat, tak můžem zvážit jejich zobrazení/skrytí pomocí nějakého nastavení uživatele.

@skvrnami skvrnami merged commit 28ac1ba into main Aug 11, 2025
4 checks passed
@skvrnami skvrnami deleted the contextual-help branch August 11, 2025 10:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants