yann feedback on the input number widget

[Ducasse]

Classified as: {OPEN}

Bonjour,

J’ai fait une relecture rapide du PDF envoyé par Stéphane.
Les pages indiqués sont celles du PDF (par exemple la première page = Page3).

Je voulais remonter les notes directement dans des issues sur GitHub mais je ne parviens pas à en créer sur ton Repo.
Et je ne pense pas qu’une pull request soit le bon outil pour remonter des remarques.
Du coup je te les mets directement dans ce mail :

Page3:

• Mettre un lien en bas de page vers les projet CoypuIDE et Mooflod.
• Si le document a vocation à être imprimé, ne faut-il pas privilégié des images de couleur claire ?
• Si oui, il faudrait changer la figure 1-1, et choisir un widget avec un background blanc par exemple.

Page4:

• "The main widget is composed of four elements: two buttons, a label, and a value. Let us detail these elements:"
• Changer par: "The main widget is composed of four elements: two buttons (the cricle with a "+" and the circle with the "-"), a label (that display "input"), and a value (that display "20"). Let us detail these elements: "
• Ou alors annoter la figure 1-1 avec les différents éléments.
• Dans l'exemple il y a des méthodes pour définir la taille et le background mais il n'y a pas de méthode pour définir la géométrie, le layout et la bordure. J'ai l'impression qu'il y a une incohérence.

Page5:

• La figure 1-2 n'est pas centré par rapport au space et ne fait pas la même taille exactement que les autres figures.
• La fonction label: créer un label pour un widget, cette fonction n'est utiliser qu'à l'initialization du widget. Son nom diffère des autres fonctions d'initialization (initializePlusButton, initializeInputValue:).
• Je propose de créer une fonction initiliazeLabel qui créer le BlTextElement et de garder la fonction label: qui change la valeur de text du BlTextElement.

Page9:

• l'initialization de la valeur du slot #callBackBlock est faite dans la méthode initilaizeInputValue:, je l'aurais plutôt mis dans la méthode initialize de BlIntegerInputElement.
• Pour moi, il est préférable que l'initialization de la "logique" soit séparer de l'initialization "graphique". initilaizeInputValue -> on est dans l'aspect graphique et initialize -> dans l'aspect "logique" + appelle à l'aspect "graphique".

Page10:

• Je modifierai le nom de la méthode callbackBlock en quelque chose de plus expressif:
• Par exemple: whenValueChangedDo:

---

Page13:

• Il manque un point à la fin de la plupart des paragraphes de cette section.
  -Figure 2-1 dupliquer avec 1-1.

Page14:

• Ajouter une référence vers la figure 2-2, voir mettre la figure 2-2 à la page 14.

Page15:

• ", this can be done in both following ways", il manque la suite.
• "Notice that the two following forms are equivalent. This is important if you want to maximize" je n'ai pas compris cette phrase.