App-like results (web results, local apps, marketplace) should be rendered within the same grid.

Amir - this is some work to follow the concepts we discussed in Paris. I've created an 'AppProvider' base class which can abstract some of the rendering logic from each provider which renders to the app grid. If you could give this one a quick review it would be appreciated. Thanks!

Looks great, much cleaner. Only 2 things I want to point out: 1. Each new result is appended separately and will trigger a repaint. Can the 'render' method support rendering a batch of results? 2. Clearing is now less efficient since we need to iterate over all the results. Can we keep the separate containers for each provider? Thanks.

(In reply to Amir Nissim (Everything.me) from comment #2) > Looks great, much cleaner. > > Only 2 things I want to point out: > 1. Each new result is appended separately and will trigger a repaint. > Can the 'render' method support rendering a batch of results? > 2. Clearing is now less efficient since we need to iterate over all the > results. > Can we keep the separate containers for each provider? Hi Amir, I've gone ahead and updated the pull request to address your concerns. I had thought that using multiple containers would be tricky, but turns out display:inline seems to work well. Please let me know if you have any further concerns.

Comment on attachment 8351581 [details] Github pull request perfect.

Landed: https://github.com/mozilla-b2g/gaia/commit/240aa40aad448ea26f482a96882841348a9d7840