edesiocs / ingatan

Automatically exported from code.google.com/p/ingatan
Other
0 stars 0 forks source link

Table Questions should have configurable number of columns. #34

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
This is an improvement on an existing feature.

Describe the new feature should be added and what the current situation is.
Currently table questions have only two columns. Some uses require extra 
columns, and for those columns to have names. For example, a word in a language 
could be a verb, noun and adjective (e.g. 'like' can be a verb or adjective - 
'I like it', 'It is like x') and so a column for each would be beneficial. At 
question time, two columns could be chosen and the question be framed as 
"translate 'like' (verb)".

How will this feature improve the usability of Ingatan?
It will make it more flexible, and more versatile.

Do you have any suggestions for code implementation?
The size of the table could be set when the table question is made, with a 
graphical dialogue (i.e. preview of the number of columns in a graphic). Maybe 
have a maximum of 5 columns. This will generate implications in the copy and 
paste of data from the `DataTable` objects, as well as how `TableQuestions` are 
treated by the `QuizManager`.

Original issue reported on code.google.com by tom.eve...@gmail.com on 19 Nov 2010 at 11:25

GoogleCodeExporter commented 9 years ago

Original comment by tom.eve...@gmail.com on 1 Feb 2011 at 8:51

GoogleCodeExporter commented 9 years ago
This was implemented, and while the feature _may_ make Ingatan more useful, it 
makes it less usable.

The generalisation of the TableQuestion type to multiple columns made the use 
of the question type difficult/confusing.

The attachment shows the implementation. No commit will be made, and the 
feature will be abandoned at least for now. The design could be refined for the 
future to implement the feature again, addressing the problems encountered this 
time.

Original comment by tom.eve...@gmail.com on 2 Feb 2011 at 8:05

Attachments: