Closed Bug 482900 Opened 14 years ago Closed 14 years ago

Provide a friendlier UI to choose which ways to sync data

Categories

(Cloud Services :: General, defect, P1)

Product:
Cloud Services
Component:
General
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: Mardak, Assigned: Mardak)

References

Details

Attachments

(2 files, 1 obsolete file)

screenshot of v1
45.79 KB, image/png
Details
v1.1
19.30 KB, patch
hello
: review+
Details | Diff | Splinter Review 
Bug 468691 provides an initial exposure to controlling data syncing, but we'll want something like bug 480057 comment 6 down the line -- a separate dialog triggered by clicking "Sync Now..."

Not sure if this bug includes beautifying the dialog with images, etc.; but at least initially we can get some text/ascii version.
Depends on: 482898
Priority: -- → P1
Target Milestone: -- → 0.3
Attached patch v1 (obsolete) — — Splinter Review 
Initial implementation as a separate dialog. I'll try making it part of the existing sync status dialog to avoid opening/closing multiple modal dialogs.
Assignee: nobody → edilee
Status: NEW → ASSIGNED
Attached image screenshot of v1 — 

    
    

    
  

      
      
      
      

        Attached patch
          
          
        v1.1
        — 
          — Splinter Review
      

    


  
I started merging the existing sync status dialog with this one, but then thought it was kinda futile to keep the "modal" dialog around. So now it'll just dismiss the dialog and run the sync in the background -- just like what we do for autologin + autosync.

I say "modal" because a modal sync dialog only prevent access to the rest of the preferences window -- not the rest of the browser. I did the same thing for server wipe and got rid of the other debugging buttons.

Should we just get rid of that modal dialog? It's now only used if you do status bar -> sync now.

        Attachment #368168 -
        Attachment is obsolete: true

        Attachment #368205 -
        Flags: review?(thunder)

    
    

    
  
Comment on attachment 368205 [details] [diff] [review]
v1.1

r+, definitely better than the current debug UI, but syncing should give you feedback (as we discussed in person earlier)

        Attachment #368205 -
        Flags: review?(thunder) → review+

    
    

    
  
http://hg.mozilla.org/labs/weave/rev/187c857350b9
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Component: Weave → General
Product: Mozilla Labs → Weave
Version: Trunk → unspecified

    
  
QA Contact: weave → general

          You need to log in
          before you can comment on or make changes to this bug.