Comment 5 for bug 549643

Revision history for this message
Benoit Garret (benoit.garret) wrote :

Much better, I can now at least see that something is happening.

I took a quick look at your animation code, but couldn't spot any obvious mistake. I'm afraid I won't be of much help here.

I'm against removing the sync menu icon for two reasons:

 * Your bar is only actionable through a touchscreen. My understanding is that it's important for accessibility reasons that the sync can be initiated by a hardware key. I'm not a usability or accessibility expert, if any lurker has experience in any of those fields, their opinion will be much appreciated.

 * I prefer to follow the guidelines and tools provided by Google concerning interface design, there's a whole chapter on the development guide dedicated to them.

Feel free to disagree though ;-) . I may not have completely understood your reasons and I would gladly be proven wrong.

One last thing, I'm curious about the reasons that led you to use a custom notification. There are at least two feedback mechanisms that are already implemented by system applications:

 * Use the progress bar in the window title : http://developer.android.com/guide/appendix/faq/commontasks.html#progressbar . It is already used by the web browser and looks like super easy to implement.

 * Use a status bar notification: http://developer.android.com/guide/topics/ui/notifiers/notifications.html . Something along the lines of what the market app does, it requires a bit more work than the previous one but has the advantage of notifying the user even when the application is hidden.

I'm not trying to downplay your work, I'm really, really glad that you stepped up to get in charge of this. I'm just trying to get an understanding of the reasons that led you to your UI choices.