On 23 November 2010 18:08, Denis Kenzior <denkenz(a)gmail.com> wrote:
On 11/23/2010 10:55 AM, Andrzej Zaborowski wrote:
> As for the clear after delay flag, there's a check for it in
> display_text_cb (line 1248). As far as I can make out the only
> difference is the response sent to the SIM.
So it has been a long time since those discussions, but it does seem to
me that using a higher timeout in the case of 'wait_for_user' flag seems
like a good idea. Do you remember why we left this out?
No, I don't, I'll send a patch to use the normal timeout instead of
short timeout when clearing after user action is requested. As for
the proposal to add a parameter clear_after_delay and the ScreenBusy
response I'm fine with both of them (as I don't think the saving from
hiding these details is worth it) but I leave it up to you and
Guillaume. It seems that a user action always clears the message but
it may be good to display a "confirm" button anyway. The ScreenBusy
response is also possible for other proactive commands so for
consistency would need to be added there too.
Best regards,
Andrew