Add folder icon to Site Assets root node in Resource Manager.#7314
Conversation
valadas
left a comment
valadas
left a comment
There was a problem hiding this comment.
The code looks fine to me. Was this tested with the module used in the context of Group mode and User mode?
no, didn't realize it was used in other places, will test and report back |
|
Tested User and Group mode (had to fake out page url for the group, as I don't know how those actually work, i.e. http://dnn_7311.dnndev.me/default.aspx?groupid=1) Both look good. Also tested "Move" popup. 
##Group Mode ##User Mode |
|
Thanks for testing that out, it would be nice to put a bit of space between the icon and the text like the others have. |
32c02b4 to
7dc87bd
Compare
|
Done |
|
Awesome, thanks @r90727 |
3 participants
Fixes #7311
Summary
Added folder icon to Site Assets root node in Resource Manager. Reused icon rendering code from
dnn-rm-folder-list-item.tsx