Closed
Bug 623144
Opened 14 years ago
Closed 13 years ago
Stop Using This Account (startOver) should clear error notification still shown
Categories
(Firefox :: Sync, defect)
Firefox
Sync
Tracking
()
RESOLVED
FIXED
People
(Reporter: philikon, Assigned: philikon)
References
Details
Attachments
(1 file)
955 bytes,
patch
|
mconnor
:
review+
mconnor
:
approval2.0+
|
Details | Diff | Splinter Review |
No description provided.
Comment 1•14 years ago
|
||
one-liner to add Notifications.removeAll() ?
Assignee | ||
Comment 2•14 years ago
|
||
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Attachment #504318 -
Flags: review?(mconnor)
Attachment #504318 -
Flags: approval2.0?
Comment 3•14 years ago
|
||
Comment on attachment 504318 [details] [diff] [review] v1 Do we want to do this here, or in the service? We should be able to call it from the service...
Assignee | ||
Comment 4•14 years ago
|
||
(In reply to comment #3) > Do we want to do this here, or in the service? We should be able to call it > from the service... So I was looking at this and and first thought about putting it in the service, but then I noticed how we have a nice separation between the service and all UI driving code (through observer notifications). I would like to keep it that way. That way we can also give that notifications code a comb-over (which it'll need to be less spammy, for instance) without touching service. tl;dr: preserving neat logic vs. UI code separation that we already have.
Comment 5•13 years ago
|
||
Comment on attachment 504318 [details] [diff] [review] v1 Okay, good enough for now. We need to fix notifications to not be a disaster, but that can be post Fx4.
Attachment #504318 -
Flags: review?(mconnor)
Attachment #504318 -
Flags: review+
Attachment #504318 -
Flags: approval2.0?
Attachment #504318 -
Flags: approval2.0+
Assignee | ||
Comment 6•13 years ago
|
||
http://hg.mozilla.org/mozilla-central/rev/82544a4cfb63
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Component: Firefox Sync: UI → Sync
Product: Cloud Services → Firefox
You need to log in
before you can comment on or make changes to this bug.
Description
•