Quote from: monkey_05_06 on Tue 18/06/2013 02:26:37
First things first, thanks for having readable code. It means a lot to me. I mean, you inject random blank lines in the middle of functions for no reason whatsoever, but I mean, at least I can read it.
Readable code is very important to me too

Quote from: monkey_05_06 on Tue 18/06/2013 02:26:37
I wanted to note though, there are some opportunities for generalization that I think would help improve this module. As an example, I don't understand why you opted to use a macro for label coloring versus a structure property. The latter would be of relatively equal simplicity to implement, but would give the end-user a customization opportunity that is presently missing. If you're worried about encapsulation, you could always use attributes instead of raw members.
You're probably right about the label coloring. I just figured label coloring would be a constant, so that's why I used a macro but I guess I can see that people might want it to be dynamic.
As for encapsulation, I literally only picked up this whole programming thing about a month or two ago so I'm not really sure what that means. I took about a week's worth of Java in high school seven years ago but dropped it after a week so I'm still quite new to the whole thing. Everything's a learning experience though!
I'm still getting used to the whole nomenclature and vernacular of everything.
Edit:
Things I've realized are missing from this module:
1. Text wrapping - needs to be added in a future release.
2. Option to sort the options bottom to top.
4. Option to change whether one remaining option is automatically spoken or not.
5. Auto numbering of options.
6. Option to set whether options that have been selected dim/change color.
None of which should be too hard to implement.