Merge pull request #501 from mapbox/feature/review-user-link

Add link to review user on changeset list

Author: willemarcel

src/components/list/row.js

@@ -6,6 +6,7 @@ import { Map } from 'immutable';
 import { SecondaryLine } from './secondary_line';
 import { PrimaryLine } from './primary_line';
 import { Title } from './title';
+import { history } from '../../store/history';
 
 export class Row extends React.PureComponent {
   props: {
@@ -42,13 +43,17 @@ export class Row extends React.PureComponent {
 
     backgroundClass += active
       ? 'light-blue'
-      : this.wasOpen ? ' bg-darken5 ' : '';
+      : this.wasOpen
+      ? ' bg-darken5 '
+      : '';
     return (
-      <Link
-        to={{
-          search: window.location.search,
-          pathname: `/changesets/${changesetId}`
-        }}
+      <div
+        onClick={() =>
+          history.push({
+            search: window.location.search,
+            pathname: `/changesets/${changesetId}`
+          })
+        }
       >
         <div className={`${backgroundClass} ${borderClass}`} ref={inputRef}>
           <div
@@ -58,20 +63,27 @@ export class Row extends React.PureComponent {
             }
           >
             <div className="flex-parent flex-parent--column">
-              <div>
-                <Title
-                  properties={properties}
-                  wasOpen={this.wasOpen}
-                  date={properties.get('date')}
-                />
-              </div>
-              <div>
-                <PrimaryLine
-                  reasons={properties.get('reasons')}
-                  tags={properties.get('tags')}
-                  comment={properties.get('comment')}
-                />
-              </div>
+              <Link
+                to={{
+                  search: window.location.search,
+                  pathname: `/changesets/${changesetId}`
+                }}
+              >
+                <div>
+                  <Title
+                    properties={properties}
+                    wasOpen={this.wasOpen}
+                    date={properties.get('date')}
+                  />
+                </div>
+                <div>
+                  <PrimaryLine
+                    reasons={properties.get('reasons')}
+                    tags={properties.get('tags')}
+                    comment={properties.get('comment')}
+                  />
+                </div>
+              </Link>
               <div>
                 <SecondaryLine
                   changesetId={changesetId}
@@ -82,7 +94,7 @@ export class Row extends React.PureComponent {
             </div>
           </div>
         </div>
-      </Link>
+      </div>
     );
   }
 }

src/components/list/secondary_line.js

@@ -1,7 +1,10 @@
 // @flow
 import React from 'react';
+import { Link } from 'react-router-dom';
+
 import { CreateDeleteModify } from '../create_delete_modify';
 import { NumberOfComments } from './comments';
+import { getObjAsQueryParam } from '../../utils/query_params';
 
 import thumbsUp from '../../assets/thumbs-up.svg';
 import thumbsDown from '../../assets/thumbs-down.svg';
@@ -10,9 +13,30 @@ export function SecondaryLine({ changesetId, date, properties }: Object) {
   return (
     <span className="flex-parent flex-parent--row justify--space-between txt-light txt-s color-gray">
       <span>
-        <span className="mr6">{changesetId}</span>
+        <Link
+          to={{
+            search: window.location.search,
+            pathname: `/changesets/${changesetId}`
+          }}
+        >
+          <span className="mr6">{changesetId}</span>
+        </Link>
         {properties.get('checked') ? (
-          <span>
+          <Link
+            to={{
+              search: getObjAsQueryParam('filters', {
+                users: [
+                  {
+                    label: properties.get('check_user'),
+                    value: properties.get('check_user')
+                  }
+                ],
+                date__gte: [{ label: '', value: '' }]
+              }),
+              pathname: '/'
+            }}
+            title={`See ${properties.get('check_user')}'s changesets`}
+          >
             {properties.get('harmful') ? (
               <img
                 src={thumbsDown}
@@ -31,7 +55,7 @@ export function SecondaryLine({ changesetId, date, properties }: Object) {
                 'check_user'
               )}`}</span>
             )}
-          </span>
+          </Link>
         ) : null}
       </span>
       <span className="flex-parent flex-parent--row">