Created attachment 1066792 [details]
Description of problem:
UI not show all vm "Run on" hosts under vm general tab
Version-Release number of selected component (if applicable):
Steps to Reproduce:
1. Create vm and pin it two hosts
2. Check vm->General Tab->Run on field
Show only one host
Must show two hosts
See screenshot for more details
Einav we may need some advice here on how to present the information without
overloading the text field.
Created attachment 1067353 [details]
VMs General Sub Tab - Run On - Improvement Suggestion
We can do one of two things:
(1) simply have a comma-separated list as the value and have it cropped if it is too long, and wait for bug 1205309 to be resolved in order to support displaying cropped values in a tool-tip upon hover.
(2) do something similar to the 'Attached To' column in the Disks main tab [see http://i.imgur.com/Yd0iXQj.png] - display the Host Name if there is a single Host, and display "x Hosts" and the Host-Names in a tool-tip upon hover if there is more than one Host (see attachment 1067353 [details]).
[this may still depend on bug 1205309, since I am not sure if ATM we have no tool-tips support in the General sub-tabs at all, or if we don't have tool-tips support in General sub-tabs only for cropped values - CC'ing Greg that may have additional insights]
You can make tooltips on anything now. They're free in grids, and otherwise as simple as new WidgetTooltip("Hello there!")
I implementet solutio(In reply to Einav Cohen from comment #3)
> We can do one of two things:
> (1) simply have a comma-separated list as the value and have it cropped if
> it is too long, and wait for bug 1205309 to be resolved in order to support
> displaying cropped values in a tool-tip upon hover.
Implemented this solution with the comma separated list and a tool-tip. It was as easy as setting the title of of the TextBoxLabel
The bug also affected the general tab of the pools in the admin portal and the VM and pool tab in the user portal.
3.6.1 has not yet been released, resetting version.
Please fix it to the version where this bug was discovered.
Putting the bug back to assigned. The tooltips need to be implemented with PatternFly -- we are no longer using setTitle() for tooltips.
this is an automated message. oVirt 3.6.0 RC3 has been released and GA is targeted to next week, Nov 4th 2015.
Please review this bug and if not a blocker, please postpone to a later release.
All bugs not postponed on GA release will be automatically re-targeted to
- 3.6.1 if severity >= high
- 4.0 if severity < high
I think this can go to 4.0.0 especially since the described problem is fixed. The tooltip is just displayed in a deprecated way.
@Grep assigning you the bug since you wanted to do some extra changes to the tooltip internals.
Roman fixed this already (comment 12).
The bug was only open until we got the long-value wrapping done, and that is also now fixed (see attached screenshot).
Created attachment 1137140 [details]
run on wraps