openlp-core team mailing list archive

Thread
Date

Re: [Merge] lp:~googol/openlp/search-placeholder into lp:openlp

To: mp+99279@xxxxxxxxxxxxxxxxxx
From: Raoul Snyman <raoul.snyman@xxxxxxxxxxxxxxxxxxxxxxxx>
Date: Thu, 29 Mar 2012 06:51:18 -0000
In-reply-to: <20120326100736.8082.92343.launchpad@ackee.canonical.com>
Reply-to: mp+99279@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

Review: Approve

Technically, PyQt 4.8 should support "setPlaceholderText" since it is supposed to be compatible with Qt 4.7. maybe one of the bugfix releases added support?

I remember trying to add this in about a year ago, but leaving it because 4.6 was the predominant version of Qt at the time.

I'm going to approve this, but I'd really like to see some more testing around which versions supports what in future.
-- 
https://code.launchpad.net/~googol/openlp/search-placeholder/+merge/99279
Your team OpenLP Core is subscribed to branch lp:openlp.

References

[Merge] lp:~googol/openlp/search-placeholder into lp:openlp
From: Andreas Preikschat, 2012-03-26