Closed Bug 436936 Opened 13 years ago Closed 12 years ago

Weave UI shows "Offline" in new windows

Categories

(Cloud Services :: General, defect, P1)

x86
Linux
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: dholbert, Assigned: avarma)

References

Details

Weave version: 0.1.30
Firefox version: Firefox 3 rc1

Steps to reproduce:
 1. Start firefox
 2. Sign in to Weave. (or, have firefox auto-login)
    --> Weave shows "Idle" in status bar
 3. Ctrl-N for new window.

EXPECTED RESULTS:
  Weave shows "Idle" in new window's status bar

ACTUAL RESULTS:
   Weave shows "Offline" in new window's status bar.  Also, if I click this icon, it gives me the "offline" menu, with the option to sign in. (though choosing 'sign in' here has no effect AFAICT -- it doesn't change the offline status shown.)
   Weave still shows "Idle" in original window, though, and the options shown for that window's weave menu make sense. (sign out, sync now, etc)

I think this may be a regression from bug 435103.
Confirmed that this bug exists in the current HG trunk.
Assignee: nobody → avarma
I've fixed the general case for this, but I'm not a big fan of it.  I think that the cleanest way to do this might be to have Weave.Service's state be completely queryable--its "currentUser" property can be inspected to see whether we're logged in or not, but there doesn't appear to be any way to tell whether the Weave service is currently busy doing something--and then add a single function to the browser window that sets the state of the UI based on the state of the Weave Service.

Another way to do this may be for the Weave service to simply "remember" what the last notification it sent was, and for new browser windows to simply ask for that when they're created and manually call observe() themselves.  But I'm not sure how dependent the methods called by observe() are on the previous state of Weave.
Status: NEW → ASSIGNED
Blocks: 433967
Priority: -- → P1
Target Milestone: -- → 0.2
This is working now, marking as fixed.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Component: Weave → General
Product: Mozilla Labs → Weave
QA Contact: weave → general
You need to log in before you can comment on or make changes to this bug.