Add a review stopwatch extension.
Review Request #7429 — Created June 15, 2015 and submitted
This adds a new extension, the review stopwatch. The goal here is for reviewers
to be able to track the amount of time that they spend reviewing a change, and
have that information available in the API.The stopwatch UI is shown in the lower-right corner of the page. Clicking it
will toggle on and off the timer. The resulting total seconds spent reviewing
is stored in the review extra_data field.The one somewhat confusing thing here is that if a user opens up multiple tabs
related to the same review request (say, review request details, diff viewer,
and a couple file attachments), there will be timers on each page. If these are
run independently, they'll step on each other. We'll just have to document how
this should be used and hope that it doesn't become an issue.
- Tested that toggle on/off worked correctly.
- Tested that the current time was saved to the review's extra_data when
toggling off. - Tested display of the time in the stopwatch overlay, review dialog, and
review boxes for various different values. - Tested that the display in the review dialog updated when the stopwatch was
still running. - Tested that publishing or discarding the review stopped the stopwatch
correctly, and (in the case of publish), saved correctly. - Tested onbeforeunload handler.
| Description | From | Last Updated |
|---|---|---|
|
Some suggestions for the look: An off-black for the border (maybe #333) would look better. Maybe add a background that … |
 chipx86
|
|
|
'settings' imported but unused |
 reviewbot
|
|
|
'patterns' imported but unused |
reviewbot
|
|
|
'include' imported but unused |
reviewbot
|
|
|
This file can be blank, no? |
 brennie
|
|
|
Same with this one. |
brennie
|
- Change Summary:
-
pep8
- Commit:
-
24ea9447672fb6d64d995f5aeea8840fb527c73281237584e2fb0a62c3f579e74174fa5f8c4f1aeb
- Diff:
-
Revision 2 (+440)
-
Tool: PEP8 Style Checker Processed Files: rbstopwatch/rbstopwatch/templatetags/stopwatch_tags.py rbstopwatch/rbstopwatch/templatetags/__init__.py rbstopwatch/setup.py rbstopwatch/rbstopwatch/extension.py rbstopwatch/rbstopwatch/__init__.py Ignored Files: rbstopwatch/rbstopwatch/static/js/stopwatch.js rbstopwatch/rbstopwatch/static/css/stopwatch.less rbstopwatch/rbstopwatch/templates/rbstopwatch-review-header.html Tool: Pyflakes Processed Files: rbstopwatch/rbstopwatch/templatetags/stopwatch_tags.py rbstopwatch/rbstopwatch/templatetags/__init__.py rbstopwatch/setup.py rbstopwatch/rbstopwatch/extension.py rbstopwatch/rbstopwatch/__init__.py Ignored Files: rbstopwatch/rbstopwatch/static/js/stopwatch.js rbstopwatch/rbstopwatch/static/css/stopwatch.less rbstopwatch/rbstopwatch/templates/rbstopwatch-review-header.html
- Change Summary:
-
Add a LICENSE and README.md
- Commit:
-
81237584e2fb0a62c3f579e74174fa5f8c4f1aebca1027ad76ad9baebc175dce2e433835cd7c07ac
- Diff:
-
Revision 3 (+507)
-
Tool: PEP8 Style Checker Processed Files: rbstopwatch/rbstopwatch/templatetags/stopwatch_tags.py rbstopwatch/rbstopwatch/templatetags/__init__.py rbstopwatch/setup.py rbstopwatch/rbstopwatch/extension.py rbstopwatch/rbstopwatch/__init__.py Ignored Files: rbstopwatch/rbstopwatch/templates/rbstopwatch-review-header.html rbstopwatch/LICENSE rbstopwatch/README.md rbstopwatch/rbstopwatch/static/css/stopwatch.less rbstopwatch/rbstopwatch/static/js/stopwatch.js Tool: Pyflakes Processed Files: rbstopwatch/rbstopwatch/templatetags/stopwatch_tags.py rbstopwatch/rbstopwatch/templatetags/__init__.py rbstopwatch/setup.py rbstopwatch/rbstopwatch/extension.py rbstopwatch/rbstopwatch/__init__.py Ignored Files: rbstopwatch/rbstopwatch/templates/rbstopwatch-review-header.html rbstopwatch/LICENSE rbstopwatch/README.md rbstopwatch/rbstopwatch/static/css/stopwatch.less rbstopwatch/rbstopwatch/static/js/stopwatch.js
-
-
Some suggestions for the look:
- An off-black for the border (maybe
#333) would look better. - Maybe add a background that helps separate it out from the other content just a bit. A light green/yellow (maybe similar to diff colors) for running/paused would be neat.
- Maybe a box shadow too, like the new update bubbles have, just to help with the pop and separation from other content.
- An off-black for the border (maybe
- Commit:
-
ca1027ad76ad9baebc175dce2e433835cd7c07ac177e154f0ee0fb65ae1695cb3b93d1187366a38e
- Diff:
-
Revision 4 (+510)
- Added Files:
-
Tool: PEP8 Style Checker Processed Files: rbstopwatch/rbstopwatch/templatetags/stopwatch_tags.py rbstopwatch/setup.py rbstopwatch/rbstopwatch/extension.py Ignored Files: rbstopwatch/rbstopwatch/__init__.py rbstopwatch/LICENSE rbstopwatch/rbstopwatch/static/js/stopwatch.js rbstopwatch/rbstopwatch/static/css/stopwatch.less rbstopwatch/rbstopwatch/templatetags/__init__.py rbstopwatch/README.md rbstopwatch/rbstopwatch/templates/rbstopwatch-review-header.html Tool: Pyflakes Processed Files: rbstopwatch/rbstopwatch/templatetags/stopwatch_tags.py rbstopwatch/setup.py rbstopwatch/rbstopwatch/extension.py Ignored Files: rbstopwatch/rbstopwatch/__init__.py rbstopwatch/LICENSE rbstopwatch/rbstopwatch/static/js/stopwatch.js rbstopwatch/rbstopwatch/static/css/stopwatch.less rbstopwatch/rbstopwatch/templatetags/__init__.py rbstopwatch/README.md rbstopwatch/rbstopwatch/templates/rbstopwatch-review-header.html
- Change Summary:
-
A couple trivial updates to formatting and code style.
- Commit:
-
177e154f0ee0fb65ae1695cb3b93d1187366a38e9ad612ae8d4fbfd8811c7cd821ba0be9a0db46e3
- Diff:
-
Revision 5 (+509)
-
Tool: PEP8 Style Checker Processed Files: rbstopwatch/rbstopwatch/templatetags/stopwatch_tags.py rbstopwatch/setup.py rbstopwatch/rbstopwatch/extension.py Ignored Files: rbstopwatch/rbstopwatch/__init__.py rbstopwatch/LICENSE rbstopwatch/rbstopwatch/static/js/stopwatch.js rbstopwatch/rbstopwatch/static/css/stopwatch.less rbstopwatch/rbstopwatch/templatetags/__init__.py rbstopwatch/README.md rbstopwatch/rbstopwatch/templates/rbstopwatch-review-header.html Tool: Pyflakes Processed Files: rbstopwatch/rbstopwatch/templatetags/stopwatch_tags.py rbstopwatch/setup.py rbstopwatch/rbstopwatch/extension.py Ignored Files: rbstopwatch/rbstopwatch/__init__.py rbstopwatch/LICENSE rbstopwatch/rbstopwatch/static/js/stopwatch.js rbstopwatch/rbstopwatch/static/css/stopwatch.less rbstopwatch/rbstopwatch/templatetags/__init__.py rbstopwatch/README.md rbstopwatch/rbstopwatch/templates/rbstopwatch-review-header.html
Tool: Pyflakes Processed Files: rbstopwatch/rbstopwatch/templatetags/stopwatch_tags.py rbstopwatch/rbstopwatch/templatetags/__init__.py rbstopwatch/setup.py rbstopwatch/rbstopwatch/extension.py rbstopwatch/rbstopwatch/__init__.py Ignored Files: rbstopwatch/rbstopwatch/static/js/stopwatch.js rbstopwatch/rbstopwatch/static/css/stopwatch.less rbstopwatch/rbstopwatch/templates/rbstopwatch-review-header.html Tool: PEP8 Style Checker Processed Files: rbstopwatch/rbstopwatch/templatetags/stopwatch_tags.py rbstopwatch/rbstopwatch/templatetags/__init__.py rbstopwatch/setup.py rbstopwatch/rbstopwatch/extension.py rbstopwatch/rbstopwatch/__init__.py Ignored Files: rbstopwatch/rbstopwatch/static/js/stopwatch.js rbstopwatch/rbstopwatch/static/css/stopwatch.less rbstopwatch/rbstopwatch/templates/rbstopwatch-review-header.html