Closed Bug 953677 Opened 11 years ago Closed 11 years ago

Replace "connected" by "connected for XX minutes"

Categories

(Instantbird Graveyard :: Account manager, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: romain, Assigned: romain)

References

Details

Attachments

(1 file, 2 obsolete files)

*** Original post on bio 231 at 2009-08-30 19:35:00 UTC *** When an account is connected, it would be interesting to know for how long. flo and I suggest that we replace the "Connected" Label in the account manager by "Connected for (a few seconds|XX Minutes|XX Hours|XX Days)." This bug depends on bug 953675 (bio 229) (for the last last connected timestamp exposure).
Blocks: 953623
Depends on: 953675
Attached patch Patch V1 (obsolete) — Splinter Review
*** Original post on bio 231 as attmnt 232 at 2009-09-01 20:35:00 UTC *** Also fixed a few missing semicolons. [0; 29] => Connected for a few seconds. [30; 89] => Connected for about 1 minute. others: - Connected for about 1 day. - Connected for about 42 days and 4 hours. The value is refreshed at most every 60 seconds (can be refreshed if a "connected" notification appears).
Attachment #8351976 - Flags: review?(florian)
Assignee: nobody → romain
Status: NEW → ASSIGNED
*** Original post on bio 231 at 2009-09-02 15:38:08 UTC *** Comment on attachment 8351976 [details] [diff] [review] (bio-attmnt 232) Patch V1 >diff --git a/instantbird/base/content/instantbird/account.xml b/instantbird/base/content/instantbird/account.xml >+ <method name="refreshConnectedLabel"> >+ <body> >+ <![CDATA[ >+ var bundle = document.getElementById("accountsBundle"); >+ var date = >+ 60 * Math.round((Date.now() - this._account.timeOfLastConnect) / 60000); >+ let value; >+ if (date > 0) { I think I'd keep "A few seconds" until we reach one minute, rather than rounding. "1 minute" for 30 seconds seems more false than "a few seconds" for 59 seconds. >+ value = bundle.getFormattedString("account.connectedInSingle", >+ value = bundle.getFormattedString("account.connectedInDouble", >+ value = bundle.getString("account.connectedForSeconds"); connectedIn or connectedFor? Looks good otherwise.
Attached patch Patch V2 (obsolete) — Splinter Review
*** Original post on bio 231 as attmnt 233 at 2009-09-02 16:12:00 UTC *** - Using Math.floor() instead of Math.round(). - changing "ConnectedForSeconds" to "ConnectedInSeconds"
Attachment #8351977 - Flags: review?(florian)
Comment on attachment 8351976 [details] [diff] [review] Patch V1 *** Original change on bio 231 attmnt 232 at 2009-09-02 16:12:19 UTC was without comment, so any subsequent comment numbers will be shifted ***
Attachment #8351976 - Attachment is obsolete: true
Attachment #8351976 - Flags: review?(florian)
Attached patch Patch V3Splinter Review
*** Original post on bio 231 as attmnt 234 at 2009-09-02 17:34:00 UTC was without comment, so any subsequent comment numbers will be shifted ***
Attachment #8351978 - Flags: review?(florian)
Comment on attachment 8351977 [details] [diff] [review] Patch V2 *** Original change on bio 231 attmnt 233 at 2009-09-02 17:34:02 UTC was without comment, so any subsequent comment numbers will be shifted ***
Attachment #8351977 - Attachment is obsolete: true
Attachment #8351977 - Flags: review?(florian)
*** Original post on bio 231 at 2009-09-02 21:29:48 UTC *** http://hg.instantbird.org/instantbird/rev/13dcfc8be03e
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Depends on: 953708
Comment on attachment 8351978 [details] [diff] [review] Patch V3 *** Original change on bio 231 attmnt 234 at 2009-11-17 20:33:21 UTC was without comment, so any subsequent comment numbers will be shifted ***
Attachment #8351978 - Flags: review?(florian) → review+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: