Bug 750292

Summary: Events list should have scrollbar and shouldn't be adding columns.
Product: [Retired] oVirt Reporter: Jaroslav Henner <jhenner>
Component: ovirt-engine-webadminAssignee: Vojtech Szocs <vszocs>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: medium Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: ecohen, iheim, Rhev-m-bugs, ykaul
Target Milestone: ---   
Target Release: 3.1   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-08-09 07:57:37 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Jaroslav Henner 2011-10-31 15:27:38 UTC
Description of problem:
 * Events list adds columns each time it is switched from Simple to Advanced mode.

 * There should be some way to display the event message when the text doesn't fit the window horizontal size -- a scroll-bar for example.

Version-Release number of selected component (if applicable):
ic146

How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Vojtech Szocs 2011-11-14 18:32:19 UTC
> Events list adds columns each time it is switched from Simple to Advanced
mode.
This bug was fixed as part of [http://gerrit.ovirt.org/94]: AbstractActionTable.removeColumn() method now ensures that both tables (header and data) have their columns in sync with each other.

Comment 2 Vojtech Szocs 2011-12-13 17:08:28 UTC
> There should be some way to display the event message when the text doesn't
fit the window horizontal size -- a scroll-bar for example.

Currently, table columns have fixed width and cannot be resized. Any table cell content that doesn't fit its space will be clipped (table cell element has CSS overflow: hidden).

We can, however, try to calculate table cell content size and when it's greater than column width, we can associate a tooltip with the content element (so when the user hovers his mouse over clipped content, a tooltip will show the entire content).

Comment 3 Vojtech Szocs 2011-12-13 18:27:23 UTC
Custom text cell with "mouseover" event handler can do this, the only tricky part is overflow detection with CSS overflow:hidden on the wrapper <td> element (comparing element.clientWidth with element.scrollWidth won't work in this case, since overflow:hidden disables scroll bars).

Text cells whose content doesn't fit can have it displayed using native browser tooltip (element title attribute), or using some custom GWT popup panel.

Comment 4 Vojtech Szocs 2011-12-14 15:25:06 UTC
Patch for Message column's clipped content: http://gerrit.ovirt.org/593

Proposed resolution: when the mouse cursor hovers over the table cell, the cell calculates whether its content fits its area, or if it overflows (is clipped). In case the cell content overflows, which means the user sees only a portion of the entire text, a tooltip will be shown to the user when moving mouse cursor over the table cell.

Comment 5 Itamar Heim 2012-08-09 07:57:37 UTC
closing ON_QA bugs as oVirt 3.1 was released:
http://www.ovirt.org/get-ovirt/