Code review comment for lp:~nskaggs/autopilot/apdocs-restruct

Revision history for this message
Christopher Lee (veebers) wrote :

I get a warning when building the docs:
..snip..docs/tutorial/advanced_autopilot.rst:427: WARNING: Inline literal start-string without end-string.

The order that the resulting menu has me pondering what the order should be.

Personally I read the column: 1st column top-to-bottom then the 2nd column top-to-bottom.

This means I see the API reference and appendice before I see mention of the tutorial.
(This way of reading it seems correct with the original layout).

If I'm an oddball though, and the only one that doesn't read it that way then this is ok. I suspect that I'm not though (i.e. reading columns, newspapers for instance, is really top to bottom then next column.) (I might still be an oddball, but not when it comes to reading columns).

review: Needs Fixing

« Back to merge proposal