nextcloud-vue icon indicating copy to clipboard operation
nextcloud-vue copied to clipboard

[NcListItem] List items highlight on hover even when they don't do anything

Open kra-mo opened this issue 1 month ago • 3 comments

I'm not sure I fully understand the underlying behavior, but the outcome is certainly unexpected as a user (of apps):

Some list items do nothing when clicked (eg. items in "Always show images from" in Mail). They only exist to display other actions (like Delete). However, they still highlight and change the cursor to a pointer on hover:

Image

The item should only highlight in such a way if clicking on it does something. Not sure if this would be breaking existing behavior people rely on, or if this is something applications can already control somehow, but it should be fixed.

This also seems to not be the case with NcListItemIcon, but not sure.

kra-mo avatar Nov 20 '25 14:11 kra-mo

From the implementation, we do not have NcListItem's that "do not do anything". It is always a link. Developers use it as an invalid href="#" link, then they need a button or just show the info...

ShGKme avatar Nov 20 '25 14:11 ShGKme

Hm, ok. So use like this is unintended?

In that case, maybe it would be better to think about this in general, what could be used where, and whether we do want a component like that. Because currently, some apps do have non-activatable lists.

kra-mo avatar Nov 20 '25 15:11 kra-mo

So use like this is unintended?

Hard to tell.

The component should either support more cases out of the box, or be split into several less general component.

Usually I'm against super components with too many features, but in case I'm not sure.

ShGKme avatar Nov 23 '25 16:11 ShGKme