Better linking in the UI Adding link icons to review comments.
Review Request #7254 — Created April 26, 2015 and discarded
Add icons to link to review comments and replies.
Manual testing:
- Clicking the link icon brings you to the link for the comment associated with it.
| Description | From | Last Updated |
|---|---|---|
|
This <li> should only be indented one space from the <ul>. Make sure to indent with spaces and not tabs. … |
 brennie
|
|
|
This change looks unintentional. |
 david
|
|
|
There's still an unintentional change to this file (you dedented this line by 1 space). |
david
|
|
|
This is duplicating the comment text. I think you just want to add the span/a/i next to the existing comment … |
david
|
-
Can you upload some screenshots for this change?
-
This
<li>should only be indented one space from the<ul>. Make sure to indent with spaces and not tabs.Also, can we use a Can you use a
<span>for the icon?Finally, can you give the
<span>atitle=attribute that describes the action of the icon (e.g., "Link to here"). Make sure to use i18n.
TE
- Commit:
-
9c9b398ccea9a5d11ace26de7c778fcf36bb68177c881125a8b4418caf6673ed357643d78537c6eb
- Diff:
-
Revision 2 (+663 -211)
-
Tool: Pyflakes Processed Files: reviewboard/diffviewer/chunk_generator.py reviewboard/accounts/pages.py reviewboard/reviews/templatetags/reviewtags.py reviewboard/diffviewer/tests.py reviewboard/diffviewer/diffutils.py reviewboard/staticbundles.py reviewboard/accounts/tests.py Ignored Files: reviewboard/templates/reviews/review_reply_section.html reviewboard/templates/reviews/table_comment_issue.html reviewboard/templates/reviews/boxes/review.html reviewboard/static/rb/js/views/commentIssueBarView.js reviewboard/static/rb/css/ui/datagrids.less reviewboard/static/rb/css/pages/reviews.less reviewboard/static/rb/js/views/issueSummaryTableView.js reviewboard/static/rb/js/views/headerView.js reviewboard/static/rb/js/views/tests/commentIssueBarViewTests.js reviewboard/static/rb/css/common.less reviewboard/templates/reviews/diff_comment_fragment.html reviewboard/templates/reviews/review_issue_summary_table.html reviewboard/static/rb/css/pages/dashboard.less Tool: PEP8 Style Checker Processed Files: reviewboard/diffviewer/chunk_generator.py reviewboard/accounts/pages.py reviewboard/reviews/templatetags/reviewtags.py reviewboard/diffviewer/tests.py reviewboard/diffviewer/diffutils.py reviewboard/staticbundles.py reviewboard/accounts/tests.py Ignored Files: reviewboard/templates/reviews/review_reply_section.html reviewboard/templates/reviews/table_comment_issue.html reviewboard/templates/reviews/boxes/review.html reviewboard/static/rb/js/views/commentIssueBarView.js reviewboard/static/rb/css/ui/datagrids.less reviewboard/static/rb/css/pages/reviews.less reviewboard/static/rb/js/views/issueSummaryTableView.js reviewboard/static/rb/js/views/headerView.js reviewboard/static/rb/js/views/tests/commentIssueBarViewTests.js reviewboard/static/rb/css/common.less reviewboard/templates/reviews/diff_comment_fragment.html reviewboard/templates/reviews/review_issue_summary_table.html reviewboard/static/rb/css/pages/dashboard.less
-
Tool: PEP8 Style Checker Ignored Files: reviewboard/templates/reviews/review_reply_section.html reviewboard/templates/reviews/boxes/review.html reviewboard/static/rb/css/pages/reviews.less Tool: Pyflakes Ignored Files: reviewboard/templates/reviews/review_reply_section.html reviewboard/templates/reviews/boxes/review.html reviewboard/static/rb/css/pages/reviews.less
-
Tool: PEP8 Style Checker Ignored Files: reviewboard/templates/reviews/review_reply_section.html reviewboard/templates/reviews/boxes/review.html reviewboard/static/rb/css/pages/reviews.less Tool: Pyflakes Ignored Files: reviewboard/templates/reviews/review_reply_section.html reviewboard/templates/reviews/boxes/review.html reviewboard/static/rb/css/pages/reviews.less
-
Tool: Pyflakes Ignored Files: reviewboard/templates/reviews/boxes/review.html reviewboard/static/rb/css/pages/reviews.less Tool: PEP8 Style Checker Ignored Files: reviewboard/templates/reviews/boxes/review.html reviewboard/static/rb/css/pages/reviews.less
-
Tool: Pyflakes Ignored Files: reviewboard/templates/reviews/boxes/review.html reviewboard/static/rb/css/pages/reviews.less Tool: PEP8 Style Checker Ignored Files: reviewboard/templates/reviews/boxes/review.html reviewboard/static/rb/css/pages/reviews.less
-
So the code is looking better now. I notice that you still have "WIP" in the summary--is there more work that you were planning on doing for this?
Tool: Pyflakes Ignored Files: reviewboard/templates/reviews/review_reply_section.html reviewboard/static/rb/css/pages/reviews.less Tool: PEP8 Style Checker Ignored Files: reviewboard/templates/reviews/review_reply_section.html reviewboard/static/rb/css/pages/reviews.less