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

Add 'breakable' optional argument to report_build_html_table method #17204

Conversation

hstastna
Copy link

@hstastna hstastna commented Mar 26, 2018

BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1428536
Depends on (PR to UI repo): ManageIQ/manageiq-ui-classic#3681


Add 'breakable' optional argument to report_build_html_table method for fixing bad formatting of VM Chargeback Preview Report table (unset word-break css attribute). The purpose was not to break/edit any other tables if not needed.

Before:
preview_before

After:
preview_after

https://bugzilla.redhat.com/show_bug.cgi?id=1428536

Add 'breakable' optional argument to report_build_html_table method
for fixing bad formatting of VM Chargeback Preview Report table.
@hstastna
Copy link
Author

@miq-bot add_label bug

@miq-bot
Copy link
Member

miq-bot commented Mar 26, 2018

Checked commit hstastna@e9ee14e with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. 👍

@hstastna
Copy link
Author

@skateman @epwinchell @himdel Could you please someone look at it? Thanks! 🌟

Copy link
Member

@skateman skateman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Seal of Approval

@epwinchell
Copy link
Contributor

Looks good

@epwinchell
Copy link
Contributor

@miq-bot add_label formatting/styling

@miq-bot
Copy link
Member

miq-bot commented Mar 27, 2018

@epwinchell Cannot apply the following label because they are not recognized: formatting/styling

@epwinchell
Copy link
Contributor

@miq-bot add_label formatting/styling

@miq-bot
Copy link
Member

miq-bot commented Mar 27, 2018

@epwinchell Cannot apply the following label because they are not recognized: formatting/styling

@martinpovolny martinpovolny merged commit a393ec3 into ManageIQ:master Apr 6, 2018
@martinpovolny martinpovolny self-assigned this Apr 6, 2018
@martinpovolny martinpovolny added this to the Sprint 83 Ending Apr 9, 2018 milestone Apr 6, 2018
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.

5 participants