ubuntu-touch-coreapps-reviewers team mailing list archive
-
ubuntu-touch-coreapps-reviewers team
-
Mailing list archive
-
Message #07033
[Merge] lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app
The proposal to merge lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app has been updated.
Description changed to:
* Use ListItemLayout for listitems to improve performance and match design guidelines
Running the analyse tool on this, this effectively removes what was done in the async loader before, it's hard to tell the exact gain. But it is around 5ms per delegate, which reduces the time by half from ~10ms to ~5ms.
You don't notice much of a gain on the mako as it was *just* hitting 60fps (needs to be less than 16ms), but on lower powered devices this will help.
For more details, see:
https://code.launchpad.net/~ahayzen/music-app/fix-1526274-use-layouts/+merge/281757
--
Your team Music App Developers is requested to review the proposed merge of lp:~ahayzen/music-app/fix-1526274-use-layouts into lp:music-app.
References