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

Issue 13415045: Expanded Databinding docs

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 6 months ago by bcsaller
Modified:
10 years, 6 months ago
Reviewers:
rharding, jeff.pihach, mp+183702
Visibility:
Public.

Description

Expanded Databinding docs Fork off and extend databinding docs from the viewlet stuff. https://code.launchpad.net/~bcsaller/juju-gui/databinding-docs/+merge/183702 (do not edit description out of merge proposal)

Patch Set 1 #

Total comments: 20

Patch Set 2 : Expanded Databinding docs #

Unified diffs Side-by-side diffs Delta from patch set Stats (+251 lines, -15 lines) Patch
A [revision details] View 1 1 chunk +2 lines, -0 lines 0 comments Download
A docs/databinding.rst View 1 1 chunk +236 lines, -0 lines 0 comments Download
M docs/index.rst View 1 chunk +1 line, -0 lines 0 comments Download
M docs/viewlets.rst View 4 chunks +12 lines, -15 lines 0 comments Download

Messages

Total messages: 4
rharding
LGTM with some update to the 'viewlets' section to point to the existing doc.
10 years, 6 months ago (2013-09-03 16:00:05 UTC) #1
jeff.pihach
LGTM with the requested changes below. Thanks for putting this large doc together! https://codereview.appspot.com/13415045/diff/1/docs/databinding.rst File ...
10 years, 6 months ago (2013-09-03 21:05:32 UTC) #2
benjamin.saller
Thanks for the reviews, made a number of additions based on feedback and then submitting. ...
10 years, 6 months ago (2013-09-03 23:45:57 UTC) #3
bcsaller
10 years, 6 months ago (2013-09-03 23:51:29 UTC) #4
*** Submitted:

Expanded Databinding docs

Fork off and extend databinding docs from the viewlet stuff.

R=rharding, jeff.pihach, benjamin.saller
CC=
https://codereview.appspot.com/13415045
Sign in to reply to this message.

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