Extensions for merge request !334

Suggested changes to the NowPlayingBottomSheet improvments introduced in merge request !334 (merged). There are five commits addressing two bugs and two UI improvements:

• c86f81b4 Do not create unnecessary Picasso objects.
• 9cd8826e Refactor CoverArt.withContext().
• ab3205c0 Prevent BottomSheet tap leaking to nav panels.
• eaf3c550 Increase player controls touchpoint size.
• b2490c61 Fix landscape view induced MainActivity leak.

The first two (c86f81b4 and 9cd8826e) fix the "Too many resources" crash.

The next two (ab3205c0 and eaf3c550) address minor UX issues.

The last (b2490c61) fixes a memory leak that crashes the app after ten or so orientation changes when landscape mode is enabled. Testing with landscape enabled uncovers issues not seen with the previous version of !310 (closed), described below, so it is left disabled.

While the last fix stabilizes the app when landscape mode is enabled, this is left disabled at present.

mentioned in merge request !331 (closed)

approved this merge request

unapproved this merge request

requested review from @christophehenry

@christophehenry Mind doing a review and approving the MR if it looks okay to you?

Looks good to me.

approved this merge request

added 8 commits

• b2490c61...4dba9e29 - 3 commits from branch develop
• f65e29af - Do not create unnecessary Picasso objects.
• feb86fe9 - Refactor CoverArt.withContext().
• 2133d4a4 - Prevent BottomSheet tap leaking to nav panels.
• b9ade479 - Increase player controls touchpoint size.
• c1eb9d6b - Fix landscape view induced MainActivity leak.

Compare with previous version

enabled an automatic merge when the pipeline for c1eb9d6b succeeds

merged

mentioned in issue #145 (closed)