Tryton Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(152)

Issue 35611002: sao: Add position and count label on one2many widget and tree view

Can't Edit
Can't Publish+Mail
Start Review
Created:
5 months, 3 weeks ago by perilla
Modified:
1 week, 5 days ago
Reviewers:
pokoli, ced, reviewbot
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : send position and count from screen #

Patch Set 3 : add label on tree views too #

Total comments: 3

Patch Set 4 : this.group.length -> this.search_count #

Total comments: 3

Patch Set 5 : instead of send label from attributtes at creation, use set_label_count on same place that gtk clie… #

Patch Set 6 : implement update label function out of screen #

Patch Set 7 : using record_message function #

Patch Set 8 : dont check save button, because activate_save is not implemented yet #

Patch Set 9 : use callback function to send data, update to tip #

Patch Set 10 : tab label like in tryton #

Patch Set 11 : label count on tab using badge #

Patch Set 12 : better position of count label #

Patch Set 13 : update to tip #

Total comments: 20

Patch Set 14 : apply ced comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+69 lines, -10 lines) Patch
M CHANGELOG View 1 2 3 4 5 6 7 8 9 10 11 12 1 chunk +1 line, -0 lines 0 comments Download
M src/screen.js View 1 2 3 4 5 6 7 8 9 10 11 12 13 2 chunks +12 lines, -1 line 0 comments Download
M src/tab.js View 1 2 3 4 5 6 7 8 9 10 11 12 13 3 chunks +30 lines, -1 line 0 comments Download
M src/view/form.js View 1 2 3 4 5 6 7 8 9 10 11 12 13 7 chunks +26 lines, -8 lines 0 comments Download

Messages

Total messages: 33
perilla
5 months, 3 weeks ago (2017-09-02 13:07:49 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/35611002
5 months, 3 weeks ago (2017-09-02 13:18:52 UTC) #2
perilla
send position and count from screen
5 months, 3 weeks ago (2017-09-05 16:13:32 UTC) #3
reviewbot
flake8 OK URL: https://codereview.tryton.org/35611002
5 months, 3 weeks ago (2017-09-05 16:21:45 UTC) #4
perilla
add label on tree views too
5 months, 3 weeks ago (2017-09-05 17:59:19 UTC) #5
reviewbot
flake8 OK URL: https://codereview.tryton.org/35611002
5 months, 3 weeks ago (2017-09-05 18:21:43 UTC) #6
ced
https://tryton-rietveld.appspot.com/35611002/diff/40001/src/screen.js File src/screen.js (right): https://tryton-rietveld.appspot.com/35611002/diff/40001/src/screen.js#newcode631 src/screen.js:631: this.label_count = attributes.label_count || null; For me, it is ...
5 months, 3 weeks ago (2017-09-05 18:35:01 UTC) #7
perilla
this.group.length -> this.search_count
5 months, 3 weeks ago (2017-09-05 18:48:17 UTC) #8
reviewbot
flake8 OK URL: https://codereview.tryton.org/35611002
5 months, 3 weeks ago (2017-09-05 18:52:22 UTC) #9
pokoli
https://tryton-rietveld.appspot.com/35611002/diff/60001/src/screen.js File src/screen.js (right): https://tryton-rietveld.appspot.com/35611002/diff/60001/src/screen.js#newcode943 src/screen.js:943: if (this.label_count !== null){ You should probably have a ...
5 months, 3 weeks ago (2017-09-06 08:19:19 UTC) #10
ced
https://tryton-rietveld.appspot.com/35611002/diff/60001/src/screen.js File src/screen.js (right): https://tryton-rietveld.appspot.com/35611002/diff/60001/src/screen.js#newcode943 src/screen.js:943: if (this.label_count !== null){ On 2017/09/06 08:19:18, pokoli wrote: ...
5 months, 3 weeks ago (2017-09-06 09:48:28 UTC) #11
perilla
instead of send label from attributtes at creation, use set_label_count on same place that gtk ...
5 months, 3 weeks ago (2017-09-06 16:30:16 UTC) #12
reviewbot
flake8 OK URL: https://codereview.tryton.org/35611002
5 months, 3 weeks ago (2017-09-06 16:54:51 UTC) #13
perilla
implement update label function out of screen
5 months, 2 weeks ago (2017-09-11 20:07:44 UTC) #14
reviewbot
flake8 OK URL: https://codereview.tryton.org/35611002
5 months, 2 weeks ago (2017-09-11 20:21:40 UTC) #15
perilla
using record_message function
5 months, 2 weeks ago (2017-09-12 16:33:32 UTC) #16
reviewbot
flake8 OK URL: https://codereview.tryton.org/35611002
5 months, 2 weeks ago (2017-09-12 16:42:24 UTC) #17
perilla
dont check save button, because activate_save is not implemented yet
5 months, 2 weeks ago (2017-09-13 15:26:46 UTC) #18
reviewbot
flake8 OK URL: https://codereview.tryton.org/35611002
5 months, 2 weeks ago (2017-09-13 15:46:17 UTC) #19
perilla
use callback function to send data, update to tip
5 months ago (2017-09-27 19:16:06 UTC) #20
reviewbot
flake8 OK URL: https://codereview.tryton.org/35611002
5 months ago (2017-09-27 19:39:15 UTC) #21
perilla
tab label like in tryton
4 months, 2 weeks ago (2017-10-14 11:49:51 UTC) #22
reviewbot
flake8 OK URL: https://codereview.tryton.org/35611002
4 months, 2 weeks ago (2017-10-14 12:07:14 UTC) #23
perilla
label count on tab using badge
4 months, 2 weeks ago (2017-10-14 12:26:12 UTC) #24
reviewbot
flake8 OK URL: https://codereview.tryton.org/35611002
4 months, 2 weeks ago (2017-10-14 12:48:19 UTC) #25
perilla
better position of count label
4 months, 1 week ago (2017-10-18 20:11:10 UTC) #26
reviewbot
flake8 OK URL: https://codereview.tryton.org/35611002
4 months, 1 week ago (2017-10-18 20:16:12 UTC) #27
perilla
2 weeks, 1 day ago (2018-02-10 04:57:36 UTC) #28
reviewbot
flake8 OK URL: https://codereview.tryton.org/35611002
2 weeks, 1 day ago (2018-02-10 05:23:20 UTC) #29
ced
https://codereview.tryton.org/35611002/diff/240001/src/screen.js File src/screen.js (right): https://codereview.tryton.org/35611002/diff/240001/src/screen.js#newcode946 src/screen.js:946: if(record){ space after 'if' https://codereview.tryton.org/35611002/diff/240001/src/tab.js File src/tab.js (right): https://codereview.tryton.org/35611002/diff/240001/src/tab.js#newcode434 ...
1 week, 5 days ago (2018-02-12 23:41:59 UTC) #30
perilla
1 week, 5 days ago (2018-02-13 13:08:47 UTC) #31
perilla
https://codereview.tryton.org/35611002/diff/240001/src/screen.js File src/screen.js (right): https://codereview.tryton.org/35611002/diff/240001/src/screen.js#newcode946 src/screen.js:946: if(record){ On 2018/02/12 23:41:58, ced wrote: > space after ...
1 week, 5 days ago (2018-02-13 13:15:39 UTC) #32
reviewbot
1 week, 5 days ago (2018-02-13 13:15:45 UTC) #33
Sign in to reply to this message.

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