App icon not visible on white theming

[jancborchardt]

Does this need another icon, or something in the code? cc @juliushaertl @nextcloud/theming @nickvergessen

[juliusknorr]

@jancborchardt Does it work when you reload the page after changing the theming color to white?

[juliusknorr]

We had an extra rule inside of the theming app to overwrite the icon for preview, but I think the general styling was handled in the notifications app. Since that one has moved from a background image to a <img> element, the preview is probably broken.

[nickvergessen]

https://github.com/nextcloud/notifications/blob/master/src/App.vue#L62

Does this not work anymore?

[juliusknorr]

Just tested, it works fine after reloading. Only the live preview after changing the theming color doesn't work anymore.

[nickvergessen]

Right, because vue.js doesn't recalculate the value