Bug 1465081
Summary: | Formatting of Provider summary PDF file generated from provider summary page is very broken | ||
---|---|---|---|
Product: | Red Hat CloudForms Management Engine | Reporter: | Satoe Imaishi <simaishi> |
Component: | UI - OPS | Assignee: | lgalis |
Status: | CLOSED ERRATA | QA Contact: | Jan Krocil <jkrocil> |
Severity: | medium | Docs Contact: | |
Priority: | high | ||
Version: | 5.7.0 | CC: | dclarizi, ehayes, ewinchel, hkataria, jhardy, jkrocil, lgalis, mpovolny, obarenbo, simaishi |
Target Milestone: | GA | Keywords: | Regression, ZStream |
Target Release: | 5.7.4 | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | 5.7.4.0 | Doc Type: | If docs needed, set a value |
Doc Text: | Story Points: | --- | |
Clone Of: | 1447940 | Environment: | |
Last Closed: | 2017-12-18 20:26:17 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | CFME Core | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | 1447940 | ||
Bug Blocks: |
Comment 2
CFME Bot
2017-07-03 21:16:56 UTC
New commit detected on ManageIQ/manageiq/euwe: https://github.com/ManageIQ/manageiq/commit/7983e694dd0e087eb2566a88d3d36602c0ad9935 commit 7983e694dd0e087eb2566a88d3d36602c0ad9935 Author: Eric Hayes <eric> AuthorDate: Mon Jul 3 16:42:17 2017 -0400 Commit: Eric Hayes <eric> CommitDate: Mon Jul 3 16:42:23 2017 -0400 Compile and write stylesheets for PDF rendering https://bugzilla.redhat.com/show_bug.cgi?id=1465081 app/assets/stylesheets/pdf/_fontawesome.scss.erb | 10 +++++ app/assets/stylesheets/pdf/_patternfly.scss.erb | 10 +++++ app/assets/stylesheets/pdf/_report.scss | 25 +++++++++++ app/assets/stylesheets/pdf/_summary.scss | 48 ++++++++++++++++++++++ app/assets/stylesheets/pdf_report.css | 31 +++----------- app/assets/stylesheets/pdf_summary.css | 46 +++------------------ app/controllers/application_controller.rb | 1 + .../application_controller/report_downloads.rb | 13 ++++++ app/helpers/image_encode_helper.rb | 31 ++++++++++++++ app/helpers/quadicon_helper.rb | 7 ++-- app/views/layouts/_show_pdf.html.haml | 2 +- app/views/shared/summary/_textual.html.haml | 3 +- .../shared/summary/_textual_multilink.html.haml | 6 ++- .../shared/summary/_textual_multivalue.html.haml | 3 +- app/views/shared/summary/_textual_tags.html.haml | 6 ++- lib/asset_writer.rb | 33 +++++++++++++++ lib/pdf_generator.rb | 4 +- lib/pdf_generator/prince_pdf_generator.rb | 19 ++++++--- spec/helpers/quadicon_helper_spec.rb | 2 + .../catalog/_sandt_tree_show.html.haml_spec.rb | 2 + .../listnav/_availability_zone.html.haml_spec.rb | 3 +- .../listnav/_cloud_network.html.haml_spec.rb | 3 +- .../listnav/_cloud_subnet.html.haml_spec.rb | 3 +- .../listnav/_cloud_tenant.html.haml_spec.rb | 3 +- .../layouts/listnav/_ems_cloud.html.haml_spec.rb | 3 +- .../layouts/listnav/_ems_cluster.html.haml_spec.rb | 3 +- .../listnav/_ems_container.html.haml_spec.rb | 3 +- .../layouts/listnav/_ems_network.html.haml_spec.rb | 3 +- .../layouts/listnav/_flavor.html.haml_spec.rb | 3 +- .../layouts/listnav/_floating_ip.html.haml_spec.rb | 3 +- .../listnav/_host_aggregate.html.haml_spec.rb | 3 +- .../listnav/_load_balancer.html.haml_spec.rb | 3 +- .../listnav/_network_port.html.haml_spec.rb | 3 +- .../listnav/_network_router.html.haml_spec.rb | 3 +- .../listnav/_orchestration_stack.html.haml_spec.rb | 3 +- .../listnav/_persistent_volume.html.haml_spec.rb | 3 +- .../listnav/_security_group.html.haml_spec.rb | 3 +- .../views/ems_common/_show.html.haml_spec.rb | 2 + 38 files changed, 256 insertions(+), 99 deletions(-) create mode 100644 app/assets/stylesheets/pdf/_fontawesome.scss.erb create mode 100644 app/assets/stylesheets/pdf/_patternfly.scss.erb create mode 100644 app/assets/stylesheets/pdf/_report.scss create mode 100644 app/assets/stylesheets/pdf/_summary.scss create mode 100644 app/helpers/image_encode_helper.rb create mode 100644 lib/asset_writer.rb New commit detected on ManageIQ/manageiq/euwe: https://github.com/ManageIQ/manageiq/commit/b744bb6ce376e7abe972fa45695497ed60d4fd18 commit b744bb6ce376e7abe972fa45695497ed60d4fd18 Author: Harpreet Kataria <hkataria> AuthorDate: Thu May 11 15:59:39 2017 -0400 Commit: Satoe Imaishi <simaishi> CommitDate: Fri Jul 7 15:24:51 2017 -0400 Merge pull request #1332 from hayesr/fix_pdf_stylesheet_ext Fix missing stylesheet extension (cherry picked from commit 6d21b2eca5efd704e0da9ede5455eb2779972ecc) https://bugzilla.redhat.com/show_bug.cgi?id=1465081 app/controllers/application_controller/report_downloads.rb | 2 +- spec/controllers/ems_common_controller_spec.rb | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) The PDF download button is still present for the provider summary upstream. Will test in 5.7.4. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2017:3484 |