openlp-core team mailing list archive

Thread
Date

Re: [Merge] lp:~felipe-q/openlp/better-remote into lp:openlp

To: Felipe Polo-Wood <felipe@xxxxxxxxxxxxx>
From: Tim Bentley <tim.bentley@xxxxxxxxx>
Date: Fri, 01 Nov 2013 17:15:29 -0000
In-reply-to: <20131101111224.24245.44870.launchpad@ackee.canonical.com>
Reply-to: mp+193576@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing

Next problem  serve_thumbnails.

Two issues here.  Direct access to presentations data structures from remote.  Plugins should be self contained.  Also the resizing in line.

We have imageManager which is a cache which should be used.  At present it resizes on screen size so that is an issue.  The thumbnails could be added to that when the presentation is added to the service then remote could read from cache.
 
-- 
https://code.launchpad.net/~felipe-q/openlp/better-remote/+merge/193576
Your team OpenLP Core is subscribed to branch lp:openlp.

References

[Merge] lp:~felipe-q/openlp/better-remote into lp:openlp
From: Felipe Polo-Wood, 2013-11-01