Re: [trojita] periodic imap refresh change mouse cursor

2014-12-18 Thread Thomas Lübking
On Donnerstag, 18. Dezember 2014 17:14:43 CEST, Jan Kundrát wrote: Could you please push it to Gerrit? :) Afaiu, Pali just wanted a patch to try for himself - but: "sure" =) https://gerrit.vesnicky.cesnet.cz/r/251 Cheers, Thomas

Re: [trojita] periodic imap refresh change mouse cursor

2014-12-18 Thread Jan Kundrát
On Thursday, 18 December 2014 16:33:21 CEST, Thomas Lübking wrote: Notice that Jan terribly hates lazy members and thus likely local statics as well, I just wanted to keep the patch as local as possible. Could you please push it to Gerrit? :) J. -- Trojitá, a fast Qt IMAP e-mail client -- ht

Re: [trojita] periodic imap refresh change mouse cursor

2014-12-18 Thread Thomas Lübking
On Donnerstag, 18. Dezember 2014 10:42:04 CEST, Pali Rohár wrote: I do not know if 500ms is enough for me (when updating about 40 imap folders), but if you can provide easy/small patch I can test it what happens... It's not a very complex patch. Given your concerns and a personal flavor, I ra

Re: [trojita] periodic imap refresh change mouse cursor

2014-12-18 Thread Pali Rohár
On Thursday 18 December 2014 10:16:18 Jan Kundrát wrote: > On Thursday, 18 December 2014 00:11:56 CEST, Thomas Lübking wrote: > > My 2¢ > > a) it should be Qt::BusyCursor, not Qt::WaitCursor (the UI > > isn't locked, you can still successfully interact w/ it) > > b) "m_busyCursorTrigger->start(500

Re: [trojita] periodic imap refresh change mouse cursor

2014-12-18 Thread Jan Kundrát
On Thursday, 18 December 2014 00:11:56 CEST, Thomas Lübking wrote: My 2¢ a) it should be Qt::BusyCursor, not Qt::WaitCursor (the UI isn't locked, you can still successfully interact w/ it) b) "m_busyCursorTrigger->start(500);" - telling the user prominently about every minor task makes Trojitá