<div dir="ltr">We could keep the same button size but just resize the search icon and putting it a bit on the top right?<br></div><div class="gmail_extra"><br><div class="gmail_quote">2016-07-19 21:05 GMT+02:00 JAZEIX Johnny <span dir="ltr"><<a href="mailto:jazeix@gmail.com" target="_blank">jazeix@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  
    
  
  <div bgcolor="#FFFFFF" text="#000000">
    <div>Hi,<br>
      <br>
      for now the search icon is part of the section bar.<br>
      <br>
      If we reduce it, it may be too small in portrait mode (the other
      icons are already small) or on tablet?<br>
      <br>
      I found a little bug: if you type a search (to filter activities),
      go to the config, press the close button, the  text filter is
      there but all the activities also (not only the filtered ones). If
      you press Escape, you have the text but only the filtered
      activities.<br>
      I displayed 'activity.inputText' in DialogActivityConfig.onClose
      and it is empty (tested with bith Qt5.5.1 and Qt5.7.0).<br>
      <br>
      Also, if virtual keyboard is visible and you resize the screen
      width (it works fine for the height), it disappears.<br>
      <br>
      Except these two issues, the code is clean for me :).<span class="HOEnZb"><font color="#888888"><br>
      <br>
      Johnny</font></span><div><div class="h5"><br>
      <br>
      On 07/19/16 13:44, Emmanuel Charruau wrote:<br>
    </div></div></div><div><div class="h5">
    <blockquote type="cite">
      <div dir="ltr">
        <div>
          <div>
            <div>
              <div>
                <div>
                  <div>
                    <div>Hi Rishab,<br>
                    </div>
                    I tested using words like multiplication color
                    weight line etc.<br>
                  </div>
                  Used virtual and real keyboard.<br>
                </div>
                Everything is working for me.<br>
              </div>
              I would have a design proposition.<br>
            </div>
            Could we not make the search icon a bit smaller to detach it
            a bit from the sections icons?<br>
          </div>
          I did a mockup to show you.<br>
          <br>
        </div>
        Emmanuel<br>
      </div>
      <div class="gmail_extra"><br>
        <div class="gmail_quote">2016-07-17 19:55 GMT+02:00 RISHABH
          GUPTA <span dir="ltr"><<a href="mailto:rishabh9511@gmail.com" target="_blank">rishabh9511@gmail.com</a>></span>:<br>
          <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
            <div dir="ltr">
              <div>
                <div>Hello all,<br>
                </div>
                <br>
                After a lot of days I got time to work on the search
                feature . The things that were previously left undone
                are now mostly done.A lot of things have been fixed.I
                have created a new pull request for the same.So,please
                review the changes and kindly make suggestions for
                things that you feel are wrong or are buggy.<br>
                <br>
              </div>
              <div>Here's the link of the pull request <a href="https://github.com/gcompris/GCompris-qt/pull/151" target="_blank">https://github.com/gcompris/GCompris-qt/pull/151</a><br>
                <br>
              </div>
              <div>Let's get the search feature out \m/<br>
              </div>
              <div>Thanks ,<br>
              </div>
              <div>Rishabh<br>
              </div>
            </div>
            <br>
            _______________________________________________<br>
            GCompris-devel mailing list<br>
            <a href="mailto:GCompris-devel@kde.org" target="_blank">GCompris-devel@kde.org</a><br>
            <a href="https://mail.kde.org/mailman/listinfo/gcompris-devel" rel="noreferrer" target="_blank">https://mail.kde.org/mailman/listinfo/gcompris-devel</a><br>
            <br>
          </blockquote>
        </div>
        <br>
      </div>
    </blockquote>
    <p><br>
    </p>
  </div></div></div>

</blockquote></div><br></div>