<table><tr><td style="">dekumar 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><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D27583#626331" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D27583#626331</a>, <a href="https://phabricator.kde.org/p/echarruau/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@echarruau</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><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>
</blockquote>

<p><a href="https://phabricator.kde.org/p/echarruau/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@echarruau</a> Regarding the layout rework, I had a discussion with <a href="https://phabricator.kde.org/p/timotheegiet/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@timotheegiet</a> earlier as he going to do the rework when the multiple datasets are merged. As mentioned about the splitting the multiple datasets I would make that changes and update the diff.<br />
You can find the task related with the changes to layout of the activity here and add comments here regarding the layout so it would be good to refer later <a href="https://phabricator.kde.org/T12809" class="remarkup-link" target="_blank" rel="noreferrer">https://phabricator.kde.org/T12809</a><br />
Thanks :)</p></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>