You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I am adding 13 compatibility for the sharepoint app. Essentially, it registered a backend for files_external. Now, apparently something changed about generating the icons for the mounts? When I create a share point mount, the Files app will want to display apps/sharepoint/img/folder-sharepoint.svg, which does not exist. Thus, no icon is shown, just whitespace:
For comparison, an external mount of a Local type will have apps/theming/img/core/filetypes/folder-external.svg as icon – themed.
When I copy over folder-external.svg from core to the sharepoint destination, this will be shown, but unthemed.
What's the correct way to have a themed icon displayed in this case? @nextcloud/theming @icewind1991
Removing it from there should just load the default external folder icon that is also themed. Not sure if there is a way to specify the icon from the sharepoint app side.
Activity
skjnldsv commentedon Jan 18, 2018
@juliushaertl :)
juliusknorr commentedon Jan 18, 2018
@blizzz The folder-sharepoint icon is currently hardcoded in the files_external app:
https://github.com/nextcloud/server/blob/master/apps/files_external/js/statusmanager.js#L563-L565
Removing it from there should just load the default external folder icon that is also themed. Not sure if there is a way to specify the icon from the sharepoint app side.
blizzz commentedon Jan 18, 2018
@juliushaertl thx! That's actually old code, funny that it apperently only pops up now. I'll remove it, we're just gonna stick with the default icon.
[-]missing/unthemed icon for external storages[/-][+]missing/unthemed icon for sharepoint[/+]