openwallet-foundation / digital-wallet-and-agent-overviews-sig

The objective of this SIG is to develop and maintain the Digital Wallets and Agents Overview. The overview should provide transparency of the characteristics of wallets and agents in order to allow for an objective comparison and effective decision making on which wallet or agent is applicable for your use case.
https://openwallet-foundation.github.io/digital-wallet-and-agent-overviews-sig/
Apache License 2.0
13 stars 15 forks source link

Show tooltips via an info icon #27

Open cre8 opened 4 months ago

cre8 commented 4 months ago

Right now the information about a column are placed as a tooltip:

Following the material design guide, I saw no information how to deal with this way https://m1.material.io/assets/0B3mOPoJlxiFhenh5SWhFdFlyajg/components-datatables-interaction-tooltip.png

It would look like this (small css errors): image

I personally don't like this, but open to be discussed.

samuelmr commented 4 months ago

I'm not fond of repeating icons.

Perhaps this could be implemented in the way that the icons are there but hidden by default and only shown on hover.

.info {
  margin-left: 5px;
  display: none;
}

th:hover .info {
  display: inline;
}
cre8 commented 4 months ago

We have the hover effect already with the tooltip. The point adding the info icon was to inform the user that there are information so he needs to move his mouse there.

Suggestion: we add another link in the navigation showing a page with some kind of glossar (what is the definition of each column)

maaikevanleuken commented 3 months ago

I agree with Samuel that the repetition of these icons is not preferable. At the same time, it would be great that at least some information is clearly visible in the overview. Maybe we can keep the hover-over for now, but add a row with descriptions directly below the 2nd row with characteristics, with some kind of toggle to collapse this row.

Another suggestion is to leave this as is for now, and validate this with our target audience (we will do this anyways, but specifically this usability aspect we can test).

cre8 commented 3 months ago

@maaikevanleuken I like your suggestion. I tagged this issue and would leave it open since we haven't found a solution for this problem yet.