← Back to team overview

openlp-core team mailing list archive

  1. openlp-core team
  2. Mailing list archive
  3. Message #08785

Re: [Merge] lp:~j-corwin/openlp/general into lp:openlp

  Thread Previous • Date Previous • • Thread Next 
Review: Approve
Looks OK to me, but why don't you use the jQuery $.each() function to loop, rather than the for (), it's a bit easier to manage.
-- 
https://code.launchpad.net/~j-corwin/openlp/general/+merge/60707
Your team OpenLP Core is subscribed to branch lp:openlp.

References

  Thread Previous • Date Previous • • Thread Next