Tree colors in one2many overruled by CSS

Bug #882175 reported by Stefan Rijnhart (Opener)
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Web (MOVED TO GITHUB)
Fix Released
Low
Unassigned

Bug Description

Hi,

Tree colors in one2many widgets do not work. Although the color is set correctly in the tr's style attribute, it is overruled by CSS: the color of ".openerp table.oe_frame td" is set in addons/web/static/src/css/base.css to a shade of grey. By definition, any CSS takes precedence over HTML style attributes.

I would recommend simply removing the color definition in the CSS.

Cheers,
Stefan.

Related branches

Revision history for this message
Xavier (Open ERP) (xmo-deactivatedaccount) wrote :

> I would recommend simply removing the color definition in the CSS.

This significantly darkens form font, as it falls back on #2c2c2c defined by jquery-ui. Two other options would be to remove some specificity from the rule (chop off the last section) or tell cells *in o2ms specifically* to inherit their color from their parent (their row in this case).

i'll have to discuss that with fme. In the meantime, do you have an example view with o2ms which should be colored and are not?

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Hi Xavier,

Thanks for bringing up more sophisticated solutions :-)

I was working on a custom module, but you can reproduce by creating a journal entry with unbalanced journal items in Accounting -> Journal Entries -> Journal Entries. Unbalanced lines should be blue.

Cheers,
Stefan.

Changed in openerp-web:
status: New → Incomplete
status: Incomplete → Confirmed
importance: Undecided → Low
assignee: nobody → OpenERP R&D Web Team (openerp-dev-web)
status: Confirmed → In Progress
Revision history for this message
Xavier (Open ERP) (xmo-deactivatedaccount) wrote :

Fixed in revision <email address hidden>

Changed in openerp-web:
assignee: OpenERP R&D Web Team (openerp-dev-web) → nobody
status: In Progress → Fix Released
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Thanks for the quick fix!

Stefan.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.