sanyaade-g2g-repos / posit-mobile

Automatically exported from code.google.com/p/posit-mobile
GNU Lesser General Public License v2.1
0 stars 0 forks source link

Show Syncing icon in the title bar #238

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1.  Choose sync from ListFinds menu
2.
3.

What is the expected output? What do you see instead?

The app syncs the Finds but provides no visual feedback.  Since Syncing is done 
in a background thread, it doesn't make sense to show a progress bar. Instead, 
it should show the sync icon in the title bar -- which is how, e.g., gmail 
syncing is shown.

Please use labels and text to provide additional information.

Original issue reported on code.google.com by ram8...@gmail.com on 20 Oct 2011 at 3:06

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Implemented a SyncStatusObserver class to listen on active syncs in the 
ListFindsActivity. 

Added a sync icon on the title next to the "POSITx" string.

Original comment by stan.f...@gmail.com on 25 Oct 2011 at 5:13

Attachments:

GoogleCodeExporter commented 9 years ago
I tried incorporating this patch, but couldn't confirm that it worked 
correctly.  I think it needs further testing. Maybe we can do a code review of 
this. 

Original comment by ram8...@gmail.com on 25 Oct 2011 at 2:34

GoogleCodeExporter commented 9 years ago
Another thought.  When gmail is syncing, an icon appears not in the title bar 
but in the bar above it (name?).  Perhaps we should try to duplicate that 
behavior.

Original comment by ram8...@gmail.com on 25 Oct 2011 at 2:36

GoogleCodeExporter commented 9 years ago
I've worked on this using the Androids NotificationManager in the 
SyncAdapter.java. I have attached screenshots for what I have so far. 

I still need to clean up the code and finish the part that removes the 
notification. 

Original comment by stan.f...@gmail.com on 1 Nov 2011 at 8:26

Attachments:

GoogleCodeExporter commented 9 years ago
I think we should wait until you are able to remove the notification. 

Original comment by ram8...@gmail.com on 6 Dec 2011 at 2:42