fix limit and request labels for containers

[zeari]

Before:

After:

@martinpovolny @himdel Please review
cc @simon3z

@miq-bot add_label compute/containers, bug

[miq-bot]

Checked commit zeari@b256d79 with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
1 file checked, 1 offense detected

app/helpers/container_helper/textual_summary.rb

• ❗ - Line 151, Col 77 - Style/MultilineMethodCallBraceLayout - Closing method call brace must be on the line after the last argument when opening brace is on a separate line from the first argument.

[simon3z]

@zeari I have the feeling that generally numbers should be right-aligned (vs text left-aligned). Should we fix that? (check for consistency with the rest of UI).

[zeari]

@zeari I have the feeling that generally numbers should be right-aligned (vs text left-aligned). Should we fix that? (check for consistency with the rest of UI).

check the screenshots, its pretty consistent with everything pre-existing there.

@himdel Can we make the resource column not as wide? have the values closer to the resource column?

[himdel]

@zeari Not really, no. That's why I was proposing a new kind of textual summary type instead of using tables.

You can try adding some CSS, but.. a html table adapts to its content, so unless you know the expected size and can afford to specifically limit the size of each td to a specific value..