Add a review request hover preview.
Review Request #8508 — Created Oct. 30, 2016 and discarded
When the cursor moves over top of a review request in the dashboard a small
preview of the review request will be shown, similar to the user preview box.
Added 2 unit tests, checked infobox on dashboard, all review requests
and user pages.
Description | From | Last Updated |
---|---|---|
Missing "Args" and "Returns" sections in docstring. |
brennie | |
This is incorrect. This would have to be calc(@z-index-dialog + 1). However, this should be its own constant. |
brennie | |
These should probably be constants. |
brennie | |
This should be the first selector under .localtime. |
brennie | |
Where does this value come from? |
brennie | |
user, and |
brennie | |
Where is the corresponding {% spaceless %} start tag? |
brennie | |
Indentation is off. |
brennie | |
Need to change this so that it doesn't have to guess about location of href |
ST stensby | |
Try deleting lines 605 to 611 here to fix your hover problem. :) |
imadueme | |
unicode |
brennie | |
unicode |
brennie | |
Alphabetical order. |
brennie | |
Missing docstring. |
brennie | |
Testing docstrings can be of the form: """Testing some really_long_function does something specific """ |
brennie | |
"Display" |
brennie | |
"The infobox" over "this". |
brennie | |
arg (type): not arg: (type): |
brennie | |
int |
brennie | |
ID |
brennie | |
django.http.HttpResponse is the type. |
brennie | |
.classes before elements, so this should be ordered as: a {} h3 {} .infobox-extra {} .infobox-text {} |
brennie | |
Revert changes to this file |
ST stensby | |
Undo this. |
brennie | |
Can you remove the "right now" |
brennie | |
Is the whitespace here for a reason? |
brennie | |
Templates should be indented as with single spacing and the opening brace of block tags (e.g., {% if %}, etc.) … |
brennie |
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 2 (+309 -7) |
-
Tool: Pyflakes Processed Files: reviewboard/reviews/views.py reviewboard/extensions/hooks.py reviewboard/reviews/urls.py Ignored Files: reviewboard/static/rb/js/ui/views/infoboxView.es6.js reviewboard/templates/reviews/review_request_infobox.html reviewboard/static/rb/css/common.less reviewboard/static/rb/js/common.es6.js Tool: PEP8 Style Checker Processed Files: reviewboard/reviews/views.py reviewboard/extensions/hooks.py reviewboard/reviews/urls.py Ignored Files: reviewboard/static/rb/js/ui/views/infoboxView.es6.js reviewboard/templates/reviews/review_request_infobox.html reviewboard/static/rb/css/common.less reviewboard/static/rb/js/common.es6.js
-
-
-
-
reviewboard/static/rb/css/common.less (Diff revision 2) This is incorrect. This would have to be
calc(@z-index-dialog + 1)
.However, this should be its own constant.
-
-
reviewboard/static/rb/css/common.less (Diff revision 2) This should be the first selector under
.localtime
. -
-
-
reviewboard/templates/reviews/review_request_infobox.html (Diff revision 2) Where is the corresponding
{% spaceless %}
start tag? -
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 3 (+262 -7) |
-
Tool: Pyflakes Processed Files: reviewboard/reviews/views.py reviewboard/extensions/hooks.py reviewboard/reviews/urls.py Ignored Files: reviewboard/static/rb/js/ui/views/infoboxView.es6.js reviewboard/templates/reviews/review_request_infobox.html reviewboard/static/rb/css/common.less reviewboard/static/rb/js/common.es6.js Tool: PEP8 Style Checker Processed Files: reviewboard/reviews/views.py reviewboard/extensions/hooks.py reviewboard/reviews/urls.py Ignored Files: reviewboard/static/rb/js/ui/views/infoboxView.es6.js reviewboard/templates/reviews/review_request_infobox.html reviewboard/static/rb/css/common.less reviewboard/static/rb/js/common.es6.js
-
-
reviewboard/static/rb/js/ui/views/infoboxView.es6.js (Diff revision 3) Need to change this so that it doesn't have to guess about location of href
Change Summary:
Fixed infobox attachment point to match other infoboxes, continuing debug work on small pixel difference compared to user infobocx.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 4 (+267 -7) |
-
Tool: PEP8 Style Checker Processed Files: reviewboard/reviews/views.py reviewboard/extensions/hooks.py reviewboard/datagrids/columns.py reviewboard/reviews/urls.py Ignored Files: reviewboard/templates/reviews/review_request_infobox.html reviewboard/static/rb/js/common.es6.js reviewboard/static/rb/css/ui/datagrids.less reviewboard/static/rb/css/common.less reviewboard/static/rb/js/ui/views/infoboxView.es6.js reviewboard/templates/datagrids/summary_cell.html Tool: Pyflakes Processed Files: reviewboard/reviews/views.py reviewboard/extensions/hooks.py reviewboard/datagrids/columns.py reviewboard/reviews/urls.py Ignored Files: reviewboard/templates/reviews/review_request_infobox.html reviewboard/static/rb/js/common.es6.js reviewboard/static/rb/css/ui/datagrids.less reviewboard/static/rb/css/common.less reviewboard/static/rb/js/ui/views/infoboxView.es6.js reviewboard/templates/datagrids/summary_cell.html
-
-
reviewboard/static/rb/css/common.less (Diff revision 4) Try deleting lines 605 to 611 here to fix your hover problem. :)
Change Summary:
Added unit test.
Summary: |
|
||||||
---|---|---|---|---|---|---|---|
Testing Done: |
|
||||||
Commit: |
|
||||||
Diff: |
Revision 5 (+326 -8) |
-
Tool: Pyflakes Processed Files: reviewboard/reviews/views.py reviewboard/reviews/tests/test_views.py reviewboard/extensions/hooks.py reviewboard/datagrids/columns.py reviewboard/reviews/urls.py Ignored Files: reviewboard/templates/reviews/review_request_infobox.html reviewboard/static/rb/js/common.es6.js reviewboard/static/rb/css/ui/datagrids.less reviewboard/static/rb/css/common.less reviewboard/static/rb/js/ui/views/infoboxView.es6.js reviewboard/templates/datagrids/summary_cell.html Tool: PEP8 Style Checker Processed Files: reviewboard/reviews/views.py reviewboard/reviews/tests/test_views.py reviewboard/extensions/hooks.py reviewboard/datagrids/columns.py reviewboard/reviews/urls.py Ignored Files: reviewboard/templates/reviews/review_request_infobox.html reviewboard/static/rb/js/common.es6.js reviewboard/static/rb/css/ui/datagrids.less reviewboard/static/rb/css/common.less reviewboard/static/rb/js/ui/views/infoboxView.es6.js reviewboard/templates/datagrids/summary_cell.html
Change Summary:
Slight tweak to docstring
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 6 (+329 -8) |
-
Tool: Pyflakes Processed Files: reviewboard/reviews/views.py reviewboard/reviews/tests/test_views.py reviewboard/extensions/hooks.py reviewboard/datagrids/columns.py reviewboard/reviews/urls.py Ignored Files: reviewboard/templates/reviews/review_request_infobox.html reviewboard/static/rb/js/common.es6.js reviewboard/static/rb/css/ui/datagrids.less reviewboard/static/rb/css/common.less reviewboard/static/rb/js/ui/views/infoboxView.es6.js reviewboard/templates/datagrids/summary_cell.html Tool: PEP8 Style Checker Processed Files: reviewboard/reviews/views.py reviewboard/reviews/tests/test_views.py reviewboard/extensions/hooks.py reviewboard/datagrids/columns.py reviewboard/reviews/urls.py Ignored Files: reviewboard/templates/reviews/review_request_infobox.html reviewboard/static/rb/js/common.es6.js reviewboard/static/rb/css/ui/datagrids.less reviewboard/static/rb/css/common.less reviewboard/static/rb/js/ui/views/infoboxView.es6.js reviewboard/templates/datagrids/summary_cell.html
-
-
-
-
-
-
reviewboard/reviews/tests/test_views.py (Diff revision 6) Testing docstrings can be of the form:
"""Testing some really_long_function does something specific """
-
-
-
-
-
-
-
reviewboard/static/rb/css/common.less (Diff revision 6) .classes
beforeelements
, so this should be ordered as:a {} h3 {} .infobox-extra {} .infobox-text {}
-
-
reviewboard/static/rb/js/ui/views/infoboxView.es6.js (Diff revision 6) Can you remove the "right now"
-
reviewboard/templates/datagrids/summary_cell.html (Diff revision 6) Is the whitespace here for a reason?
-
reviewboard/templates/reviews/review_request_infobox.html (Diff revision 6) Templates should be indented as with single spacing and the opening brace of block tags (e.g.,
{% if %}
, etc.) should be left aligned, such as:<div class="foo"> <div class="bar"> {% if foo %} {% if bar %} bar {% endif %} {% endif %} </div> </div>
Change Summary:
Addressed Barret's feedback.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 7 (+323 -6) |
-
Tool: Pyflakes Processed Files: reviewboard/reviews/views.py reviewboard/reviews/tests/test_views.py reviewboard/extensions/hooks.py reviewboard/datagrids/columns.py reviewboard/reviews/urls.py Ignored Files: reviewboard/static/rb/js/ui/views/infoboxView.es6.js reviewboard/templates/datagrids/summary_cell.html reviewboard/templates/reviews/review_request_infobox.html reviewboard/static/rb/css/common.less reviewboard/static/rb/js/common.es6.js Tool: PEP8 Style Checker Processed Files: reviewboard/reviews/views.py reviewboard/reviews/tests/test_views.py reviewboard/extensions/hooks.py reviewboard/datagrids/columns.py reviewboard/reviews/urls.py Ignored Files: reviewboard/static/rb/js/ui/views/infoboxView.es6.js reviewboard/templates/datagrids/summary_cell.html reviewboard/templates/reviews/review_request_infobox.html reviewboard/static/rb/css/common.less reviewboard/static/rb/js/common.es6.js