> 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)
> 

Previously was a bug,node overriding. There are two nodes of Invoice in sale 
orders process.

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

I think graph_get must return True state for active node.
> 
> 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