On 11/09/2013 12:23, b...@openinworld.com wrote: > Some minor nit-picks... > > > "please do so as we are going to iterate over what was done there." > I don't understand what you mean by 'iterate over'? Do you me 'build on'?
Yes, that's what I meant: "improve". If you find it hard to understand, let me know. > > > "let's first settle for a target design. For the rest of this > episode, we will try to achieve the following design. " > I get to the bottom and the last snapshot it doesn't look the same as > the one at the top. At the top you should mention that your target > design is achieved over several posts (if that is the case). No wonder it doesn't match :-) Figure 1 is not a snapshot from the system. It is the UI design done in LibreOffice draw. I'll mention that. > > > "Note lines 9, 10, 17 and 18." > "Note new lines 9, 10, 17 and 18." > > * At the top you should specify what version of Pharo the tutorial > relates to. Information on the web hangs around for a very long time > and someone could get trapped later trying your tutorial on a later > version of Pharo. Correct. That's why at the beginning of this episode the reader is asked to first proceed to episode 1 if he hasn't done so already and in episode one in Prerequisites section (http://www.bahmanm.com/blogs/spec-part-1-basics#section-1) it is mentioned that for this series Pharo 2.0 will be used. > > * Regarding #defaultSpec, I happen to like what is used in PharoLauncher > specLayout > <spec: #default> > ^ SpecLayout composed ...etc I don't understand your point. Could you elaborate a bit more on this? > > Overall it a nice next step introducing just a bit more > functionality. It reads very well. I look forward to the next one. Your comments are much appreciated. It's always very useful to have another couple of eyes, QC a document :-) Thanks. -- Bahman Movaqar (http://BahmanM.com) ERP Evaluation, Implementation & Deployment Consultant PGP Key ID: 0x6AB5BD68 (keyserver2.pgp.com)
signature.asc
Description: OpenPGP digital signature