-
Notifications
You must be signed in to change notification settings - Fork 358
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
Improve VM Chargeback Preview Report table formatting #3681
Improve VM Chargeback Preview Report table formatting #3681
Conversation
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1428536 Add non-breakable class for VM Chargeback Preview Report table for better formatting.
Improve VM Chargeback Preview Report table formatting by adding` false` value as an optional argument of `report_build_html_table` to unset `word-break` css attribute.
@miq-bot add_label bug |
@miq-bot add_label pending core |
Checked commits hstastna/manageiq-ui-classic@0ac6a34~...2c4bd64 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 |
@skateman @epwinchell @himdel Could you please someone look at it? Thanks! 🌟 |
@hstastna do we really need 2 separate PRs for this? |
@skateman the other PR is to main MIQ repo, this one is to ui classic repo. If you have any idea how to have just one PR to fix it, let me know :) |
@hstastna oh, did not notice it 😮 I think these things should be moved into ui-classic. |
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.
Looks good |
@miq-bot remove_label pending core |
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1428536
PR for main MIQ repo: ManageIQ/manageiq#17204
Improve VM Chargeback Preview Report table formatting by:
non-breakable
html table class for VM Chargeback Preview Report tablefalse
value as an (optional) argument ofreport_build_html_table
to achieve unsettingword-break
css attributeBefore:
After: