Bug 770716 - Deployments Properties page : Font and spacing
Summary: Deployments Properties page : Font and spacing
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: aeolus-all
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: rc
Assignee: Shveta
QA Contact: wes hayutin
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-12-28 15:26 UTC by Shveta
Modified: 2012-08-30 17:14 UTC (History)
5 users (show)

Fixed In Version: v0.8.0-35
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-30 17:14:03 UTC


Attachments (Terms of Use)
err (213.48 KB, image/png)
2011-12-28 15:26 UTC, Shveta
no flags Details
fixed table (32.58 KB, image/png)
2012-02-16 07:08 UTC, Andy Fitzsimon
no flags Details
font_spacing (213.37 KB, image/png)
2012-02-23 07:10 UTC, Shveta
no flags Details

Description Shveta 2011-12-28 15:26:42 UTC
Created attachment 549837 [details]
err

Description of problem:


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1. Deployment properties page 
2.  Property Name 	Value
       Pool     	Default
       Created 	        2011-12-28 15:01:51 UTC
       Owner 	        aeolus user
       Name 	        ec2_1
       Global Uptime 	N/A
       Uptime 1st instance running N/A
3.
  
Actual results:


Expected results: Property Name and Value are heading .
They should be in bold or different font or in bold.

Also proper spacing is required in "Uptime 1st instance running N/A".
It is not alligned as other properties are.


Additional info:

Comment 1 wes hayutin 2012-01-10 17:11:22 UTC
adding to ce-sprint-next

Comment 2 wes hayutin 2012-01-10 17:14:43 UTC
adding to ce-sprint-next

Comment 3 wes hayutin 2012-01-12 16:33:23 UTC
adding to ce-sprint

Comment 4 wes hayutin 2012-01-12 16:39:55 UTC
removing ce-sprint-next tracker

Comment 5 Andy Fitzsimon 2012-02-14 02:41:42 UTC
it seems my comment is missing.

here are proposed fixes depending on the implementation.
http://jsfiddle.net/andyfitz/6bJfn/

I really need access to this view to give you a real fix however.  is there an installed instance you have somewhere that i can get to this exact view?  i'm unable to reproduce it on my install.

Comment 6 Shveta 2012-02-14 08:38:00 UTC
1) Launch an instance 
2) Pretty view -- Applications
3) Select that application -- > Properties

https://qe-blade-08.idm.lab.bos.redhat.com/conductor/deployments/1?view=filter

Comment 7 Andy Fitzsimon 2012-02-16 03:56:50 UTC
Hey I cant get to that instance.  looks like your machine isn't up.


tried visiting : https://qe-blade-08.idm.lab.bos.redhat.com/conductor/ also

Comment 9 Andy Fitzsimon 2012-02-16 07:07:15 UTC
the folllowing scss fixes this

-----------------------


.properties_table {
  margin-top: 2em;
  th {
    border-bottom: .1em solid #ccc;
    padding-bottom: .5em;
    color: #aaa;
    &:first-child {
      text-align: right;
      padding-right: 2em;
      width: 18.15em; }
    &:last-child {
      text-align: left; } }
  td {
    &:first-child {
      font-weight: bold;
      text-align: right;
      padding-right: 2em;
      &:after {
        content: " :"; } }
    padding-top: .5em; } }

Comment 10 Andy Fitzsimon 2012-02-16 07:08:07 UTC
Created attachment 562403 [details]
fixed table

here's a rendering of the table with appropriate headings, labels and alignment

Comment 11 wes hayutin 2012-02-22 18:44:35 UTC
not fixed in 

[root@qeblade31 ~]# rpm -qa | grep aeolus
rubygem-aeolus-image-0.3.0-8.el6.noarch
aeolus-configure-2.5.0-14.el6.noarch
aeolus-conductor-doc-0.8.0-33.el6.noarch
aeolus-conductor-daemons-0.8.0-33.el6.noarch
aeolus-conductor-0.8.0-33.el6.noarch
rubygem-aeolus-cli-0.3.0-9.el6.noarch
aeolus-all-0.8.0-33.el6.noarch

Comment 12 Tzu-Mainn Chen 2012-02-22 20:26:26 UTC
Tested, committed, and pushed the above scss change to master:

commit dff48de80a675170182a21434ad2a8d19d01ec97
BZ 770716 added table.properties_table for Andy Fitzsimon

Comment 13 Shveta 2012-02-23 07:10:47 UTC
Created attachment 565201 [details]
font_spacing

Looking good now...
rpm -qa|grep aeolus
aeolus-conductor-0.8.0-35.el6.noarch
rubygem-aeolus-cli-0.3.0-10.el6.noarch
aeolus-conductor-doc-0.8.0-35.el6.noarch
rubygem-aeolus-image-0.3.0-9.el6.noarch
aeolus-all-0.8.0-35.el6.noarch
aeolus-conductor-daemons-0.8.0-35.el6.noarch
aeolus-configure-2.5.0-15.el6.noarch


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