Vidhin Mehta  (OpenERP) has proposed merging 
lp:~openerp-dev/openerp-web/trunk-bug-914515-vme into lp:openerp-web.

Requested reviews:
  OpenERP R&D Web Team (openerp-dev-web)
Related bugs:
  Bug #914515 in OpenERP Web: "tree view missing indent + structure"
  https://bugs.launchpad.net/openerp-web/+bug/914515

For more details, see:
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-914515-vme/+merge/88355

Fix bug issue,improved classes for tree view in css and integer/float value in 
tree view now shows on right alignment.
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-914515-vme/+merge/88355
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/trunk-bug-914515-vme.
=== modified file 'addons/web/static/src/css/base.css'
--- addons/web/static/src/css/base.css	2012-01-11 16:07:46 +0000
+++ addons/web/static/src/css/base.css	2012-01-12 11:52:26 +0000
@@ -1850,15 +1850,28 @@
 }
 .openerp .oe-treeview-table {
     width: 100%;
+    background-color : #FFFFFF;
+    border-spacing: 0;
+}
+.openerp .oe-treeview-table tr:hover{
+    color: blue;
+    background-color : #D8D8D8;
 }
 .treeview-tr, .treeview-td {
     cursor: pointer;
     vertical-align: top;
     text-align: left;
+    border-bottom: 1px solid #CFCCCC;
+}
+.oe-number{
+   text-align: right !important;
 }
 .treeview-tr span, .treeview-td span {
+    font-size: 90%;
+    font-weight: normal;
+    white-space: nowrap;
     display: block;
-}
+    }
 .treeview-tr:first-of-type {
     background: transparent url(/web/static/src/img/expand.gif) 0 50% no-repeat;
 }
@@ -1871,8 +1884,11 @@
 }
 
 .treeview-header {
+    vertical-align: top;
+    background-color : #D8D8D8;
+    white-space: nowrap;
     text-align: left;
-    vertical-align: top;
+    padding: 4px 5px;
 }
 /* Shortcuts*/
 .oe-shortcut-toggle {

=== modified file 'addons/web/static/src/xml/base.xml'
--- addons/web/static/src/xml/base.xml	2012-01-12 10:02:06 +0000
+++ addons/web/static/src/xml/base.xml	2012-01-12 11:52:26 +0000
@@ -546,22 +546,27 @@
         </tbody>
     </table>
 </t>
-<tr t-name="TreeView.rows"
+<tr t-name="TreeView.rows" 
         t-foreach="records" t-as="record"
         t-att-id="'treerow_' + record.id"
         t-att-data-id="record.id" t-att-data-level="level + 1">
     <t t-set="children" t-value="record[children_field]"/>
     <t t-set="class" t-value="children and children.length ? 'treeview-tr' : 'treeview-td'"/>
-
+    <t t-set="style" t-value="'background-position: ' + 19*level + 'px; padding-left: ' + 19*level + 'px;'"/>
+    
     <td t-foreach="fields_view" t-as="field"
         t-if="!field.attrs.modifiers.tree_invisible"
         t-att-data-id="record.id"
-        t-att-style="color_for(record) + (!field_index ? 'background-position: ' + 19*level + 'px; padding-left: ' + 19*level + 'px;' : '')"
-        t-att-class="class">
-        <span t-if="!field.attrs.modifiers.invisible">
+        t-att-style="color_for(record) + style "
+        t-att-class="(fields[field.attrs.name].type === 'float') or (fields[field.attrs.name].type === 'integer')
+                    ? (class +' ' +'oe-number') : class">
+                    
+        <span t-if="!field.attrs.modifiers.invisible" >
             <t t-esc="render(record[field.attrs.name], fields[field.attrs.name])" />
         </span>
+        
         <t t-set="class" t-value="'treeview-td'"/>
+        <t t-set="style" t-value="''"/>
     </td>
 </tr>
 

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help   : https://help.launchpad.net/ListHelp

Reply via email to