Bug 802504
Summary: | Cannot view directory entries in drift snapshot view | ||||||
---|---|---|---|---|---|---|---|
Product: | [Other] RHQ Project | Reporter: | John Sanda <jsanda> | ||||
Component: | Core UI | Assignee: | Jay Shaughnessy <jshaughn> | ||||
Status: | CLOSED CURRENTRELEASE | QA Contact: | Mike Foley <mfoley> | ||||
Severity: | high | Docs Contact: | |||||
Priority: | unspecified | ||||||
Version: | 4.3 | CC: | hrupp, jkandasa, jshaughn | ||||
Target Milestone: | --- | ||||||
Target Release: | RHQ 4.4.0 | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | |||||||
: | 815757 (view as bug list) | Environment: | |||||
Last Closed: | 2013-08-31 10:14:20 UTC | Type: | --- | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Bug Depends On: | |||||||
Bug Blocks: | 735475 | ||||||
Attachments: |
|
Description
John Sanda
2012-03-12 17:11:53 UTC
Created attachment 569462 [details]
screen shot of the issue
I have just tested and verified that the issue is happening in RHQ 4.3.0. master commit 2977e0a0a40af3757693fa226f5c8d6145d0a688 The smartgwt upgrade had a bad effect on our expansion components. The drift snapshot and drift template views both use expandable rows. There were a couple of contributing issues rooted in the fact that our expansion components extended Table. Table manipulates the canvas and grids in a pretty heavy manner, and the expansion components needed to be able to override a couple of things. Another problem, for whatever reason, was that Table's base component is an HLayout and in the constructor we set the HLayout to Overflow.Hidden. This messed up expansion even though it seemed removed. Anyway, This setting was put in place long ago and I don't think it's actually needed. So, I've commented it out. *** Review Alert *** This is a change to Table that *seems* ok, if you know otherwise please let me know !!! If necessary, it could be added back in a way that it could be overridden by a protected method, or set via constructor. *** Bug 802501 has been marked as a duplicate of this bug. *** Verified on the Build: Version: 4.4.0-SNAPSHOT Build Number: 5ffafd2 GWT Version: 2.4.0 SmartGWT Version: 3.0 OS: RHEL 6.1 X86_64 Browser: Firefox ESR 10.0.3 Listing directory view as expected. Bulk close of old bugs in VERIFIED state. |