Review: Needs Information

Code-wise I guess there's no better way to get the "current record" since we 
don't really have a good API for that, but I see (at least) two issues with 
this, at least when testing on sale orders:

1. A new, phantom "Invoice" node appears after merging the branch, that phantom 
"invoice" node has a gray title background (rather than the usual red)

2. The "In Progress" sale order state does not highlight any node in the 
process, that may be normal

Also, more minor but I think it's kind of hard to notice which state is 
selected. The red highlight on the left is easy to miss. Not sure that's easy 
to change though.
-- 
https://code.launchpad.net/~openerp-dev/openerp-web/trunk-bug-919175-case1-vda/+merge/90677
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openerp-web/trunk-bug-919175-case1-vda.

_______________________________________________
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