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

Issue 41181002: sao: Change new record shortcut (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 months, 1 week ago by xcodinas
Modified:
6 months ago
Reviewers:
rietveld-bot, ced, reviewbot
Visibility:
Public.

Description

issue7218 COLLABORATOR=nicolas.evrard@b2ck.com

Patch Set 1 #

Patch Set 2 : change shorcut to a key combination w/o shift #

Patch Set 3 : Use shortcuts that don't conflict with native shortcuts #

Patch Set 4 : Add CHANGELOG #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -2 lines) Patch
M CHANGELOG View 1 2 3 1 chunk +1 line, -0 lines 0 comments Download
M src/sao.js View 1 2 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 11
xcodinas
9 months, 1 week ago (2018-03-13 08:44:01 UTC) #1
reviewbot
flake8 OK URL: https://codereview.tryton.org/41181002
9 months, 1 week ago (2018-03-13 08:55:49 UTC) #2
xcodinas
change shorcut to a key combinatioC w/o shift
9 months ago (2018-03-23 13:30:36 UTC) #3
reviewbot
flake8 OK URL: https://codereview.tryton.org/41181002
9 months ago (2018-03-23 13:51:17 UTC) #4
nicoe
Use shortcuts that don't conflict with native shortcuts
6 months, 2 weeks ago (2018-06-04 20:33:01 UTC) #5
reviewbot
flake8 OK URL: https://codereview.tryton.org/41181002
6 months, 2 weeks ago (2018-06-04 21:02:08 UTC) #6
ced
Missing changelog
6 months, 2 weeks ago (2018-06-04 21:43:56 UTC) #7
nicoe
Add CHANGELOG
6 months, 2 weeks ago (2018-06-05 08:53:53 UTC) #8
reviewbot
flake8 OK URL: https://codereview.tryton.org/41181002
6 months, 2 weeks ago (2018-06-05 09:22:29 UTC) #9
ced
LGTM
6 months, 2 weeks ago (2018-06-05 09:31:57 UTC) #10
rietveld-bot_tryton.org
6 months ago (2018-06-18 13:48:14 UTC) #11
New changeset 86b046ccf3be by Nicolas ?vrard in branch 'default':
Use non conflicting keyboard shortcuts
https://hg.tryton.org/sao/rev/86b046ccf3be
Sign in to reply to this message.

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