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 icon font family class #4920

Merged
merged 1 commit into from Nov 19, 2018
Merged

fix icon font family class #4920

merged 1 commit into from Nov 19, 2018

Conversation

epwinchell
Copy link
Contributor

@epwinchell epwinchell commented Nov 14, 2018

Several PF font icons were using the "fa" class instead of "pficon" for some reason. It works fine on master but is broken when doing PF4 testing. This PR corrects that.

@epwinchell
Copy link
Contributor Author

@miq-bot add_label bug, hammer/no

@epwinchell
Copy link
Contributor Author

@miq-bot assign @himdel

@himdel
Copy link
Contributor

himdel commented Nov 14, 2018

Makes sense, but I'm seeing a bunch more @epwinchell ...

$ ag fa.*pficon
app/decorators/miq_action_decorator.rb
19:      'fa pficon-cpu'

app/helpers/application_helper/toolbar/ems_container_center.rb
23:          'fa pficon-import fa-lg',

app/helpers/application_helper/toolbar/ems_containers_center.rb
21:          'fa pficon-import fa-lg',

app/helpers/application_helper/toolbar/ems_physical_infra_center.rb
83:      'fa pficon-screen fa-lg',

app/helpers/application_helper/toolbar/container_project_view.rb
22:      'fa pficon-topology',

app/helpers/application_helper/toolbar/x_vm_center.rb
253:      'fa pficon-screen fa-lg',

app/helpers/application_helper/toolbar/physical_server_center.rb
113:        'fa pficon-screen fa-lg',

app/helpers/application_helper/toolbar/cloud/instance_operations_button_group_mixin.rb
106:        'fa pficon-screen fa-lg',

app/helpers/application_helper/toolbar/dashboard_summary_toggle_view.rb
22:      'fa pficon-topology',

app/helpers/application_helper/toolbar/stack_orchestration_template_center.rb
23:          'fa pficon-info fa-lg',

@epwinchell
Copy link
Contributor Author

@miq-bot add_label wip

@miq-bot miq-bot changed the title fix icon font family class [WIP] fix icon font family class Nov 14, 2018
@miq-bot miq-bot added the wip label Nov 14, 2018
@epwinchell
Copy link
Contributor Author

@miq-bot rm_label wip

@miq-bot miq-bot changed the title [WIP] fix icon font family class fix icon font family class Nov 16, 2018
@miq-bot miq-bot removed the wip label Nov 16, 2018
@miq-bot
Copy link
Member

miq-bot commented Nov 16, 2018

Checked commit https://github.com/epwinchell/manageiq-ui-classic/commit/48e6e45b3ccc08f2f9b26eab079b4971528086f5 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
12 files checked, 1 offense detected

**

  • 💣 💥 🔥 🚒 - Linter/Haml - Linter::Haml STDERR:
warning: parser/current is loading parser/ruby23, which recognizes
warning: 2.3.6-compliant syntax, but you are running 2.3.3.
warning: please see https://github.com/whitequark/parser#compatibility-with-ruby-mri.

@himdel himdel added this to the Sprint 99 Ending Nov 19, 2018 milestone Nov 19, 2018
@himdel
Copy link
Contributor

himdel commented Nov 19, 2018

LGTM, I'm not seeing any more of these :)

(also checked for fa + ff-, and fa + glyphicon)

@himdel himdel merged commit 8be68a9 into ManageIQ:master Nov 19, 2018
@epwinchell epwinchell deleted the fix_icon_classes branch March 20, 2019 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants