Fix icon-class, re-add default-icon support for Actions

[raimund-schluessler]

This brings back the support for a default-icon and fixes the support of icon for the Actions component.
Follow-up to #3006, fixes #3016.

Please test this well, I only checked in in the docs.

[CarlSchwan]

Probably make sense to at some point have a slot for material icon since we are migrating to that slowly in other places.

But definitively not blocking or urgent, let's first get back default-icon so we can port our apps to Nextcloud Vue 6 :)

Thanks!

[raimund-schluessler]

Probably make sense to at some point have a slot for material icon since we are migrating to that slowly in other places.

But definitively not blocking or urgent, let's first get back default-icon so we can port our apps to Nextcloud Vue 6 :)

Thanks!

The component has a slot already, so you can start migrating immediately 😉
That was actually the reason why I didn't bother to re-implement the icon class support, because one can do everything with material design icons just as well.

[PVince81]

👍

[PVince81]

I did some migrations already back then on Talk and remember that it wasn't always straightforward because you had to find a matching MD icon and sometimes styles started to go crazy. This is why I wouldn't want to rush removing the icon attribute support.

[nickvergessen]

I did some migrations already back then on Talk and remember that it wasn't always straightforward because you had to find a matching MD icon and sometimes styles started to go crazy. This is why I wouldn't want to rush removing the icon attribute support.

Last week I replaced a lot of icons and buttons in Talk, it was mostly all pretty straight forward