-
Notifications
You must be signed in to change notification settings - Fork 2
Contextual help #167
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
Contextual help #167
Conversation
R/contextual_help.R
Outdated
| margin-left: 5px; | ||
| cursor: pointer; | ||
| vertical-align: middle; | ||
| visibility: hidden; |
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.
nemělo by to být defaultně viditelné (aby bylo jasné, že nápověda k prvku existuje)?
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.
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.
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.
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.
Adds an ability to provide contextual help for selected tools.
with_help()function wraps content with an on-hover help iconhelp_iteminwith_help()must have a corresponding help definition inhelp_items()function