Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(2720)

Issue 181104: Button State depending of an empty Many2Many field (tryton issue 1353) (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
14 years, 4 months ago by udono
Modified:
10 years, 9 months ago
Reviewers:
ced, yangoon1
Visibility:
Public.

Description

S.a. https://bugs.tryton.org/roundup/issue1353

Patch Set 1 #

Total comments: 3

Patch Set 2 : revert e57439781124 line 42-43 and fix focus #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -1 line) Patch
M tryton/gui/window/view_form/view/form_gtk/many2many.py View 1 2 chunks +3 lines, -1 line 2 comments Download

Messages

Total messages: 6
yangoon1
14 years, 4 months ago (2009-12-30 16:33:23 UTC) #1
ced
http://codereview.appspot.com/181104/diff/1/2 File tryton/gui/window/view_form/view/form_gtk/many2many.py (right): http://codereview.appspot.com/181104/diff/1/2#newcode178 tryton/gui/window/view_form/view/form_gtk/many2many.py:178: self.screen.clear() It is here that we lost focus
14 years, 4 months ago (2009-12-30 16:57:31 UTC) #2
udono
http://codereview.appspot.com/181104/diff/1/2 File tryton/gui/window/view_form/view/form_gtk/many2many.py (right): http://codereview.appspot.com/181104/diff/1/2#newcode178 tryton/gui/window/view_form/view/form_gtk/many2many.py:178: self.screen.clear() On 2009/12/30 16:57:31, ced wrote: > It is ...
14 years, 4 months ago (2009-12-30 18:14:20 UTC) #3
udono
http://codereview.appspot.com/181104/diff/1/2 File tryton/gui/window/view_form/view/form_gtk/many2many.py (right): http://codereview.appspot.com/181104/diff/1/2#newcode178 tryton/gui/window/view_form/view/form_gtk/many2many.py:178: self.screen.clear() Ups, sorry, I didn't recognize the skipped lines... ...
14 years, 4 months ago (2009-12-30 18:17:38 UTC) #4
udono
new patch http://codereview.appspot.com/181104/diff/1004/1005 File tryton/gui/window/view_form/view/form_gtk/many2many.py (right): http://codereview.appspot.com/181104/diff/1004/1005#newcode128 tryton/gui/window/view_form/view/form_gtk/many2many.py:128: Here I reverted e57439781124 http://codereview.appspot.com/181104/diff/1004/1005#newcode178 tryton/gui/window/view_form/view/form_gtk/many2many.py:178: self.screen.load(ids, ...
14 years, 4 months ago (2009-12-30 19:08:48 UTC) #5
ced
14 years, 3 months ago (2010-01-08 16:34:08 UTC) #6
Can be closed
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b