← Back to team overview

yellow team mailing list archive

Re: D3 Components Documentation (issue 6858085)

 

On 2012/11/28 12:33:03, teknico wrote:
> Thanks for these docs, they're much useful.

> I made quite a few small adjustments to the text flow and markup, so
instead of
> wasting your time and mine by making lots of small comments inline, I
just
> created a new branch and made all changes there. I did not reflow
paragraphs
> even when lines got too long, to avoid obscuring the changes in the
diff.

> Note that some additions and deletions of blank lines are needed to
make the
> document compile to HTML correctly.

> I also added the document to the index.rst file, so that it appears in
the
> generated output.

> I'm sorry for the extensive rework: I do believe it significantly
improves
> overall readability, however.

> Please find the changes at lp:~teknico/juju-gui/framework-docs .

Thank you! I merged your changes to this branch and will re-propose if
there is another reviewer.


https://codereview.appspot.com/6858085/

-- 
https://code.launchpad.net/~bcsaller/juju-gui/framework-docs/+merge/136578
Your team Juju GUI Hackers is requested to review the proposed merge of lp:~bcsaller/juju-gui/framework-docs into lp:juju-gui.


References