Bug 750292 - Events list should have scrollbar and shouldn't be adding columns.
Summary: Events list should have scrollbar and shouldn't be adding columns.
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: oVirt
Classification: Retired
Component: ovirt-engine-webadmin
Version: unspecified
Hardware: x86_64
OS: Linux
unspecified
medium
Target Milestone: ---
: 3.1
Assignee: Vojtech Szocs
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-10-31 15:27 UTC by Jaroslav Henner
Modified: 2012-08-09 07:57 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-09 07:57:37 UTC
oVirt Team: ---


Attachments (Terms of Use)

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/


Note You need to log in before you can comment on or make changes to this bug.