Bug 1600245 - Bricks view has extra 'X' that is confusing and redundant
Summary: Bricks view has extra 'X' that is confusing and redundant
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Gluster Storage
Classification: Red Hat Storage
Component: web-admin-tendrl-ui
Version: rhgs-3.4
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
: ---
Assignee: Neha Gupta
QA Contact: sds-qe-bugs
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-07-11 20:04 UTC by Ju Lim
Modified: 2019-05-08 20:14 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-05-08 20:14:55 UTC
Embargoed:


Attachments (Terms of Use)
Bricks page (329.92 KB, image/png)
2018-07-11 20:04 UTC, Ju Lim
no flags Details

Description Ju Lim 2018-07-11 20:04:30 UTC
Created attachment 1458217 [details]
Bricks page

Description of problem:
Paul Cuzner: “In the bricks view I find the 'X' on the right hand side confusing. Seems redundant given the row above (replica set) can be clicked to show/hide details.”

Version-Release number of selected component (if applicable):
[root@tendrl-server tendrl-cleanup]# rpm -qa | grep tendrl | sort
tendrl-api-1.6.3-20180704T133554.c5e3ec9.noarch
tendrl-api-httpd-1.6.3-20180704T133554.c5e3ec9.noarch
tendrl-commons-1.6.3-20180704T133946.a2e704b.noarch
tendrl-grafana-plugins-1.6.3-20180705T173252.db411a2.noarch
tendrl-grafana-selinux-1.5.4-20180227T085901.984600c.noarch
tendrl-monitoring-integration-1.6.3-20180705T173252.db411a2.noarch
tendrl-node-agent-1.6.3-20180704T133946.0768af2.noarch
tendrl-notifier-1.6.3-4.el7.noarch
tendrl-selinux-1.5.4-20180227T085901.984600c.noarch
tendrl-ui-1.6.3-20180706T125350.cd9dac7.noarch


How reproducible:
100%

Steps to Reproduce:
1. Install WA
2. Import Cluster
3. Navigate to Bricks page for any volume in the Volumes list.

Actual results:
see screenshot.

Expected results:
No need for the X as redundant

Additional info:

Comment 2 Ju Lim 2018-07-11 20:29:50 UTC
+1 on the “X” being redundant and potentially confusing.  This is part of PatternFly design (http://www.patternfly.org/pattern-library/content-views/list-view/), but I agree it is not needed. 

@mcarrano - Can we remove the "X"?  See screenshot for details.
Note: That user can expand and collapse via the the chevron and the parent row (blue row), and the "X" being the 3rd way of doing so and a little awkward since user expanded it via the chevron or parent row or the "Expand All" option at the top.

Comment 3 Matt Carrano 2018-07-11 20:46:01 UTC
Yes, I agree that the 'X' can be removed. We recently updated PatternFly to call this out as optional as it does conflict with content in some cases.


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