Bug 790817

Summary: Add tooltips for the scoreboards on Pools Index and Pools Details pages
Product: [Retired] CloudForms Cloud Engine Reporter: Tomas Sedovic <tsedovic>
Component: aeolus-conductorAssignee: Shveta <sshveta>
Status: CLOSED ERRATA QA Contact: wes hayutin <whayutin>
Severity: medium Docs Contact:
Priority: medium    
Version: 1.0.0CC: akarol, athomas, dajohnso, deltacloud-maint, dmacpher, hbrock, jtomasek, juwu, matt.wagner, morazi, psharma, slinaber, ssachdev, tzumainn
Target Milestone: rcKeywords: Triaged
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: aeolus-conductor-0.13.20-1.el6cf Doc Type: Bug Fix
Doc Text:
The icons and numbers in the scoreboard of Cloud Engine were ambiguous to users. This bug fix adds tooltips in the scoreboards on Pools Index and Pools Details pages. Users find out the intended meaning by hovering their cursor over the icons and numbers.
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-12-04 14:56:35 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
Icon explanations
none
title-text patch for /app/views/pools/_scoreboard_index.html.haml
none
Tool tip none

Description Tomas Sedovic 2012-02-15 14:00:18 UTC
The icons and numbers in the scoreboard are a bit confusing.
 
 Adding tooltips with a more detailed explanation would make things easier to grasp.

Comment 1 Angus Thomas 2012-02-15 14:03:23 UTC
Ken,

Could you please provide a list of the intended meanings of each of the icons, as they currently appear in the pools index and pools details page?

Comment 2 Ken Keiter 2012-02-21 16:28:06 UTC
Certainly. I'll do it on a mockup immediately. Didn't see this ticket come in. Please hold..

Comment 3 Ken Keiter 2012-02-21 16:48:32 UTC
Created attachment 564750 [details]
Icon explanations

Hopefully the icon_explanations.png file which I've attached should be helpful. It's from a mock of Conductor (before some tweaks) but the necessary headers should be there. I'm unsure about the wording, though.

Angus, do you have any opinions on the wording of the tooltips?

Comment 4 Hugh Brock 2012-02-22 18:57:43 UTC
We've missed the string freeze so we will have no way to translate the tooltips. Pushing to 1.1.0.

Comment 5 Ken Keiter 2012-02-23 16:30:52 UTC
Do you want this moved to v1.0beta1?

Comment 6 Andy Fitzsimon 2012-08-30 05:27:46 UTC
Created attachment 608046 [details]
title-text patch for /app/views/pools/_scoreboard_index.html.haml

Could sombody check this and verify it changes the scorecard in the monitors view ?

Comment 7 Jiri Tomasek 2012-09-04 14:51:32 UTC
Hi, the scoreboard partial is html.mustache template. It looks like this: https://github.com/aeolusproject/conductor/blob/master/src/app/views/pools/_scoreboard_index.html.mustache

so to add titles, use plain html. Only thing is that the titles have to be translated so create the titles like this:

<dd class="count pool partial" title="<%= t('pools.scoreboard.total_pools')%>">{{pools_in_use}}</dd>

and in src/config/locales/en.yml add

pools:
  scoreboard:
    total_pools: "Your Pools Total"

Comment 8 Matt Wagner 2012-09-04 21:37:15 UTC
See https://github.com/aeolusproject/conductor/pull/38

Comment 9 Jiri Tomasek 2012-09-05 10:42:23 UTC
Pull request for adding tooltips for pool show page scoreboard
https://github.com/aeolusproject/conductor/pull/40

Comment 10 Jiri Tomasek 2012-09-05 14:05:39 UTC
fixes pushed to master(c91c995, a97afb1) and 1.1(4728468, 376173e)

Comment 11 Steve Linabery 2012-09-07 21:35:48 UTC
in build aeolus-conductor-0.13.3-1.el6cf

Comment 13 pushpesh sharma 2012-09-18 11:02:55 UTC
There is no tooltips active on any control.I am anyways opening a new Bug(https://bugzilla.redhat.com/show_bug.cgi?id=858223)for that till then this can also be moved to modified as there is no solution(or solution can not be verified) for this bug in the current build.

[root@dhcp201-113 ~]# rpm -qa|grep aeolus
aeolus-conductor-doc-0.13.7-1.el6cf.noarch
aeolus-all-0.13.7-1.el6cf.noarch
rubygem-aeolus-cli-0.7.1-1.el6cf.noarch
aeolus-configure-2.8.6-1.el6cf.noarch
rubygem-aeolus-image-0.3.0-12.el6.noarch
aeolus-conductor-0.13.7-1.el6cf.noarch
aeolus-conductor-daemons-0.13.7-1.el6cf.noarch
[root@dhcp201-113 ~]#

Comment 14 Mike Orazi 2012-09-18 20:25:45 UTC
We removed the tooltips because the strings weren't frozen in time for translations to happen.  Will move to 2.0 and fix at that point.

Comment 15 Tzu-Mainn Chen 2012-10-15 20:08:18 UTC
Back onto 1.1:

commit 403face15d33d248fe3c2f8826ff5feb09c1a128
Author: Tzu-Mainn Chen <tzumainn>
Date:   Mon Oct 15 11:03:12 2012 -0400

    BZ790817 - reimplement jtomasek's tooltips for pool scoreboards now that translations for the tooltips exist
    
    Signed-off-by: Matt Wagner <matt.wagner>

Comment 16 Shveta 2012-10-17 22:20:00 UTC
Created attachment 629111 [details]
Tool tip

Tool tip added for scoreboard icons.
 rpm -qa|grep aeolus
rubygem-aeolus-image-0.3.0-12.el6.noarch
aeolus-conductor-daemons-0.13.20-1.el6cf.noarch
rubygem-aeolus-cli-0.7.5-1.el6cf.noarch
aeolus-conductor-doc-0.13.20-1.el6cf.noarch
aeolus-configure-2.8.9-1.el6cf.noarch
aeolus-all-0.13.20-1.el6cf.noarch
aeolus-conductor-0.13.20-1.el6cf.noarch

Comment 19 errata-xmlrpc 2012-12-04 14:56:35 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHEA-2012-1516.html