Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1014438

Summary: [RFE] Improve layout of system details page
Product: [Retired] Beaker Reporter: Nick Coghlan <ncoghlan>
Component: web UIAssignee: Dan Callaghan <dcallagh>
Status: CLOSED CURRENTRELEASE QA Contact: tools-bugs <tools-bugs>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 0.15CC: abenoit, aigao, asaha, dcallagh, jingwang, jzhao, mjia, qwan, rmancy, tools-bugs
Target Milestone: 19.0Keywords: FutureFeature, Patch
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of:
: 1052043 (view as bug list) Environment:
Last Closed: 2014-11-25 07:17:42 UTC Type: Bug
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: 619335, 692777, 880724, 884399, 980352, 999444, 1009323, 1011284, 1011293, 1012174, 1020107, 1037280, 1062086, 1062706, 1070036, 1072336, 1093226, 1118884, 1122464, 1124381, 1144190    

Description Nick Coghlan 2013-10-02 03:20:26 UTC
The top half of the system details page is a confusing mix of information about the system (updated by clicking the Edit Details page) and various actions that can be taken *with* the system (like loaning, reserving, changing the owner, contacting the owner, and updating the notify CC list).

It would be better to separate these out more clearly.

It would also be nice to do something about the proliferation of tabs on the systems page, but that's beyond the scope of this RFE.

Comment 1 Nick Coghlan 2013-10-02 05:51:18 UTC
A few more specific ideas/questions:

* The read only view shouldn't use faded out text edit boxes to display data.

* Owner, current user, loaned to and notify cc should be in a separate block below the system information

* The NDA flag should be displayed as a checkbox in the read-only as well as the Edit view

* Rather than displaying Lab Controller and Location separately, perhaps they could be displayed more concisely as a single field: beaker-lc.site.example.com (Example location)


* Perhaps the edit view should become a bootstrap modal instead of a separate page?

* Perhaps it would make sense to move some of the existing fields in the top half of the page down into the "Details" tab? Primarily thinking of Vendor, Lender and Serial Number, but perhaps Mac Address as well. Main challenge with this idea is what to do about the editing UI (so perhaps worth skipping for now)

Comment 2 Dan Callaghan 2013-10-02 23:47:51 UTC
(In reply to Nick Coghlan from comment #1)

I like all your suggestions, just some comments on this bit:

> * Perhaps it would make sense to move some of the existing fields in the top
> half of the page down into the "Details" tab? Primarily thinking of Vendor,
> Lender and Serial Number, but perhaps Mac Address as well. Main challenge
> with this idea is what to do about the editing UI (so perhaps worth skipping
> for now)

Vendor and Model are interesting ones because they are the only fields which are both editable by the system owner and also updated by the inventory script. Serial Number and MAC Address could also be updated by the inventory script but it doesn't have the necessary smarts at present. So all of those fields could reasonably be said to belong in the Details tab.

The rest of the hardware info on the Details tab is currently not editable, but only because nobody has written a UI for it (there is a very old RFE for it somewhere).

Lender, on the other hand, is completely unrelated to hardware details and should not be treated as such.

Comment 3 Nick Coghlan 2013-10-03 23:51:19 UTC
*** Bug 1015100 has been marked as a duplicate of this bug. ***

Comment 4 Nick Coghlan 2013-10-03 23:54:00 UTC
Bumping the priority on this to be addressed in 0.15.1 (since it's just UI updates, no db changes)

Comment 5 Dan Callaghan 2014-01-15 03:39:12 UTC
The improved system page UI as discussed here:

http://thread.gmane.org/gmane.comp.systems.beaker.devel/912

is now fully implemented, with one exception: there is no search bar for the system activity grid (I'm still not sure what to do about that, suggestions welcome).

Patches are on Gerrit targetting the new-system-page branch:

http://gerrit.beaker-project.org/#/q/project:beaker+branch:new-system-page,n,z

Comment 6 Nick Coghlan 2014-01-15 04:03:33 UTC
I've been considering how best to communicate this change to users before we land it in a release (that is, I don't think we can wait and just explain it in the release notes of whichever release it lands in - we need to introduce people to the updated workflows well in advance of that happening).

I'm currently thinking of the following approach:

- before we include it in a release, we create an after-the-fact design proposal on beaker-project.org that describes the key changes and the rationale behind each of them. This design proposal should also point out the command line additions made in 0.15 to avoid the need to screen scrape the web UI for operations and information previously not available through the bkr CLI. This would need a one or two screenshots to provide a before-and-after comparison relative to Beaker 0.14, but wouldn't need to provide screenshots for everything.

- when we include it in a release, we also include a task-oriented update to the documentation of the system page that explains the various workflows it supports

- after we include it in a release, we get Fedora to upgrade their test instance, and then use that to create a screencast we can post on beaker-project.org covering the various updated workflows. This could be done as a general introduction to the Beaker system page, so it's hopefully useful for new users as well as to users that will be migrating from the existing system page

Comment 8 Dan Callaghan 2014-02-10 00:31:12 UTC
After-the-fact design proposal is here:

http://beaker-project.org/dev/proposals/system-page-improvements.html

Comment 9 Dan Callaghan 2014-02-26 06:19:55 UTC
A demo Beaker instance based on the new-system-page branch is running here, for people to experiment with the modified UI:

http://209.132.179.159/bkr/

Username is "demo", password is "demo".

Comment 10 Dan Callaghan 2014-09-16 22:58:26 UTC
(In reply to Dan Callaghan from comment #5)
> The improved system page UI as discussed here:
> 
> http://thread.gmane.org/gmane.comp.systems.beaker.devel/912
> 
> is now fully implemented, with one exception: there is no search bar for the
> system activity grid (I'm still not sure what to do about that, suggestions
> welcome).

This last missing piece of the puzzle is now done:

http://gerrit.beaker-project.org/3328
http://gerrit.beaker-project.org/3329
http://gerrit.beaker-project.org/3330
http://gerrit.beaker-project.org/3331

Comment 11 Dan Callaghan 2014-09-18 03:58:14 UTC
*** Bug 1086508 has been marked as a duplicate of this bug. ***

Comment 12 Dan Callaghan 2014-09-18 05:43:04 UTC
The new-system-page branch has been merged back to develop.

Any remaining bugs/improvements relating to the system page will be filed as new bugs against 'develop' version, rather than prolonging this bug.

Comment 15 Dan Callaghan 2014-11-25 07:17:42 UTC
Beaker 19.0 has been released.