← Back to team overview

openlp-core team mailing list archive

Thread
Date

Re: [Merge] lp:~trb143/openlp/general into lp:openlp

To: mp+54979@xxxxxxxxxxxxxxxxxx
From: Andreas Preikschat <andreaspreikschat@xxxxxxxxxx>
Date: Sat, 26 Mar 2011 20:01:27 -0000
In-reply-to: <20110326195548.6224.90478.launchpad@loganberry.canonical.com>
Reply-to: mp+54979@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Needs Fixing
Don't you want to use QTimeEdit instead of QSpinBox?

If you don't want to, then we need to make sure, that the second and minutes cannot be higher than 59, and that the start time cannot be set to a later timer than the end time.
-- 
https://code.launchpad.net/~trb143/openlp/general/+merge/54979
Your team OpenLP Core is subscribed to branch lp:openlp.

References

[Merge] lp:~trb143/openlp/general into lp:openlp
From: Tim Bentley, 2011-03-26