<table><tr><td style="">echarruau added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D27583">View Revision</a></tr></table><br /><div><div><p>There are mistakes in the multipledataset option text, see my online comments.</p>

<p>We could split multipledataset 2 2 in two different levels according to the number of items.</p>

<p>We could add an additional level but this includes quite a lots of change.<br />
For this we need to have a different size between the model and the target. Let me explain you why.<br />
In the current mosaic (single line option) children do not need to recall the order of the elements. They simply have to recognise one elements and put the same under. This is not what we are aiming at :)<br />
Solution: have a smaller model that the target.</p>

<p>I recognize that the layout could be better if it would be centered. Nonetheless well done :) These are just details.</p>

<p>To sum up<br />
multipledata1: as it is now but centered<br />
multipledata2: as it is now but with a smaller model than the target (should be centered and we need to add an attribute to the multipledata)<br />
multipledata3: first half of multipledata2<br />
multipledata4: second half of multipledata2</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D27583#inline-158453">View Inline</a><span style="color: #4b4d51; font-weight: bold;">Data.qml:24</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span style="color: #004012">Dataset</span> <span class="p">{</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #aa4000">objective:</span> <span style="color: #004012">qsTr</span><span class="p">(</span><span style="color: #766510">"Rebuid the mosaic when items are not on a single line."</span><span class="p">)</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #aa4000">difficulty:</span> <span style="color: #601200">3</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">"Rebuild" not rebuid. Anyway the we could have a shorter description:<br />
Items are on a single line.<br />
Items on multiple lines.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R2 GCompris</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D27583">https://phabricator.kde.org/D27583</a></div></div><br /><div><strong>To: </strong>dekumar, GCompris: Improvements, jjazeix, timotheegiet, echarruau<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>