nextcloud / groupware

Meta repo for the groupware team
9 stars 0 forks source link

Migrate to material icons #38

Closed ChristophWurst closed 2 years ago

ChristophWurst commented 2 years ago

Describe the task

Nextcloud 25 has a hard breaking change and we need to change our icons for it https://github.com/nextcloud/server/issues/32060

Affected components

To do

ChristophWurst commented 2 years ago

How far are we with this migration @GretaD?

GretaD commented 2 years ago

How far are we with this migration @GretaD?

99% for mail, there are only 2 open PRs, i did a general test last week and i think thats all we have to migrate here(one pr is blocked) On calendar i think its all done, but i havent test it yet like i did with mail to see if i missed smth. Contacts, i havent started yet.

If you need an ETA, i can finish all this week (but the blocked PR).

ChristophWurst commented 2 years ago

https://github.com/nextcloud/calendar-availability-vue/blob/main/src/CalendarAvailability.vue also uses two icon-* classes that need replacing

GretaD commented 2 years ago

Mail

Note: These can be done only after vue loading icon is released

GretaD commented 2 years ago

To do on contacts with stoppers

To do but that are leftovers, and will be done in one pr this week

GretaD commented 2 years ago

To do on contacts, no stoppers

ChristophWurst commented 2 years ago

https://github.com/nextcloud/contacts/issues/2960 misses an icon upgrade.

ChristophWurst commented 2 years ago

Bildschirmfoto vom 2022-10-10 12-36-16