Modify

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#6469 closed enhancement (fixed)

Add Stopwatch To TandE plugin

Reported by: Russ Tyndall Owned by: Russ Tyndall
Priority: normal Component: TimingAndEstimationPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

Tay Ray Chuan has some work completed to add a stopwatch to the ticket interface. Pull this in to the main plugin branches and test.

Code at: http://github.com/rctay/tae-stopwatch

Attachments (0)

Change History (9)

comment:1 Changed 14 years ago by Ryan J Ollos

Type: defectenhancement

comment:2 Changed 14 years ago by rctay

Hi,

I've hooked it as 'TicketStopwatch' onto the timingandestimationplugin package.

If you want to test, you can get a fully-functional version of timingandestimationplugin with the stopwatch from the branch 'master' (tarball).

If you want to import it over your existing svn repo, I've an import-friendly version containing only the files that are new and/or touched at the branch 'export' (tarball)

comment:3 Changed 14 years ago by Russ Tyndall

(In [7435]) Merged in stopwatch changes to trac0.11 branch re #6469

comment:4 Changed 14 years ago by Russ Tyndall

(In [7436]) merged stopwatch into trac0.11-Permissions branch re #6469

comment:5 Changed 14 years ago by Russ Tyndall

Resolution: fixed
Status: newclosed

Thanks again for your work benefiting us all. I have added you to the authors lists.

It would be really great if the stopwatch persisted its state with a few ajax requests so that a browser crash doesn't lose the stopwatch time you are trying to preserve. Obviously this would have to be based per ticket so as not to have multiple tickets quash each other.

comment:6 Changed 14 years ago by Russ Tyndall

(In [7437]) added author note re #6469

comment:7 Changed 14 years ago by Russ Tyndall

Also I just noticed that you are just counting up.

With javascript being cooperatively multithreaded and often times sharing a single process across tabs, you cannot assume that your interval will actually fire every x ms as specified. You should instead store the start time and then calculate how much time has elapsed since then and update your display accordingly. It seems like you are just counting up which could cause your times to be very much under the amount of time actually spent.

comment:8 in reply to:  5 ; Changed 14 years ago by Russ Tyndall

Replying to bobbysmith007:

It would be really great if the stopwatch persisted its state with a few ajax requests so that a browser crash doesn't lose the stopwatch time you are trying to preserve. Obviously this would have to be based per ticket so as not to have multiple tickets quash each other.

The time would also need to be stored per user so that two separate people who start a stopwatch on the same ticket can record their time separately

comment:9 in reply to:  8 Changed 14 years ago by rctay

Replying to bobbysmith007:

Replying to bobbysmith007:

It would be really great if the stopwatch persisted its state with a few ajax requests so that a browser crash doesn't lose the stopwatch time you are trying to preserve. Obviously this would have to be based per ticket so as not to have multiple tickets quash each other.

The time would also need to be stored per user so that two separate people who start a stopwatch on the same ticket can record their time separately

This is a pretty cool feature - I'll probably look into it sometime.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Russ Tyndall.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.