Closed Bug 953677 Opened 10 years ago Closed 10 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: 10 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.