-
-
Notifications
You must be signed in to change notification settings - Fork 529
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Resource icons in resource groups tree (#16099)
### What does it do? Abstract logic for getting icon classes from `Resource\GetNodes` processor to the `modResource::getIconClasses`. Use the new method in `Resource\GetNodes` and `ResourceGroup\GetNodes`. Lock icon is not part of this - as it's specific for resource tree and has no real benefit in resource group tree, same with classes around permissions. ### Why is it needed? Fixes the BUG that resource group's tree doesn't show resource icons. (100% viable for 3.0) ### How to test Create resource group, add resource, check if it has an icon :) ### Related issue(s)/PR(s) Different approach in #16011
- Loading branch information
Showing
3 changed files
with
92 additions
and
59 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters