Overlay in available page #1449
Conversation
|
would be nice to summarize the solution (UI change) in a few words |
| @@ -41,7 +41,10 @@ | |||
| end | |||
| %> | |||
| <tr> | |||
| <td><%= overlay_description((name + "\n" + description).strip) %></td> | |||
| <td> | |||
| <%= name %> | |||
There was a problem hiding this comment.
Why do you have name in both places?
There was a problem hiding this comment.
I kept what was in the tooltip previously.
|
It works. Yet a) since now the behavior of (more) is different from other pages: for instance the tool list from menu preservers old behaviour, I would rename one of them, say into (help), (about), (info), +, ... or use a unicode symbol or emoji 🛈, ℹ️ , 💡, 🤷, 🔧 or similar. Underlining the tool name so user is enticed to point the mouse is good too. b) I think @bryancaron suggested that we might wish to add scroll bar if tool info is over one page. I know that so far tool descriptions are concise, so does not feel critical. Maybe check with him. |
| <td><%= overlay_description((name + "\n" + description).strip) %></td> | ||
| <td> | ||
| <%= name %> | ||
| <%= closable_overlay_content_link(name, "(iz)", description) %> |
There was a problem hiding this comment.
What "(iz)" is? Is it some abbreviation? Would "(i)" be better?
|
Thanks for renaming 'more', though I am not sure what "(iz)" stands for |
This PR replace #t1440