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

Issue 48711002: tryton: Set the focus on the first entry of filter window and use present

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 weeks, 2 days ago by ced
Modified:
4 days, 18 hours ago
Reviewers:
reviewbot
Visibility:
Public.

Description

The window can not grab the focus. Instead present should be used which put the focus. Then we put the focus on the first entry. issue8061

Patch Set 1 #

Patch Set 2 : replace show by present #

Patch Set 3 : Use Popover #

Patch Set 4 : Do not take unecessary height #

Patch Set 5 : Update to tip #

Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -69 lines) Patch
M CHANGELOG View 1 2 3 4 1 chunk +1 line, -0 lines 0 comments Download
M tryton/gui/window/view_form/screen/screen.py View 1 2 1 chunk +0 lines, -1 line 0 comments Download
M tryton/gui/window/view_form/view/screen_container.py View 1 2 3 9 chunks +17 lines, -68 lines 0 comments Download

Messages

Total messages: 10
ced
2 weeks, 2 days ago (2019-01-30 22:58:40 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/48711002
2 weeks, 2 days ago (2019-01-30 23:13:20 UTC) #2
ced
replace show by present
2 weeks, 2 days ago (2019-01-31 13:18:01 UTC) #3
reviewbot
flake8 OK URL: https://codereview.tryton.org/48711002
2 weeks, 2 days ago (2019-01-31 13:37:58 UTC) #4
ced
Use Popover
4 days, 21 hours ago (2019-02-11 22:17:22 UTC) #5
reviewbot
patch is not applicable URL: https://codereview.tryton.org/48711002
4 days, 20 hours ago (2019-02-11 22:41:23 UTC) #6
ced
Do not take unecessary height
4 days, 19 hours ago (2019-02-12 00:03:49 UTC) #7
reviewbot
patch is not applicable URL: https://codereview.tryton.org/48711002
4 days, 19 hours ago (2019-02-12 00:06:41 UTC) #8
ced
Update to tip
4 days, 19 hours ago (2019-02-12 00:10:20 UTC) #9
reviewbot
4 days, 18 hours ago (2019-02-12 00:44:32 UTC) #10
Sign in to reply to this message.

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