Skip to content
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

Fix broken Automate icons #351

Merged
merged 5 commits into from
Feb 14, 2017
Merged

Fix broken Automate icons #351

merged 5 commits into from
Feb 14, 2017

Conversation

epwinchell
Copy link
Contributor

@epwinchell epwinchell commented Feb 10, 2017

The following outdated "product" font family icons need to be replaced:

product-ae_instance => fa-file-text-o
product-ae_namespace => pficon-folder-close

Closes #349

@epwinchell
Copy link
Contributor Author

@miq-bot add_label bug, euwe/no

@epwinchell
Copy link
Contributor Author

@miq-bot add_label wip

@epwinchell
Copy link
Contributor Author

@miq-bot assign @skateman

@himdel
Copy link
Contributor

himdel commented Feb 10, 2017

@epwinchell if you update the description to say Closes #349 (or fixes), it will auto-close when merged..

@epwinchell epwinchell changed the title [WIP] Fix broken Automate icons Fix broken Automate icons Feb 13, 2017
@epwinchell
Copy link
Contributor Author

@miq-bot rm_label wip

@miq-bot miq-bot removed the wip label Feb 13, 2017
@epwinchell
Copy link
Contributor Author

@miq-bot assign @himdel

@miq-bot miq-bot assigned himdel and unassigned skateman Feb 13, 2017
@skateman
Copy link
Member

@miq-bot add_label wip
Do not merge as there is a PR against this PR, sorry @epwinchell, forgot to add this yesterday 😢

@miq-bot miq-bot added the wip label Feb 14, 2017
@miq-bot miq-bot changed the title Fix broken Automate icons [WIP] Fix broken Automate icons Feb 14, 2017
@himdel
Copy link
Contributor

himdel commented Feb 14, 2017

OK, waiting until @epwinchell merges that :)

@epwinchell epwinchell changed the title [WIP] Fix broken Automate icons Fix broken Automate icons Feb 14, 2017
@epwinchell
Copy link
Contributor Author

@miq-bot rm_label wip

@miq-bot miq-bot removed the wip label Feb 14, 2017
@himdel
Copy link
Contributor

himdel commented Feb 14, 2017

Test failure seems relevant..

  1) TreeNode::MiqAeMethod#icon returns with product product-method
     Failure/Error: expect(subject.icon).to eq(icon)
     
       expected: "product product-method"
            got: "product product-ae_method"
     
       (compared using ==)
     Shared Example Group: "TreeNode::Node#icon" called from ./spec/presenters/tree_node/miq_ae_method_spec.rb:8
     # ./spec/shared/presenters/tree_node/common.rb:20:in `block (3 levels) in <top (required)>'

(no idea which one is right, we seem to have both)

Use the MiqAe decorators in TreeNode::MiqAeNode when possible
@miq-bot
Copy link
Member

miq-bot commented Feb 14, 2017

Checked commits https://github.com/epwinchell/manageiq-ui-classic/compare/8c57772b8da558c6c4ccfadfe576e5ffe342c616~...a6f494663389cc81b4eb02ad96657d938e448307 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
12 files checked, 0 offenses detected
Everything looks good. 🍰

@epwinchell
Copy link
Contributor Author

@himdel All set

@himdel himdel merged commit 3a08db2 into ManageIQ:master Feb 14, 2017
@himdel himdel added this to the Sprint 55 Ending Feb 27, 2017 milestone Feb 14, 2017
@epwinchell
Copy link
Contributor Author

@miq-bot add_label graphics

@epwinchell epwinchell deleted the broken_ae_icons branch October 25, 2017 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Broken icons in Automate Explorer
4 participants