-
Notifications
You must be signed in to change notification settings - Fork 357
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Prefer fonticon for resource pool if it's not vapp #4048
Conversation
@miq-bot assign @h-kataria |
35a914b
to
83b476b
Compare
@miq-bot add_label wip |
@miq-bot unassign @h-kataria |
@epwinchell unrecognized command 'unassign', ignoring... Accepted commands are: add_label, add_reviewer, assign, close_issue, move_issue, remove_label, rm_label, set_milestone |
@miq-bot assign @epwinchell |
83b476b
to
b1ac112
Compare
@@ -4,12 +4,13 @@ def self.fonticon | |||
end | |||
|
|||
def fileicon | |||
vapp ? '100/vapp.png' : '100/resource_pool.png' | |||
vapp ? '100/vapp.png' : nil | |||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@skateman how about svg/vmware.svg
?
b1ac112
to
0326b9f
Compare
@skateman I approve |
Checked commit skateman@0326b9f with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 |
The
resource_pool.png
is obsolete, thepficon pficon-resource-pool
fully replaces it, but I haven't found any alternative for thevapp.png
.Parent issue: #4051
@miq-bot add_label gaprindashvili/no, graphics
@miq-bot add_reviewer @epwinchell